comment lines in Cmm outputs
Ben Gamari
ben at smart-cactus.org
Sun May 29 15:14:52 UTC 2016
Ömer Sinan Ağacan <omeragacan at gmail.com> writes:
> I'm reading a lot of Cmm these days and comments added by Cmm dump (which are
> apparently added after 8.0.1) are so annoying becuase they're not saying
> anything useful (what's the point of adding "// CmmCall" to a "call" line or
> "// CmmCondBranch" to a "if" line?) but making a lot of noise. Why were those
> added? Can we remove them?
It looks like they've been with us since the initial merge of the new
codegen. I agree that they don't add much value but I don't have any
strong opinion either way. The only comment that we might want to
retain is the one for CmmUnsafeForeignCall as it looks quite similar to
a CmmCall.
Cheers,
- Ben
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 472 bytes
Desc: not available
URL: <http://mail.haskell.org/pipermail/ghc-devs/attachments/20160529/88baf616/attachment.sig>
More information about the ghc-devs
mailing list