#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