(F)SLIT -> (f)sLit in StaticFlags
[ghc-hetmet.git] / compiler / simplCore / FloatOut.lhs
index 1991531..d0a27de 100644 (file)
@@ -6,9 +6,14 @@
 ``Long-distance'' floating of bindings towards the top level.
 
 \begin{code}
-module FloatOut ( floatOutwards ) where
+{-# OPTIONS -w #-}
+-- The above warning supression flag is a temporary kludge.
+-- While working on this module you are encouraged to remove it and fix
+-- any warnings in the module. See
+--     http://hackage.haskell.org/trac/ghc/wiki/Commentary/CodingStyle#Warnings
+-- for details
 
-#include "HsVersions.h"
+module FloatOut ( floatOutwards ) where
 
 import CoreSyn
 import CoreUtils
@@ -24,6 +29,7 @@ import SetLevels      ( Level(..), LevelledExpr, LevelledBind,
 import UniqSupply       ( UniqSupply )
 import List            ( partition )
 import Outputable
+import FastString
 \end{code}
 
        -----------------
@@ -129,9 +135,9 @@ floatOutwards float_sws dflags us pgm
        let { (tlets, ntlets, lams) = get_stats (sum_stats fss) };
 
        dumpIfSet_dyn dflags Opt_D_dump_simpl_stats "FloatOut stats:"
-               (hcat [ int tlets,  ptext SLIT(" Lets floated to top level; "),
-                       int ntlets, ptext SLIT(" Lets floated elsewhere; from "),
-                       int lams,   ptext SLIT(" Lambda groups")]);
+               (hcat [ int tlets,  ptext (sLit " Lets floated to top level; "),
+                       int ntlets, ptext (sLit " Lets floated elsewhere; from "),
+                       int lams,   ptext (sLit " Lambda groups")]);
 
        endPass dflags float_msg  Opt_D_verbose_core2core (concat binds_s')
                        {- no specific flag for dumping float-out -} 
@@ -308,13 +314,11 @@ floatExpr lvl (Note note@(SCC cc) expr)
          = Rec [(binder, mkSCC dupd_cc rhs) | (binder, rhs) <- pairs]
 
 floatExpr lvl (Note InlineMe expr)     -- Other than SCCs
-  = case floatExpr InlineCtxt expr of { (fs, floating_defns, expr') ->
-       -- There can be some floating_defns, arising from
-       -- ordinary lets that were there all the time.  It seems
-       -- more efficient to test once here than to avoid putting
-       -- them into floating_defns (which would mean testing for
-       -- inlineCtxt  at every let)
-    (fs, [], Note InlineMe (install floating_defns expr')) }   -- See notes in SetLevels
+  = (zeroStats, [], Note InlineMe (unTag expr))
+       -- Do no floating at all inside INLINE. [_$_]
+       -- The SetLevels pass did not clone the bindings, so it's
+       -- unsafe to do any floating, even if we dump the results
+       -- inside the Note (which is what we used to do).
 
 floatExpr lvl (Note note expr) -- Other than SCCs
   = case (floatExpr lvl expr)    of { (fs, floating_defns, expr') ->
@@ -356,6 +360,22 @@ floatList f [] = (zeroStats, [], [])
 floatList f (a:as) = case f a           of { (fs_a,  binds_a,  b)  ->
                     case floatList f as of { (fs_as, binds_as, bs) ->
                     (fs_a `add_stats` fs_as, binds_a ++ binds_as, b:bs) }}
+
+unTagBndr :: TaggedBndr tag -> CoreBndr
+unTagBndr (TB b _) = b
+
+unTag :: TaggedExpr tag -> CoreExpr
+unTag (Var v)            = Var v
+unTag (Lit l)            = Lit l
+unTag (Type ty)   = Type ty
+unTag (Note n e)  = Note n (unTag e)
+unTag (App e1 e2) = App (unTag e1) (unTag e2)
+unTag (Lam b e)   = Lam (unTagBndr b) (unTag e)
+unTag (Cast e co) = Cast (unTag e) co
+unTag (Let (Rec prs) e)    = Let (Rec [(unTagBndr b,unTag r) | (b, r) <- prs]) (unTag e)
+unTag (Let (NonRec b r) e) = Let (NonRec (unTagBndr b) (unTag r)) (unTag e)
+unTag (Case e b ty alts)   = Case (unTag e) (unTagBndr b) ty
+                                 [(c, map unTagBndr bs, unTag r) | (c,bs,r) <- alts]
 \end{code}
 
 %************************************************************************