[GHC] #4064: SpecConstr broken for NOINLINE loops in 6.13

[GHC]

#4064: SpecConstr broken for NOINLINE loops in 6.13
--------------------------------------+-------------------------------------
  Reporter:  rl                       |          Owner:                  
      Type:  bug                      |         Status:  closed          
  Priority:  normal                   |      Milestone:                  
 Component:  Compiler                 |        Version:  6.13            
Resolution:  fixed                    |       Keywords:                  
Difficulty:                           |             Os:  Unknown/Multiple
  Testcase:                           |   Architecture:  Unknown/Multiple
   Failure:  Runtime performance bug  |  
--------------------------------------+-------------------------------------
Changes (by simonpj):

  * status:  new => closed
  * resolution:  => fixed


Comment:

 This patch stops `SpecConstr` doing anything for NOINLINE things
 {{{
 * Don't do SpecConstr on NOINLINE things (Trac #4064)

   Since the RULE from specialising gets the same Activation as
   the inlining for the Id itself there's no point in specialising
   a NOINLINE thing, because the rule will be permanently switched
   off.

   See Note [Transfer activation] in SpecConstr
   and Note [Auto-specialisation and RULES] in Specialise.

     M ./compiler/specialise/SpecConstr.lhs -1 +9
 }}}
 I think that's enough to close the ticket

 Simon

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