X-Git-Url: http://git.megacz.com/?a=blobdiff_plain;f=compiler%2FcoreSyn%2FCoreSubst.lhs;h=b02bc805784b21986e809d433c288cb757556f79;hb=2662dbc5b2c30fc11ccb99e7f9b2dba794d680ba;hp=14eccc61a40e33aacae4144fce51a83787b2bb16;hpb=a263737bbf44050a7b5ecbe267ddf85d410b73e5;p=ghc-hetmet.git diff --git a/compiler/coreSyn/CoreSubst.lhs b/compiler/coreSyn/CoreSubst.lhs index 14eccc6..b02bc80 100644 --- a/compiler/coreSyn/CoreSubst.lhs +++ b/compiler/coreSyn/CoreSubst.lhs @@ -523,13 +523,11 @@ substUnfolding _ unf = unf -- NoUnfolding, OtherCon ------------------- substInlineRuleInfo :: Subst -> InlineRuleInfo -> InlineRuleInfo -substInlineRuleInfo subst (InlWrapper wkr) - = case lookupIdSubst subst wkr of - Var w1 -> InlWrapper w1 - other -> WARN( not (exprIsTrivial other), text "CoreSubst.substWorker:" <+> ppr wkr ) - InlVanilla -- Worker has got substituted away altogether - -- (This can happen if it's trivial, via - -- postInlineUnconditionally, hence only warning) +substInlineRuleInfo (Subst in_scope ids _) (InlWrapper wkr) + | Just (Var w1) <- lookupVarEnv ids wkr = InlWrapper w1 + | Just w1 <- lookupInScope in_scope wkr = InlWrapper w1 + | otherwise = WARN( True, text "Interesting! CoreSubst.substWorker:" <+> ppr wkr ) + InlVanilla -- Note [Worker inlining] substInlineRuleInfo _ info = info ------------------ @@ -584,6 +582,18 @@ substVarSet subst fvs | otherwise = Type.tyVarsOfType (lookupTvSubst subst fv) \end{code} +Note [Worker inlining] +~~~~~~~~~~~~~~~~~~~~~~ +A worker can get sustituted away entirely. + - it might be trivial + - it might simply be very small +We do not treat an InlWrapper as an 'occurrence' in the occurence +analyser, so it's possible that the worker is not even in scope any more. + +In all all these cases we simply drop the special case, returning to +InlVanilla. The WARN is just so I can see if it happens a lot. + + %************************************************************************ %* * The Very Simple Optimiser