[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