#7514 closed bug (fixed)

Rename cmmz flags

Reported by: thoughtpolice Owned by: simonpj
Priority: normal Milestone:
Component: Compiler Version: 7.6.1
Keywords: Cc:
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: None/Unknown Difficulty: Unknown
Test Case: Blocked By:
Blocking: Related Tickets:

Description

This patch renames the cmmz flags to cmm for consistency. It was originally posted here:

http://www.haskell.org/pipermail/cvs-ghc/2012-December/079222.html

I'm creating this ticket so it doesn't get lost.

Attachments (1)

rename-cmmz-flags.patch (12.9 KB) - added by thoughtpolice 16 months ago.
Patch to rename flags

Download all attachments as: .zip

Change History (4)

Changed 16 months ago by thoughtpolice

Patch to rename flags

comment:1 Changed 16 months ago by thoughtpolice

  • Status changed from new to patch

comment:2 Changed 16 months ago by simonpj

  • Difficulty set to Unknown
  • Owner set to simonpj

I'll do this.

comment:3 Changed 16 months ago by simonpj

  • Resolution set to fixed
  • Status changed from patch to closed

Done. Thanks Austin!

commit f6f881f09c1ac69be2ae1850cb3067459f623360
Author: Austin Seipp <mad.one@gmail.com>
Date:   Mon Dec 17 04:21:50 2012 -0600

    Rename all of the 'cmmz' flags and make them more consistent.
    
    There's only a single compiler backend now, so the 'z' suffix means
    nothing. Also, the flags were confusingly named ('cmm-foo' vs
    'foo-cmm',) and counter-intuitively, '-ddump-cmm' did not do at all what
    you expected since the new backend went live.
    
    Basically, all of the -ddump-cmmz-* flags are now -ddump-cmm-*. Some were
    renamed to be more consistent.
    
    This doesn't update the manual; it already mentions '-ddump-cmm' and
    that flag implies all the others anyway, which is probably what you
    want.
    
    Signed-off-by: Austin Seipp <mad.one@gmail.com>

 compiler/cmm/CmmPipeline.hs  |   37 +++++++++++++++----------------
 compiler/cmm/CmmProcPoint.hs |    2 +-
 compiler/main/DynFlags.hs    |   48 ++++++++++++++++++++---------------------
 compiler/main/HscMain.hs     |    8 +++---
 4 files changed, 46 insertions(+), 49 deletions(-)
Note: See TracTickets for help on using tickets.