[project @ 1999-06-08 16:46:44 by simonpj]
[ghc-hetmet.git] / ghc / compiler / coreSyn / CoreUnfold.lhs
index a42e659..39740c7 100644 (file)
-%\r
-% (c) The AQUA Project, Glasgow University, 1994-1998\r
-%\r
-\section[CoreUnfold]{Core-syntax unfoldings}\r
-\r
-Unfoldings (which can travel across module boundaries) are in Core\r
-syntax (namely @CoreExpr@s).\r
-\r
-The type @Unfolding@ sits ``above'' simply-Core-expressions\r
-unfoldings, capturing ``higher-level'' things we know about a binding,\r
-usually things that the simplifier found out (e.g., ``it's a\r
-literal'').  In the corner of a @CoreUnfolding@ unfolding, you will\r
-find, unsurprisingly, a Core expression.\r
-\r
-\begin{code}\r
-module CoreUnfold (\r
-       Unfolding(..), UnfoldingGuidance, -- types\r
-\r
-       noUnfolding, mkUnfolding, getUnfoldingTemplate,\r
-       isEvaldUnfolding, hasUnfolding,\r
-\r
-       couldBeSmallEnoughToInline, \r
-       certainlySmallEnoughToInline, \r
-       okToUnfoldInHiFile,\r
-\r
-       calcUnfoldingGuidance,\r
-\r
-       callSiteInline, blackListed\r
-    ) where\r
-\r
-#include "HsVersions.h"\r
-\r
-import CmdLineOpts     ( opt_UF_CreationThreshold,\r
-                         opt_UF_UseThreshold,\r
-                         opt_UF_ScrutConDiscount,\r
-                         opt_UF_FunAppDiscount,\r
-                         opt_UF_PrimArgDiscount,\r
-                         opt_UF_KeenessFactor,\r
-                         opt_UF_CheapOp, opt_UF_DearOp, opt_UF_NoRepLit,\r
-                         opt_UnfoldCasms, opt_PprStyle_Debug,\r
-                         opt_D_dump_inlinings\r
-                       )\r
-import CoreSyn\r
-import PprCore         ( pprCoreExpr )\r
-import OccurAnal       ( occurAnalyseGlobalExpr )\r
-import BinderInfo      ( )\r
-import CoreUtils       ( coreExprType, exprIsTrivial, mkFormSummary, whnfOrBottom,\r
-                         FormSummary(..) )\r
-import Id              ( Id, idType, idUnique, isId, \r
-                         getIdSpecialisation, getInlinePragma, getIdUnfolding\r
-                       )\r
-import VarSet\r
-import Const           ( Con(..), isLitLitLit, isWHNFCon )\r
-import PrimOp          ( PrimOp(..), primOpIsDupable )\r
-import IdInfo          ( ArityInfo(..), InlinePragInfo(..), OccInfo(..) )\r
-import TyCon           ( tyConFamilySize )\r
-import Type            ( splitAlgTyConApp_maybe, splitFunTy_maybe )\r
-import Const           ( isNoRepLit )\r
-import Unique          ( Unique, buildIdKey, augmentIdKey, runSTRepIdKey )\r
-import Maybes          ( maybeToBool )\r
-import Bag\r
-import Util            ( isIn, lengthExceeds )\r
-import Outputable\r
-\end{code}\r
-\r
-%************************************************************************\r
-%*                                                                     *\r
-\subsection{@Unfolding@ and @UnfoldingGuidance@ types}\r
-%*                                                                     *\r
-%************************************************************************\r
-\r
-\begin{code}\r
-data Unfolding\r
-  = NoUnfolding\r
-\r
-  | OtherCon [Con]             -- It ain't one of these\r
-                               -- (OtherCon xs) also indicates that something has been evaluated\r
-                               -- and hence there's no point in re-evaluating it.\r
-                               -- OtherCon [] is used even for non-data-type values\r
-                               -- to indicated evaluated-ness.  Notably:\r
-                               --      data C = C !(Int -> Int)\r
-                               --      case x of { C f -> ... }\r
-                               -- Here, f gets an OtherCon [] unfolding.\r
-\r
-  | CoreUnfolding                      -- An unfolding with redundant cached information\r
-               FormSummary             -- Tells whether the template is a WHNF or bottom\r
-               UnfoldingGuidance       -- Tells about the *size* of the template.\r
-               CoreExpr                -- Template; binder-info is correct\r
-\end{code}\r
-\r
-\begin{code}\r
-noUnfolding = NoUnfolding\r
-\r
-mkUnfolding expr\r
-  = let\r
-     -- strictness mangling (depends on there being no CSE)\r
-     ufg = calcUnfoldingGuidance opt_UF_CreationThreshold expr\r
-     occ = occurAnalyseGlobalExpr expr\r
-    in\r
-    CoreUnfolding (mkFormSummary expr) ufg occ\r
-\r
-getUnfoldingTemplate :: Unfolding -> CoreExpr\r
-getUnfoldingTemplate (CoreUnfolding _ _ expr) = expr\r
-getUnfoldingTemplate other = panic "getUnfoldingTemplate"\r
-\r
-isEvaldUnfolding :: Unfolding -> Bool\r
-isEvaldUnfolding (OtherCon _)                    = True\r
-isEvaldUnfolding (CoreUnfolding ValueForm _ expr) = True\r
-isEvaldUnfolding other                           = False\r
-\r
-hasUnfolding :: Unfolding -> Bool\r
-hasUnfolding NoUnfolding = False\r
-hasUnfolding other      = True\r
-\r
-data UnfoldingGuidance\r
-  = UnfoldNever\r
-  | UnfoldAlways               -- There is no "original" definition,\r
-                               -- so you'd better unfold.  Or: something\r
-                               -- so cheap to unfold (e.g., 1#) that\r
-                               -- you should do it absolutely always.\r
-\r
-  | UnfoldIfGoodArgs   Int     -- and "n" value args\r
-\r
-                       [Int]   -- Discount if the argument is evaluated.\r
-                               -- (i.e., a simplification will definitely\r
-                               -- be possible).  One elt of the list per *value* arg.\r
-\r
-                       Int     -- The "size" of the unfolding; to be elaborated\r
-                               -- later. ToDo\r
-\r
-                       Int     -- Scrutinee discount: the discount to substract if the thing is in\r
-                               -- a context (case (thing args) of ...),\r
-                               -- (where there are the right number of arguments.)\r
-\end{code}\r
-\r
-\begin{code}\r
-instance Outputable UnfoldingGuidance where\r
-    ppr UnfoldAlways    = ptext SLIT("ALWAYS")\r
-    ppr UnfoldNever    = ptext SLIT("NEVER")\r
-    ppr (UnfoldIfGoodArgs v cs size discount)\r
-      = hsep [ptext SLIT("IF_ARGS"), int v,\r
-              if null cs       -- always print *something*\r
-               then char 'X'\r
-               else hcat (map (text . show) cs),\r
-              int size,\r
-              int discount ]\r
-\end{code}\r
-\r
-\r
-%************************************************************************\r
-%*                                                                     *\r
-\subsection[calcUnfoldingGuidance]{Calculate ``unfolding guidance'' for an expression}\r
-%*                                                                     *\r
-%************************************************************************\r
-\r
-\begin{code}\r
-calcUnfoldingGuidance\r
-       :: Int                  -- bomb out if size gets bigger than this\r
-       -> CoreExpr             -- expression to look at\r
-       -> UnfoldingGuidance\r
-calcUnfoldingGuidance bOMB_OUT_SIZE expr\r
-  | exprIsTrivial expr         -- Often trivial expressions are never bound\r
-                               -- to an expression, but it can happen.  For\r
-                               -- example, the Id for a nullary constructor has\r
-                               -- a trivial expression as its unfolding, and\r
-                               -- we want to make sure that we always unfold it.\r
-  = UnfoldAlways\r
\r
-  | otherwise\r
-  = case collectBinders expr of { (binders, body) ->\r
-    let\r
-       val_binders = filter isId binders\r
-    in\r
-    case (sizeExpr bOMB_OUT_SIZE val_binders body) of\r
-\r
-      TooBig -> UnfoldNever\r
-\r
-      SizeIs size cased_args scrut_discount\r
-       -> UnfoldIfGoodArgs\r
-                       (length val_binders)\r
-                       (map discount_for val_binders)\r
-                       (I# size)\r
-                       (I# scrut_discount)\r
-       where        \r
-           discount_for b \r
-               | num_cases == 0 = 0\r
-               | is_fun_ty      = num_cases * opt_UF_FunAppDiscount\r
-               | is_data_ty     = num_cases * tyConFamilySize tycon * opt_UF_ScrutConDiscount\r
-               | otherwise      = num_cases * opt_UF_PrimArgDiscount\r
-               where\r
-                 num_cases           = foldlBag (\n b' -> if b==b' then n+1 else n) 0 cased_args\r
-                                       -- Count occurrences of b in cased_args\r
-                 arg_ty              = idType b\r
-                 is_fun_ty           = maybeToBool (splitFunTy_maybe arg_ty)\r
-                 (is_data_ty, tycon) = case (splitAlgTyConApp_maybe (idType b)) of\r
-                                         Nothing       -> (False, panic "discount")\r
-                                         Just (tc,_,_) -> (True,  tc)\r
-       }\r
-\end{code}\r
-\r
-\begin{code}\r
-sizeExpr :: Int            -- Bomb out if it gets bigger than this\r
-        -> [Id]            -- Arguments; we're interested in which of these\r
-                           -- get case'd\r
-        -> CoreExpr\r
-        -> ExprSize\r
-\r
-sizeExpr (I# bOMB_OUT_SIZE) args expr\r
-  = size_up expr\r
-  where\r
-    size_up (Type t)         = sizeZero        -- Types cost nothing\r
-    size_up (Var v)           = sizeOne\r
-\r
-    size_up (Note InlineMe _) = sizeTwo                -- The idea is that this is one more\r
-                                               -- than the size of the "call" (i.e. 1)\r
-                                               -- We want to reply "no" to noSizeIncrease\r
-                                               -- for a bare reference (i.e. applied to no args) \r
-                                               -- to an INLINE thing\r
-\r
-    size_up (Note _ body)     = size_up body   -- Notes cost nothing\r
-\r
-    size_up (App fun (Type t)) = size_up fun\r
-    size_up (App fun arg)      = size_up_app fun `addSize` size_up arg\r
-\r
-    size_up (Con con args) = foldr (addSize . size_up) \r
-                                  (size_up_con con args)\r
-                                  args\r
-\r
-    size_up (Lam b e) | isId b    = size_up e `addSizeN` 1\r
-                     | otherwise = size_up e\r
-\r
-    size_up (Let (NonRec binder rhs) body)\r
-      = nukeScrutDiscount (size_up rhs)                `addSize`\r
-       size_up body                            `addSizeN`\r
-       1       -- For the allocation\r
-\r
-    size_up (Let (Rec pairs) body)\r
-      = nukeScrutDiscount rhs_size             `addSize`\r
-       size_up body                            `addSizeN`\r
-       length pairs            -- For the allocation\r
-      where\r
-       rhs_size = foldr (addSize . size_up . snd) sizeZero pairs\r
-\r
-    size_up (Case scrut _ alts)\r
-      = nukeScrutDiscount (size_up scrut)              `addSize`\r
-       arg_discount scrut                              `addSize`\r
-       foldr (addSize . size_up_alt) sizeZero alts     `addSizeN`\r
-       case (splitAlgTyConApp_maybe (coreExprType scrut)) of\r
-               Nothing       -> 1\r
-               Just (tc,_,_) -> tyConFamilySize tc\r
-\r
-    ------------ \r
-       -- A function application with at least one value argument\r
-       -- so if the function is an argument give it an arg-discount\r
-    size_up_app (App fun arg) = size_up_app fun  `addSize` size_up arg\r
-    size_up_app fun          = arg_discount fun `addSize` size_up fun\r
-\r
-    ------------ \r
-    size_up_alt (con, bndrs, rhs) = size_up rhs\r
-           -- Don't charge for args, so that wrappers look cheap\r
-\r
-    ------------\r
-    size_up_con (Literal lit) args | isNoRepLit lit = sizeN opt_UF_NoRepLit\r
-                                  | otherwise      = sizeOne\r
-\r
-    size_up_con (DataCon dc) args = conSizeN (valArgCount args)\r
-                            \r
-    size_up_con (PrimOp op) args = foldr addSize (sizeN op_cost) (map arg_discount args)\r
-               -- Give an arg-discount if a primop is applies to\r
-               -- one of the function's arguments\r
-      where\r
-       op_cost | primOpIsDupable op = opt_UF_CheapOp\r
-               | otherwise          = opt_UF_DearOp\r
-\r
-    ------------\r
-       -- We want to record if we're case'ing, or applying, an argument\r
-    arg_discount (Var v) | v `is_elem` args = scrutArg v\r
-    arg_discount other                     = sizeZero\r
-\r
-    is_elem :: Id -> [Id] -> Bool\r
-    is_elem = isIn "size_up_scrut"\r
-\r
-    ------------\r
-       -- These addSize things have to be here because\r
-       -- I don't want to give them bOMB_OUT_SIZE as an argument\r
-\r
-    addSizeN TooBig          _ = TooBig\r
-    addSizeN (SizeIs n xs d) (I# m)\r
-      | n_tot -# d <# bOMB_OUT_SIZE = SizeIs n_tot xs d\r
-      | otherwise                  = TooBig\r
-      where\r
-       n_tot = n +# m\r
-    \r
-    addSize TooBig _ = TooBig\r
-    addSize _ TooBig = TooBig\r
-    addSize (SizeIs n1 xs d1) (SizeIs n2 ys d2)\r
-      | (n_tot -# d_tot) <# bOMB_OUT_SIZE = SizeIs n_tot xys d_tot\r
-      | otherwise                        = TooBig\r
-      where\r
-       n_tot = n1 +# n2\r
-       d_tot = d1 +# d2\r
-       xys   = xs `unionBags` ys\r
-\end{code}\r
-\r
-Code for manipulating sizes\r
-\r
-\begin{code}\r
-\r
-data ExprSize = TooBig\r
-             | SizeIs Int#     -- Size found\r
-                      (Bag Id) -- Arguments cased herein\r
-                      Int#     -- Size to subtract if result is scrutinised \r
-                               -- by a case expression\r
-\r
-sizeZero       = SizeIs 0# emptyBag 0#\r
-sizeOne        = SizeIs 1# emptyBag 0#\r
-sizeTwo        = SizeIs 2# emptyBag 0#\r
-sizeN (I# n)   = SizeIs n  emptyBag 0#\r
-conSizeN (I# n) = SizeIs 1# emptyBag (n +# 1#)\r
-       -- Treat constructors as size 1, that unfoldAlways responsds 'False'\r
-       -- when asked about 'x' when x is bound to (C 3#).\r
-       -- This avoids gratuitous 'ticks' when x itself appears as an\r
-       -- atomic constructor argument.\r
-                                               \r
-scrutArg v     = SizeIs 0# (unitBag v) 0#\r
-\r
-nukeScrutDiscount (SizeIs n vs d) = SizeIs n vs 0#\r
-nukeScrutDiscount TooBig         = TooBig\r
-\end{code}\r
-\r
-\r
-%************************************************************************\r
-%*                                                                     *\r
-\subsection[considerUnfolding]{Given all the info, do (not) do the unfolding}\r
-%*                                                                     *\r
-%************************************************************************\r
-\r
-We have very limited information about an unfolding expression: (1)~so\r
-many type arguments and so many value arguments expected---for our\r
-purposes here, we assume we've got those.  (2)~A ``size'' or ``cost,''\r
-a single integer.  (3)~An ``argument info'' vector.  For this, what we\r
-have at the moment is a Boolean per argument position that says, ``I\r
-will look with great favour on an explicit constructor in this\r
-position.'' (4)~The ``discount'' to subtract if the expression\r
-is being scrutinised. \r
-\r
-Assuming we have enough type- and value arguments (if not, we give up\r
-immediately), then we see if the ``discounted size'' is below some\r
-(semi-arbitrary) threshold.  It works like this: for every argument\r
-position where we're looking for a constructor AND WE HAVE ONE in our\r
-hands, we get a (again, semi-arbitrary) discount [proportion to the\r
-number of constructors in the type being scrutinized].\r
-\r
-If we're in the context of a scrutinee ( \tr{(case <expr > of A .. -> ...;.. )})\r
-and the expression in question will evaluate to a constructor, we use\r
-the computed discount size *for the result only* rather than\r
-computing the argument discounts. Since we know the result of\r
-the expression is going to be taken apart, discounting its size\r
-is more accurate (see @sizeExpr@ above for how this discount size\r
-is computed).\r
-\r
-We use this one to avoid exporting inlinings that we ``couldn't possibly\r
-use'' on the other side.  Can be overridden w/ flaggery.\r
-Just the same as smallEnoughToInline, except that it has no actual arguments.\r
-\r
-\begin{code}\r
-couldBeSmallEnoughToInline :: UnfoldingGuidance -> Bool\r
-couldBeSmallEnoughToInline UnfoldNever = False\r
-couldBeSmallEnoughToInline other       = True\r
-\r
-certainlySmallEnoughToInline :: UnfoldingGuidance -> Bool\r
-certainlySmallEnoughToInline UnfoldNever                  = False\r
-certainlySmallEnoughToInline UnfoldAlways                 = True\r
-certainlySmallEnoughToInline (UnfoldIfGoodArgs _ _ size _) = size <= opt_UF_UseThreshold\r
-\end{code}\r
-\r
-@okToUnfoldInHifile@ is used when emitting unfolding info into an interface\r
-file to determine whether an unfolding candidate really should be unfolded.\r
-The predicate is needed to prevent @_casm_@s (+ lit-lits) from being emitted\r
-into interface files. \r
-\r
-The reason for inlining expressions containing _casm_s into interface files\r
-is that these fragments of C are likely to mention functions/#defines that\r
-will be out-of-scope when inlined into another module. This is not an\r
-unfixable problem for the user (just need to -#include the approp. header\r
-file), but turning it off seems to the simplest thing to do.\r
-\r
-\begin{code}\r
-okToUnfoldInHiFile :: CoreExpr -> Bool\r
-okToUnfoldInHiFile e = opt_UnfoldCasms || go e\r
- where\r
-    -- Race over an expression looking for CCalls..\r
-    go (Var _)                = True\r
-    go (Con (Literal lit) _)  = not (isLitLitLit lit)\r
-    go (Con (PrimOp op) args) = okToUnfoldPrimOp op && all go args\r
-    go (Con con args)         = True -- con args are always atomic\r
-    go (App fun arg)          = go fun && go arg\r
-    go (Lam _ body)           = go body\r
-    go (Let binds body)       = and (map go (body :rhssOfBind binds))\r
-    go (Case scrut bndr alts) = and (map go (scrut:rhssOfAlts alts))\r
-    go (Note _ body)          = go body\r
-    go (Type _)                      = True\r
-\r
-    -- ok to unfold a PrimOp as long as it's not a _casm_\r
-    okToUnfoldPrimOp (CCallOp _ is_casm _ _) = not is_casm\r
-    okToUnfoldPrimOp _                       = True\r
-\end{code}\r
-\r
-\r
-%************************************************************************\r
-%*                                                                     *\r
-\subsection{callSiteInline}\r
-%*                                                                     *\r
-%************************************************************************\r
-\r
-This is the key function.  It decides whether to inline a variable at a call site\r
-\r
-callSiteInline is used at call sites, so it is a bit more generous.\r
-It's a very important function that embodies lots of heuristics.\r
-A non-WHNF can be inlined if it doesn't occur inside a lambda,\r
-and occurs exactly once or \r
-    occurs once in each branch of a case and is small\r
-\r
-If the thing is in WHNF, there's no danger of duplicating work, \r
-so we can inline if it occurs once, or is small\r
-\r
-\begin{code}\r
-callSiteInline :: Bool                 -- True <=> the Id is black listed\r
-              -> Bool                  -- 'inline' note at call site\r
-              -> Id                    -- The Id\r
-              -> [CoreExpr]            -- Arguments\r
-              -> Bool                  -- True <=> continuation is interesting\r
-              -> Maybe CoreExpr        -- Unfolding, if any\r
-\r
-\r
-callSiteInline black_listed inline_call id args interesting_cont\r
-  = case getIdUnfolding id of {\r
-       NoUnfolding -> Nothing ;\r
-       OtherCon _  -> Nothing ;\r
-       CoreUnfolding form guidance unf_template ->\r
-\r
-    let\r
-       result | yes_or_no = Just unf_template\r
-              | otherwise = Nothing\r
-\r
-       inline_prag = getInlinePragma id\r
-       arg_infos   = map interestingArg val_args\r
-       val_args    = filter isValArg args\r
-       whnf        = whnfOrBottom form\r
-\r
-       yes_or_no =\r
-           case inline_prag of\r
-               IAmDead           -> pprTrace "callSiteInline: dead" (ppr id) False\r
-               IMustNotBeINLINEd -> False\r
-               IAmALoopBreaker   -> False\r
-               IMustBeINLINEd    -> True       -- Overrides absolutely everything, including the black list\r
-               ICanSafelyBeINLINEd in_lam one_br -> consider in_lam    one_br\r
-               NoInlinePragInfo                  -> consider InsideLam False\r
-\r
-       consider in_lam one_branch \r
-         | black_listed = False\r
-         | inline_call  = True\r
-         | one_branch  -- Be very keen to inline something if this is its unique occurrence; that\r
-                       -- gives a good chance of eliminating the original binding for the thing.\r
-                       -- The only time we hold back is when substituting inside a lambda;\r
-                       -- then if the context is totally uninteresting (not applied, not scrutinised)\r
-                       -- there is no point in substituting because it might just increase allocation.\r
-         = case in_lam of\r
-               NotInsideLam -> True\r
-               InsideLam    -> whnf && (not (null args) || interesting_cont)\r
-\r
-         | otherwise   -- Occurs (textually) more than once, so look at its size\r
-         = case guidance of\r
-             UnfoldAlways -> True\r
-             UnfoldNever  -> False\r
-             UnfoldIfGoodArgs n_vals_wanted arg_discounts size res_discount\r
-               | enough_args && size <= (n_vals_wanted + 1)\r
-                       -- No size increase\r
-                       -- Size of call is n_vals_wanted (+1 for the function)\r
-               -> case in_lam of\r
-                       NotInsideLam -> True\r
-                       InsideLam    -> whnf\r
-\r
-               | not (or arg_infos || really_interesting_cont)\r
-                       -- If it occurs more than once, there must be something interesting \r
-                       -- about some argument, or the result, to make it worth inlining\r
-               -> False\r
-  \r
-               | otherwise\r
-               -> case in_lam of\r
-                       NotInsideLam -> small_enough\r
-                       InsideLam    -> whnf && small_enough\r
-\r
-               where\r
-                 n_args                  = length arg_infos\r
-                 enough_args             = n_args >= n_vals_wanted\r
-                 really_interesting_cont | n_args <  n_vals_wanted = False     -- Too few args\r
-                                         | n_args == n_vals_wanted = interesting_cont\r
-                                         | otherwise               = True      -- Extra args\r
-                       -- This rather elaborate defn for really_interesting_cont is important\r
-                       -- Consider an I# = INLINE (\x -> I# {x})\r
-                       -- The unfolding guidance deems it to have size 2, and no arguments.\r
-                       -- So in an application (I# y) we must take the extra arg 'y' as\r
-                       -- evidene of an interesting context!\r
-                       \r
-                 small_enough = (size - discount) <= opt_UF_UseThreshold\r
-                 discount     = computeDiscount n_vals_wanted arg_discounts res_discount \r
-                                                arg_infos really_interesting_cont\r
-\r
-                               \r
-    in    \r
-#ifdef DEBUG\r
-    if opt_D_dump_inlinings then\r
-       pprTrace "Considering inlining"\r
-                (ppr id <+> vcat [text "black listed" <+> ppr black_listed,\r
-                                  text "inline prag:" <+> ppr inline_prag,\r
-                                  text "arg infos" <+> ppr arg_infos,\r
-                                  text "interesting continuation" <+> ppr interesting_cont,\r
-                                  text "whnf" <+> ppr whnf,\r
-                                  text "guidance" <+> ppr guidance,\r
-                                  text "ANSWER =" <+> if yes_or_no then text "YES" else text "NO",\r
-                                  if yes_or_no then\r
-                                       text "Unfolding =" <+> pprCoreExpr unf_template\r
-                                  else empty])\r
-                 result\r
-    else\r
-#endif\r
-    result\r
-    }\r
-\r
--- An argument is interesting if it has *some* structure\r
--- We are here trying to avoid unfolding a function that\r
--- is applied only to variables that have no unfolding\r
--- (i.e. they are probably lambda bound): f x y z\r
--- There is little point in inlining f here.\r
-interestingArg (Type _)                 = False\r
-interestingArg (App fn (Type _)) = interestingArg fn\r
-interestingArg (Var v)          = hasUnfolding (getIdUnfolding v)\r
-interestingArg other            = True\r
-\r
-\r
-computeDiscount :: Int -> [Int] -> Int -> [Bool] -> Bool -> Int\r
-computeDiscount n_vals_wanted arg_discounts res_discount arg_infos result_used\r
-       -- We multiple the raw discounts (args_discount and result_discount)\r
-       -- ty opt_UnfoldingKeenessFactor because the former have to do with\r
-       -- *size* whereas the discounts imply that there's some extra \r
-       -- *efficiency* to be gained (e.g. beta reductions, case reductions) \r
-       -- by inlining.\r
-\r
-       -- we also discount 1 for each argument passed, because these will\r
-       -- reduce with the lambdas in the function (we count 1 for a lambda\r
-       -- in size_up).\r
-  = length (take n_vals_wanted arg_infos) +\r
-                       -- Discount of 1 for each arg supplied, because the \r
-                       -- result replaces the call\r
-    round (opt_UF_KeenessFactor * \r
-          fromInt (arg_discount + result_discount))\r
-  where\r
-    arg_discount = sum (zipWith mk_arg_discount arg_discounts arg_infos)\r
-\r
-    mk_arg_discount discount is_evald | is_evald  = discount\r
-                                     | otherwise = 0\r
-\r
-       -- Don't give a result discount unless there are enough args\r
-    result_discount | result_used = res_discount       -- Over-applied, or case scrut\r
-                   | otherwise   = 0\r
-\end{code}\r
-\r
-\r
-%************************************************************************\r
-%*                                                                     *\r
-\subsection{Black-listing}\r
-%*                                                                     *\r
-%************************************************************************\r
-\r
-Inlining is controlled by the "Inline phase" number, which is set\r
-by the per-simplification-pass '-finline-phase' flag.\r
-\r
-For optimisation we use phase 1,2 and nothing (i.e. no -finline-phase flag)\r
-in that order.  The meanings of these are determined by the @blackListed@ function\r
-here.\r
-\r
-\begin{code}\r
-blackListed :: IdSet           -- Used in transformation rules\r
-           -> Maybe Int        -- Inline phase\r
-           -> Id -> Bool       -- True <=> blacklisted\r
-       \r
--- The blackListed function sees whether a variable should *not* be \r
--- inlined because of the inline phase we are in.  This is the sole\r
--- place that the inline phase number is looked at.\r
-\r
--- Phase 0: used for 'no inlinings please'\r
-blackListed rule_vars (Just 0)\r
-  = \v -> True\r
-\r
--- Phase 1: don't inline any rule-y things or things with specialisations\r
-blackListed rule_vars (Just 1)\r
-  = \v -> let v_uniq = idUnique v\r
-         in v `elemVarSet` rule_vars\r
-         || not (isEmptyCoreRules (getIdSpecialisation v))\r
-         || v_uniq == runSTRepIdKey\r
-\r
--- Phase 2: allow build/augment to inline, and specialisations\r
-blackListed rule_vars (Just 2)\r
-  = \v -> let v_uniq = idUnique v\r
-         in (v `elemVarSet` rule_vars && not (v_uniq == buildIdKey || \r
-                                              v_uniq == augmentIdKey))\r
-         || v_uniq == runSTRepIdKey\r
-\r
--- Otherwise just go for it\r
-blackListed rule_vars phase\r
-  = \v -> False\r
-\end{code}\r
-\r
-\r
-SLPJ 95/04: Why @runST@ must be inlined very late:\r
-\begin{verbatim}\r
-f x =\r
-  runST ( \ s -> let\r
-                   (a, s')  = newArray# 100 [] s\r
-                   (_, s'') = fill_in_array_or_something a x s'\r
-                 in\r
-                 freezeArray# a s'' )\r
-\end{verbatim}\r
-If we inline @runST@, we'll get:\r
-\begin{verbatim}\r
-f x = let\r
-       (a, s')  = newArray# 100 [] realWorld#{-NB-}\r
-       (_, s'') = fill_in_array_or_something a x s'\r
-      in\r
-      freezeArray# a s''\r
-\end{verbatim}\r
-And now the @newArray#@ binding can be floated to become a CAF, which\r
-is totally and utterly wrong:\r
-\begin{verbatim}\r
-f = let\r
-    (a, s')  = newArray# 100 [] realWorld#{-NB-} -- YIKES!!!\r
-    in\r
-    \ x ->\r
-       let (_, s'') = fill_in_array_or_something a x s' in\r
-       freezeArray# a s''\r
-\end{verbatim}\r
-All calls to @f@ will share a {\em single} array!  \r
-\r
-Yet we do want to inline runST sometime, so we can avoid\r
-needless code.  Solution: black list it until the last moment.\r
-\r
+%
+% (c) The AQUA Project, Glasgow University, 1994-1998
+%
+\section[CoreUnfold]{Core-syntax unfoldings}
+
+Unfoldings (which can travel across module boundaries) are in Core
+syntax (namely @CoreExpr@s).
+
+The type @Unfolding@ sits ``above'' simply-Core-expressions
+unfoldings, capturing ``higher-level'' things we know about a binding,
+usually things that the simplifier found out (e.g., ``it's a
+literal'').  In the corner of a @CoreUnfolding@ unfolding, you will
+find, unsurprisingly, a Core expression.
+
+\begin{code}
+module CoreUnfold (
+       Unfolding(..), UnfoldingGuidance, -- types
+
+       noUnfolding, mkUnfolding, getUnfoldingTemplate,
+       isEvaldUnfolding, hasUnfolding,
+
+       couldBeSmallEnoughToInline, 
+       certainlySmallEnoughToInline, 
+       okToUnfoldInHiFile,
+
+       calcUnfoldingGuidance,
+
+       callSiteInline, blackListed
+    ) where
+
+#include "HsVersions.h"
+
+import CmdLineOpts     ( opt_UF_CreationThreshold,
+                         opt_UF_UseThreshold,
+                         opt_UF_ScrutConDiscount,
+                         opt_UF_FunAppDiscount,
+                         opt_UF_PrimArgDiscount,
+                         opt_UF_KeenessFactor,
+                         opt_UF_CheapOp, opt_UF_DearOp, opt_UF_NoRepLit,
+                         opt_UnfoldCasms, opt_PprStyle_Debug,
+                         opt_D_dump_inlinings
+                       )
+import CoreSyn
+import PprCore         ( pprCoreExpr )
+import OccurAnal       ( occurAnalyseGlobalExpr )
+import BinderInfo      ( )
+import CoreUtils       ( coreExprType, exprIsTrivial, mkFormSummary, whnfOrBottom,
+                         FormSummary(..) )
+import Id              ( Id, idType, idUnique, isId, 
+                         getIdSpecialisation, getInlinePragma, getIdUnfolding
+                       )
+import VarSet
+import Const           ( Con(..), isLitLitLit, isWHNFCon )
+import PrimOp          ( PrimOp(..), primOpIsDupable )
+import IdInfo          ( ArityInfo(..), InlinePragInfo(..), OccInfo(..) )
+import TyCon           ( tyConFamilySize )
+import Type            ( splitAlgTyConApp_maybe, splitFunTy_maybe )
+import Const           ( isNoRepLit )
+import Unique          ( Unique, buildIdKey, augmentIdKey, runSTRepIdKey )
+import Maybes          ( maybeToBool )
+import Bag
+import Util            ( isIn, lengthExceeds )
+import Outputable
+\end{code}
+
+%************************************************************************
+%*                                                                     *
+\subsection{@Unfolding@ and @UnfoldingGuidance@ types}
+%*                                                                     *
+%************************************************************************
+
+\begin{code}
+data Unfolding
+  = NoUnfolding
+
+  | OtherCon [Con]             -- It ain't one of these
+                               -- (OtherCon xs) also indicates that something has been evaluated
+                               -- and hence there's no point in re-evaluating it.
+                               -- OtherCon [] is used even for non-data-type values
+                               -- to indicated evaluated-ness.  Notably:
+                               --      data C = C !(Int -> Int)
+                               --      case x of { C f -> ... }
+                               -- Here, f gets an OtherCon [] unfolding.
+
+  | CoreUnfolding                      -- An unfolding with redundant cached information
+               FormSummary             -- Tells whether the template is a WHNF or bottom
+               UnfoldingGuidance       -- Tells about the *size* of the template.
+               CoreExpr                -- Template; binder-info is correct
+\end{code}
+
+\begin{code}
+noUnfolding = NoUnfolding
+
+mkUnfolding expr
+  = let
+     -- strictness mangling (depends on there being no CSE)
+     ufg = calcUnfoldingGuidance opt_UF_CreationThreshold expr
+     occ = occurAnalyseGlobalExpr expr
+    in
+    CoreUnfolding (mkFormSummary expr) ufg occ
+
+getUnfoldingTemplate :: Unfolding -> CoreExpr
+getUnfoldingTemplate (CoreUnfolding _ _ expr) = expr
+getUnfoldingTemplate other = panic "getUnfoldingTemplate"
+
+isEvaldUnfolding :: Unfolding -> Bool
+isEvaldUnfolding (OtherCon _)                    = True
+isEvaldUnfolding (CoreUnfolding ValueForm _ expr) = True
+isEvaldUnfolding other                           = False
+
+hasUnfolding :: Unfolding -> Bool
+hasUnfolding NoUnfolding = False
+hasUnfolding other      = True
+
+data UnfoldingGuidance
+  = UnfoldNever
+  | UnfoldAlways               -- There is no "original" definition,
+                               -- so you'd better unfold.  Or: something
+                               -- so cheap to unfold (e.g., 1#) that
+                               -- you should do it absolutely always.
+
+  | UnfoldIfGoodArgs   Int     -- and "n" value args
+
+                       [Int]   -- Discount if the argument is evaluated.
+                               -- (i.e., a simplification will definitely
+                               -- be possible).  One elt of the list per *value* arg.
+
+                       Int     -- The "size" of the unfolding; to be elaborated
+                               -- later. ToDo
+
+                       Int     -- Scrutinee discount: the discount to substract if the thing is in
+                               -- a context (case (thing args) of ...),
+                               -- (where there are the right number of arguments.)
+\end{code}
+
+\begin{code}
+instance Outputable UnfoldingGuidance where
+    ppr UnfoldAlways    = ptext SLIT("ALWAYS")
+    ppr UnfoldNever    = ptext SLIT("NEVER")
+    ppr (UnfoldIfGoodArgs v cs size discount)
+      = hsep [ptext SLIT("IF_ARGS"), int v,
+              if null cs       -- always print *something*
+               then char 'X'
+               else hcat (map (text . show) cs),
+              int size,
+              int discount ]
+\end{code}
+
+
+%************************************************************************
+%*                                                                     *
+\subsection[calcUnfoldingGuidance]{Calculate ``unfolding guidance'' for an expression}
+%*                                                                     *
+%************************************************************************
+
+\begin{code}
+calcUnfoldingGuidance
+       :: Int                  -- bomb out if size gets bigger than this
+       -> CoreExpr             -- expression to look at
+       -> UnfoldingGuidance
+calcUnfoldingGuidance bOMB_OUT_SIZE expr
+  | exprIsTrivial expr         -- Often trivial expressions are never bound
+                               -- to an expression, but it can happen.  For
+                               -- example, the Id for a nullary constructor has
+                               -- a trivial expression as its unfolding, and
+                               -- we want to make sure that we always unfold it.
+  = UnfoldAlways
+  | otherwise
+  = case collectBinders expr of { (binders, body) ->
+    let
+       val_binders = filter isId binders
+    in
+    case (sizeExpr bOMB_OUT_SIZE val_binders body) of
+
+      TooBig -> UnfoldNever
+
+      SizeIs size cased_args scrut_discount
+       -> UnfoldIfGoodArgs
+                       (length val_binders)
+                       (map discount_for val_binders)
+                       (I# size)
+                       (I# scrut_discount)
+       where        
+           discount_for b 
+               | num_cases == 0 = 0
+               | is_fun_ty      = num_cases * opt_UF_FunAppDiscount
+               | is_data_ty     = num_cases * tyConFamilySize tycon * opt_UF_ScrutConDiscount
+               | otherwise      = num_cases * opt_UF_PrimArgDiscount
+               where
+                 num_cases           = foldlBag (\n b' -> if b==b' then n+1 else n) 0 cased_args
+                                       -- Count occurrences of b in cased_args
+                 arg_ty              = idType b
+                 is_fun_ty           = maybeToBool (splitFunTy_maybe arg_ty)
+                 (is_data_ty, tycon) = case (splitAlgTyConApp_maybe (idType b)) of
+                                         Nothing       -> (False, panic "discount")
+                                         Just (tc,_,_) -> (True,  tc)
+       }
+\end{code}
+
+\begin{code}
+sizeExpr :: Int            -- Bomb out if it gets bigger than this
+        -> [Id]            -- Arguments; we're interested in which of these
+                           -- get case'd
+        -> CoreExpr
+        -> ExprSize
+
+sizeExpr (I# bOMB_OUT_SIZE) args expr
+  = size_up expr
+  where
+    size_up (Type t)         = sizeZero        -- Types cost nothing
+    size_up (Var v)           = sizeOne
+
+    size_up (Note InlineMe _) = sizeTwo                -- The idea is that this is one more
+                                               -- than the size of the "call" (i.e. 1)
+                                               -- We want to reply "no" to noSizeIncrease
+                                               -- for a bare reference (i.e. applied to no args) 
+                                               -- to an INLINE thing
+
+    size_up (Note _ body)     = size_up body   -- Notes cost nothing
+
+    size_up (App fun (Type t))  = size_up fun
+    size_up (App fun arg)       = size_up_app fun [arg]
+
+    size_up (Con con args) = foldr (addSize . size_up) 
+                                  (size_up_con con args)
+                                  args
+
+    size_up (Lam b e) | isId b    = size_up e `addSizeN` 1
+                     | otherwise = size_up e
+
+    size_up (Let (NonRec binder rhs) body)
+      = nukeScrutDiscount (size_up rhs)                `addSize`
+       size_up body                            `addSizeN`
+       1       -- For the allocation
+
+    size_up (Let (Rec pairs) body)
+      = nukeScrutDiscount rhs_size             `addSize`
+       size_up body                            `addSizeN`
+       length pairs            -- For the allocation
+      where
+       rhs_size = foldr (addSize . size_up . snd) sizeZero pairs
+
+    size_up (Case scrut _ alts)
+      = nukeScrutDiscount (size_up scrut)              `addSize`
+       arg_discount scrut                              `addSize`
+       foldr (addSize . size_up_alt) sizeZero alts     `addSizeN`
+       case (splitAlgTyConApp_maybe (coreExprType scrut)) of
+               Nothing       -> 1
+               Just (tc,_,_) -> tyConFamilySize tc
+
+    ------------ 
+    size_up_app (App fun arg) args   = size_up_app fun (arg:args)
+    size_up_app fun          args   = foldr (addSize . size_up) (fun_discount fun) args
+
+       -- A function application with at least one value argument
+       -- so if the function is an argument give it an arg-discount
+       -- Also behave specially if the function is a build
+    fun_discount (Var fun) | idUnique fun == buildIdKey = buildSize
+                          | fun `is_elem` args         = scrutArg fun
+    fun_discount other                                 = sizeZero
+
+    ------------ 
+    size_up_alt (con, bndrs, rhs) = size_up rhs
+           -- Don't charge for args, so that wrappers look cheap
+
+    ------------
+    size_up_con (Literal lit) args | isNoRepLit lit = sizeN opt_UF_NoRepLit
+                                  | otherwise      = sizeOne
+
+    size_up_con (DataCon dc) args = conSizeN (valArgCount args)
+                            
+    size_up_con (PrimOp op) args = foldr addSize (sizeN op_cost) (map arg_discount args)
+               -- Give an arg-discount if a primop is applies to
+               -- one of the function's arguments
+      where
+       op_cost | primOpIsDupable op = opt_UF_CheapOp
+               | otherwise          = opt_UF_DearOp
+
+       -- We want to record if we're case'ing, or applying, an argument
+    arg_discount (Var v) | v `is_elem` args = scrutArg v
+    arg_discount other                     = sizeZero
+
+    ------------
+    is_elem :: Id -> [Id] -> Bool
+    is_elem = isIn "size_up_scrut"
+
+    ------------
+       -- These addSize things have to be here because
+       -- I don't want to give them bOMB_OUT_SIZE as an argument
+
+    addSizeN TooBig          _ = TooBig
+    addSizeN (SizeIs n xs d) (I# m)
+      | n_tot -# d <# bOMB_OUT_SIZE = SizeIs n_tot xs d
+      | otherwise                  = TooBig
+      where
+       n_tot = n +# m
+    
+    addSize TooBig _ = TooBig
+    addSize _ TooBig = TooBig
+    addSize (SizeIs n1 xs d1) (SizeIs n2 ys d2)
+      | (n_tot -# d_tot) <# bOMB_OUT_SIZE = SizeIs n_tot xys d_tot
+      | otherwise                        = TooBig
+      where
+       n_tot = n1 +# n2
+       d_tot = d1 +# d2
+       xys   = xs `unionBags` ys
+\end{code}
+
+Code for manipulating sizes
+
+\begin{code}
+
+data ExprSize = TooBig
+             | SizeIs Int#     -- Size found
+                      (Bag Id) -- Arguments cased herein
+                      Int#     -- Size to subtract if result is scrutinised 
+                               -- by a case expression
+
+sizeZero       = SizeIs 0# emptyBag 0#
+sizeOne        = SizeIs 1# emptyBag 0#
+sizeTwo        = SizeIs 2# emptyBag 0#
+sizeN (I# n)   = SizeIs n  emptyBag 0#
+conSizeN (I# n) = SizeIs 1# emptyBag (n +# 1#)
+       -- Treat constructors as size 1, that unfoldAlways responsds 'False'
+       -- when asked about 'x' when x is bound to (C 3#).
+       -- This avoids gratuitous 'ticks' when x itself appears as an
+       -- atomic constructor argument.
+
+buildSize = SizeIs (-2#) emptyBag 4#
+       -- We really want to inline applications of build
+       -- build t (\cn -> e) should cost only the cost of e (because build will be inlined later)
+       -- Indeed, we should add a result_discount becuause build is 
+       -- very like a constructor.  We don't bother to check that the
+       -- build is saturated (it usually is).  The "-2" discounts for the \c n
+       -- The "4" is rather arbitrary.
+                                               
+scrutArg v     = SizeIs 0# (unitBag v) 0#
+
+nukeScrutDiscount (SizeIs n vs d) = SizeIs n vs 0#
+nukeScrutDiscount TooBig         = TooBig
+\end{code}
+
+
+%************************************************************************
+%*                                                                     *
+\subsection[considerUnfolding]{Given all the info, do (not) do the unfolding}
+%*                                                                     *
+%************************************************************************
+
+We have very limited information about an unfolding expression: (1)~so
+many type arguments and so many value arguments expected---for our
+purposes here, we assume we've got those.  (2)~A ``size'' or ``cost,''
+a single integer.  (3)~An ``argument info'' vector.  For this, what we
+have at the moment is a Boolean per argument position that says, ``I
+will look with great favour on an explicit constructor in this
+position.'' (4)~The ``discount'' to subtract if the expression
+is being scrutinised. 
+
+Assuming we have enough type- and value arguments (if not, we give up
+immediately), then we see if the ``discounted size'' is below some
+(semi-arbitrary) threshold.  It works like this: for every argument
+position where we're looking for a constructor AND WE HAVE ONE in our
+hands, we get a (again, semi-arbitrary) discount [proportion to the
+number of constructors in the type being scrutinized].
+
+If we're in the context of a scrutinee ( \tr{(case <expr > of A .. -> ...;.. )})
+and the expression in question will evaluate to a constructor, we use
+the computed discount size *for the result only* rather than
+computing the argument discounts. Since we know the result of
+the expression is going to be taken apart, discounting its size
+is more accurate (see @sizeExpr@ above for how this discount size
+is computed).
+
+We use this one to avoid exporting inlinings that we ``couldn't possibly
+use'' on the other side.  Can be overridden w/ flaggery.
+Just the same as smallEnoughToInline, except that it has no actual arguments.
+
+\begin{code}
+couldBeSmallEnoughToInline :: UnfoldingGuidance -> Bool
+couldBeSmallEnoughToInline UnfoldNever = False
+couldBeSmallEnoughToInline other       = True
+
+certainlySmallEnoughToInline :: UnfoldingGuidance -> Bool
+certainlySmallEnoughToInline UnfoldNever                  = False
+certainlySmallEnoughToInline UnfoldAlways                 = True
+certainlySmallEnoughToInline (UnfoldIfGoodArgs _ _ size _) = size <= opt_UF_UseThreshold
+\end{code}
+
+@okToUnfoldInHifile@ is used when emitting unfolding info into an interface
+file to determine whether an unfolding candidate really should be unfolded.
+The predicate is needed to prevent @_casm_@s (+ lit-lits) from being emitted
+into interface files. 
+
+The reason for inlining expressions containing _casm_s into interface files
+is that these fragments of C are likely to mention functions/#defines that
+will be out-of-scope when inlined into another module. This is not an
+unfixable problem for the user (just need to -#include the approp. header
+file), but turning it off seems to the simplest thing to do.
+
+\begin{code}
+okToUnfoldInHiFile :: CoreExpr -> Bool
+okToUnfoldInHiFile e = opt_UnfoldCasms || go e
+ where
+    -- Race over an expression looking for CCalls..
+    go (Var _)                = True
+    go (Con (Literal lit) _)  = not (isLitLitLit lit)
+    go (Con (PrimOp op) args) = okToUnfoldPrimOp op && all go args
+    go (Con con args)         = True -- con args are always atomic
+    go (App fun arg)          = go fun && go arg
+    go (Lam _ body)           = go body
+    go (Let binds body)       = and (map go (body :rhssOfBind binds))
+    go (Case scrut bndr alts) = and (map go (scrut:rhssOfAlts alts))
+    go (Note _ body)          = go body
+    go (Type _)                      = True
+
+    -- ok to unfold a PrimOp as long as it's not a _casm_
+    okToUnfoldPrimOp (CCallOp _ is_casm _ _) = not is_casm
+    okToUnfoldPrimOp _                       = True
+\end{code}
+
+
+%************************************************************************
+%*                                                                     *
+\subsection{callSiteInline}
+%*                                                                     *
+%************************************************************************
+
+This is the key function.  It decides whether to inline a variable at a call site
+
+callSiteInline is used at call sites, so it is a bit more generous.
+It's a very important function that embodies lots of heuristics.
+A non-WHNF can be inlined if it doesn't occur inside a lambda,
+and occurs exactly once or 
+    occurs once in each branch of a case and is small
+
+If the thing is in WHNF, there's no danger of duplicating work, 
+so we can inline if it occurs once, or is small
+
+\begin{code}
+callSiteInline :: Bool                 -- True <=> the Id is black listed
+              -> Bool                  -- 'inline' note at call site
+              -> Id                    -- The Id
+              -> [CoreExpr]            -- Arguments
+              -> Bool                  -- True <=> continuation is interesting
+              -> Maybe CoreExpr        -- Unfolding, if any
+
+
+callSiteInline black_listed inline_call id args interesting_cont
+  = case getIdUnfolding id of {
+       NoUnfolding -> Nothing ;
+       OtherCon _  -> Nothing ;
+       CoreUnfolding form guidance unf_template ->
+
+    let
+       result | yes_or_no = Just unf_template
+              | otherwise = Nothing
+
+       inline_prag = getInlinePragma id
+       arg_infos   = map interestingArg val_args
+       val_args    = filter isValArg args
+       whnf        = whnfOrBottom form
+
+       yes_or_no =
+           case inline_prag of
+               IAmDead           -> pprTrace "callSiteInline: dead" (ppr id) False
+               IMustNotBeINLINEd -> False
+               IAmALoopBreaker   -> False
+               IMustBeINLINEd    -> True       -- Overrides absolutely everything, including the black list
+               ICanSafelyBeINLINEd in_lam one_br -> consider in_lam    one_br
+               NoInlinePragInfo                  -> consider InsideLam False
+
+       consider in_lam one_branch 
+         | black_listed = False
+         | inline_call  = True
+         | one_branch  -- Be very keen to inline something if this is its unique occurrence; that
+                       -- gives a good chance of eliminating the original binding for the thing.
+                       -- The only time we hold back is when substituting inside a lambda;
+                       -- then if the context is totally uninteresting (not applied, not scrutinised)
+                       -- there is no point in substituting because it might just increase allocation.
+         = WARN( case in_lam of { NotInsideLam -> True; other -> False },
+                 text "callSiteInline:oneOcc" <+> ppr id )
+               -- If it has one occurrence, not inside a lambda, PreInlineUnconditionally
+               -- should have zapped it already
+           whnf && (not (null args) || interesting_cont)
+
+         | otherwise   -- Occurs (textually) more than once, so look at its size
+         = case guidance of
+             UnfoldAlways -> True
+             UnfoldNever  -> False
+             UnfoldIfGoodArgs n_vals_wanted arg_discounts size res_discount
+               | enough_args && size <= (n_vals_wanted + 1)
+                       -- No size increase
+                       -- Size of call is n_vals_wanted (+1 for the function)
+               -> case in_lam of
+                       NotInsideLam -> True
+                       InsideLam    -> whnf
+
+               | not (or arg_infos || really_interesting_cont)
+                       -- If it occurs more than once, there must be something interesting 
+                       -- about some argument, or the result, to make it worth inlining
+               -> False
+  
+               | otherwise
+               -> case in_lam of
+                       NotInsideLam -> small_enough
+                       InsideLam    -> whnf && small_enough
+
+               where
+                 n_args                  = length arg_infos
+                 enough_args             = n_args >= n_vals_wanted
+                 really_interesting_cont | n_args <  n_vals_wanted = False     -- Too few args
+                                         | n_args == n_vals_wanted = interesting_cont
+                                         | otherwise               = True      -- Extra args
+                       -- This rather elaborate defn for really_interesting_cont is important
+                       -- Consider an I# = INLINE (\x -> I# {x})
+                       -- The unfolding guidance deems it to have size 2, and no arguments.
+                       -- So in an application (I# y) we must take the extra arg 'y' as
+                       -- evidence of an interesting context!
+                       
+                 small_enough = (size - discount) <= opt_UF_UseThreshold
+                 discount     = computeDiscount n_vals_wanted arg_discounts res_discount 
+                                                arg_infos really_interesting_cont
+
+                               
+    in    
+#ifdef DEBUG
+    if opt_D_dump_inlinings then
+       pprTrace "Considering inlining"
+                (ppr id <+> vcat [text "black listed" <+> ppr black_listed,
+                                  text "inline prag:" <+> ppr inline_prag,
+                                  text "arg infos" <+> ppr arg_infos,
+                                  text "interesting continuation" <+> ppr interesting_cont,
+                                  text "whnf" <+> ppr whnf,
+                                  text "guidance" <+> ppr guidance,
+                                  text "ANSWER =" <+> if yes_or_no then text "YES" else text "NO",
+                                  if yes_or_no then
+                                       text "Unfolding =" <+> pprCoreExpr unf_template
+                                  else empty])
+                 result
+    else
+#endif
+    result
+    }
+
+-- An argument is interesting if it has *some* structure
+-- We are here trying to avoid unfolding a function that
+-- is applied only to variables that have no unfolding
+-- (i.e. they are probably lambda bound): f x y z
+-- There is little point in inlining f here.
+interestingArg (Type _)                 = False
+interestingArg (App fn (Type _)) = interestingArg fn
+interestingArg (Var v)          = hasUnfolding (getIdUnfolding v)
+interestingArg other            = True
+
+
+computeDiscount :: Int -> [Int] -> Int -> [Bool] -> Bool -> Int
+computeDiscount n_vals_wanted arg_discounts res_discount arg_infos result_used
+       -- We multiple the raw discounts (args_discount and result_discount)
+       -- ty opt_UnfoldingKeenessFactor because the former have to do with
+       -- *size* whereas the discounts imply that there's some extra 
+       -- *efficiency* to be gained (e.g. beta reductions, case reductions) 
+       -- by inlining.
+
+       -- we also discount 1 for each argument passed, because these will
+       -- reduce with the lambdas in the function (we count 1 for a lambda
+       -- in size_up).
+  = length (take n_vals_wanted arg_infos) +
+                       -- Discount of 1 for each arg supplied, because the 
+                       -- result replaces the call
+    round (opt_UF_KeenessFactor * 
+          fromInt (arg_discount + result_discount))
+  where
+    arg_discount = sum (zipWith mk_arg_discount arg_discounts arg_infos)
+
+    mk_arg_discount discount is_evald | is_evald  = discount
+                                     | otherwise = 0
+
+       -- Don't give a result discount unless there are enough args
+    result_discount | result_used = res_discount       -- Over-applied, or case scrut
+                   | otherwise   = 0
+\end{code}
+
+
+%************************************************************************
+%*                                                                     *
+\subsection{Black-listing}
+%*                                                                     *
+%************************************************************************
+
+Inlining is controlled by the "Inline phase" number, which is set
+by the per-simplification-pass '-finline-phase' flag.
+
+For optimisation we use phase 1,2 and nothing (i.e. no -finline-phase flag)
+in that order.  The meanings of these are determined by the @blackListed@ function
+here.
+
+\begin{code}
+blackListed :: IdSet           -- Used in transformation rules
+           -> Maybe Int        -- Inline phase
+           -> Id -> Bool       -- True <=> blacklisted
+       
+-- The blackListed function sees whether a variable should *not* be 
+-- inlined because of the inline phase we are in.  This is the sole
+-- place that the inline phase number is looked at.
+
+-- Phase 0: used for 'no inlinings please'
+blackListed rule_vars (Just 0)
+  = \v -> True
+
+-- Phase 1: don't inline any rule-y things or things with specialisations
+blackListed rule_vars (Just 1)
+  = \v -> let v_uniq = idUnique v
+         in v `elemVarSet` rule_vars
+         || not (isEmptyCoreRules (getIdSpecialisation v))
+         || v_uniq == runSTRepIdKey
+
+-- Phase 2: allow build/augment to inline, and specialisations
+blackListed rule_vars (Just 2)
+  = \v -> let v_uniq = idUnique v
+         in (v `elemVarSet` rule_vars && not (v_uniq == buildIdKey || 
+                                              v_uniq == augmentIdKey))
+         || v_uniq == runSTRepIdKey
+
+-- Otherwise just go for it
+blackListed rule_vars phase
+  = \v -> False
+\end{code}
+
+
+SLPJ 95/04: Why @runST@ must be inlined very late:
+\begin{verbatim}
+f x =
+  runST ( \ s -> let
+                   (a, s')  = newArray# 100 [] s
+                   (_, s'') = fill_in_array_or_something a x s'
+                 in
+                 freezeArray# a s'' )
+\end{verbatim}
+If we inline @runST@, we'll get:
+\begin{verbatim}
+f x = let
+       (a, s')  = newArray# 100 [] realWorld#{-NB-}
+       (_, s'') = fill_in_array_or_something a x s'
+      in
+      freezeArray# a s''
+\end{verbatim}
+And now the @newArray#@ binding can be floated to become a CAF, which
+is totally and utterly wrong:
+\begin{verbatim}
+f = let
+    (a, s')  = newArray# 100 [] realWorld#{-NB-} -- YIKES!!!
+    in
+    \ x ->
+       let (_, s'') = fill_in_array_or_something a x s' in
+       freezeArray# a s''
+\end{verbatim}
+All calls to @f@ will share a {\em single} array!  
+
+Yet we do want to inline runST sometime, so we can avoid
+needless code.  Solution: black list it until the last moment.
+