[GHC] #11028: Refactor ConDecl

[GHC]

#11028: Refactor ConDecl
-------------------------------------+-------------------------------------
        Reporter:  simonpj           |                Owner:  alanz
            Type:  bug               |               Status:  patch
        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 simonpj):

 The old one is clearly wrong when we have
 {{{
   cml_true, cml_false :: !Label
 }}}
 We could fix it to provide ''both'' a prefix signature ''and'' list the
 fields; and that is presumably what the old version was supposed to do.
 And the new form which looks like
 {{{
   CmmCondBranch :: -> CmmNode O C
     cml_true, cml_false :: !Label
     ...etc...
 }}}
 does look very weird.  `CmmCondBranch :: -> CmmNode O C` just isn't
 Haskell.

 Why not just print it out in the source syntax form:
 {{{
   CmmCondBranch :: { cml_true, cml_false :: ! Label
                    , ...etc...
                    } -> CmmNode O C
 }}}
 Admittedly that's more work, because it's not what Haddock does right now.
 But it would make more sense to users wouldn't it?

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