[GHC] #11028: Refactor ConDecl

GHC ghc-devs at haskell.org
Sat Dec 5 15:12:58 UTC 2015


#11028: Refactor ConDecl
-------------------------------------+-------------------------------------
        Reporter:  simonpj           |                Owner:  alanz
            Type:  bug               |               Status:  new
        Priority:  normal            |            Milestone:  8.0.1
       Component:  Compiler          |              Version:  7.10.2
      Resolution:                    |             Keywords:
Operating System:  Unknown/Multiple  |         Architecture:
                                     |  Unknown/Multiple
 Type of failure:  None/Unknown      |            Test Case:
      Blocked By:                    |             Blocking:
 Related Tickets:  #11155            |  Differential Rev(s):  Phab:D1558
       Wiki Page:                    |
-------------------------------------+-------------------------------------

Comment (by alanz):

 I am finishing off the haddock changes for this.

 The original documentation generated for `CmmNode` was
 [[Image(https://ghc.haskell.org/trac/ghc/attachment/ticket/11028/old-
 condecl.png)]]

 With the update from the patch this becomes
 [[Image(https://ghc.haskell.org/trac/ghc/attachment/ticket/11028/new-
 condecl.png)]]

 See in particular `CmmCondBranch`. In the original it gives the (supposed)
 prefix constructor version, followed by the fields. But because `cml_true`
 and `cml_false` are part of the same field declaration, only one instance
 of `!Label` appears.

 In the new one, only the record fields are listed at the moment.

 Question: Should a prefix type signature be given at all, for a record
 GADT ConDecl? Or should it be left as per the new image?

--
Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/11028#comment:40>
GHC <http://www.haskell.org/ghc/>
The Glasgow Haskell Compiler


More information about the ghc-tickets mailing list