diff options
author | Simon Peyton Jones <simonpj@microsoft.com> | 2016-02-25 15:55:56 +0000 |
---|---|---|
committer | Simon Peyton Jones <simonpj@microsoft.com> | 2016-02-26 17:14:59 +0000 |
commit | 4ddfe1352e20d805a0ad6eeea0400ee218023bfb (patch) | |
tree | fcfd63adb55270b5d7664edb71326b6f85a8d28f /compiler/specialise | |
parent | a81e9d57439f338ac3c202b929b4b9e991ee7c20 (diff) | |
download | haskell-4ddfe1352e20d805a0ad6eeea0400ee218023bfb.tar.gz |
Get the right in-scope set in specUnfolding
This fixes Trac #11600
Diffstat (limited to 'compiler/specialise')
-rw-r--r-- | compiler/specialise/Specialise.hs | 21 |
1 files changed, 16 insertions, 5 deletions
diff --git a/compiler/specialise/Specialise.hs b/compiler/specialise/Specialise.hs index a8380d863b..477092e09b 100644 --- a/compiler/specialise/Specialise.hs +++ b/compiler/specialise/Specialise.hs @@ -1309,9 +1309,17 @@ specCalls mb_mod env rules_for_me calls_for_me fn rhs = (inl_prag { inl_inline = EmptyInlineSpec }, noUnfolding) | otherwise - = (inl_prag, specUnfolding dflags (se_subst env) - poly_tyvars (ty_args ++ spec_dict_args) - fn_unf) + = (inl_prag, specUnfolding dflags spec_unf_subst poly_tyvars + spec_unf_args fn_unf) + + spec_unf_args = ty_args ++ spec_dict_args + spec_unf_subst = CoreSubst.setInScope (se_subst env) + (CoreSubst.substInScope (se_subst rhs_env2)) + -- Extend the in-scope set to satisfy the precondition of + -- specUnfolding, namely that in-scope(unf_subst) includes + -- the free vars of spec_unf_args. The in-scope set of rhs_env2 + -- is just the ticket; but the actual substitution we want is + -- the same old one from 'env' -------------------------------------- -- Adding arity information just propagates it a bit faster @@ -1357,9 +1365,12 @@ bindAuxiliaryDicts env@(SE { se_subst = subst, se_interesting = interesting }) = (env', dx_binds, spec_dict_args) where (dx_binds, spec_dict_args) = go call_ds inst_dict_ids - env' = env { se_subst = CoreSubst.extendIdSubstList subst (orig_dict_ids `zip` spec_dict_args) + env' = env { se_subst = subst `CoreSubst.extendIdSubstList` + (orig_dict_ids `zip` spec_dict_args) + `CoreSubst.extendInScopeList` dx_ids , se_interesting = interesting `unionVarSet` interesting_dicts } + dx_ids = [dx_id | (NonRec dx_id _, _) <- dx_binds] interesting_dicts = mkVarSet [ dx_id | (NonRec dx_id dx, _) <- dx_binds , interestingDict env dx ] -- See Note [Make the new dictionaries interesting] @@ -1367,7 +1378,7 @@ bindAuxiliaryDicts env@(SE { se_subst = subst, se_interesting = interesting }) go :: [CoreExpr] -> [CoreBndr] -> ([DictBind], [CoreExpr]) go [] _ = ([], []) go (dx:dxs) (dx_id:dx_ids) - | exprIsTrivial dx = (dx_binds, dx:args) + | exprIsTrivial dx = (dx_binds, dx : args) | otherwise = (mkDB (NonRec dx_id dx) : dx_binds, Var dx_id : args) where (dx_binds, args) = go dxs dx_ids |