[project @ 1996-07-25 20:43:49 by partain]
[ghc-hetmet.git] / ghc / compiler / coreSyn / FreeVars.lhs
index 54a2426..979fd67 100644 (file)
@@ -1,5 +1,5 @@
 %
-% (c) The GRASP/AQUA Project, Glasgow University, 1992-1995
+% (c) The GRASP/AQUA Project, Glasgow University, 1992-1996
 %
 Taken quite directly from the Peyton Jones/Lester paper.
 
@@ -9,46 +9,37 @@ Taken quite directly from the Peyton Jones/Lester paper.
 module FreeVars (
        freeVars,
 
-#ifdef DPH
--- ToDo: DPH: you should probably use addExprFVs now... [WDP]
-       freeStuff,      -- Need a function that gives fvs of 
-                       -- an expression. I therefore need a 
-                       -- way of passing in candidates or top 
-                       -- level will always be empty.
-#endif {- Data Parallel Haskell -}
-
        -- cheap and cheerful variant...
        addTopBindsFVs,
 
        freeVarsOf, freeTyVarsOf,
-       FVCoreExpr(..), FVCoreBinding(..),
-
-       CoreExprWithFVs(..),            -- For the above functions
-       AnnCoreExpr(..),                -- Dito 
-       FVInfo(..), LeakInfo(..),
+       SYN_IE(FVCoreExpr), SYN_IE(FVCoreBinding),
 
-       -- and to make the interface self-sufficient...
-       CoreExpr, Id, IdSet(..), TyVarSet(..), UniqSet(..), UniType,
-       AnnCoreExpr', AnnCoreBinding, AnnCoreCaseAlternatives,
-       AnnCoreCaseDefault
+       SYN_IE(CoreExprWithFVs),                -- For the above functions
+       SYN_IE(AnnCoreExpr),            -- Dito
+       FVInfo(..), LeakInfo(..)
     ) where
 
+IMP_Ubiq(){-uitous-}
 
-import PlainCore       -- input
 import AnnCoreSyn      -- output
 
-import AbsPrel         ( PrimOp(..), PrimKind -- for CCallOp
-                         IF_ATTACK_PRAGMAS(COMMA tagOf_PrimOp)
-                         IF_ATTACK_PRAGMAS(COMMA pprPrimOp)
+import CoreSyn
+import Id              ( idType, getIdArity, isBottomingId,
+                         emptyIdSet, unitIdSet, mkIdSet,
+                         elementOfIdSet, minusIdSet, unionManyIdSets,
+                         SYN_IE(IdSet)
+                       )
+import IdInfo          ( arityMaybe )
+import PrimOp          ( PrimOp(..) )
+import Type            ( tyVarsOfType )
+import TyVar           ( emptyTyVarSet, unitTyVarSet, minusTyVarSet,
+                         intersectTyVarSets,
+                         SYN_IE(TyVarSet)
                        )
-import AbsUniType      ( extractTyVarsFromTy )
-import BasicLit                ( typeOfBasicLit )
-import Id              ( getIdUniType, getIdArity, toplevelishId, isBottomingId )
-import IdInfo          -- Wanted for arityMaybe, but it seems you have
-                       -- to import it all...  (Death to the Instance Virus!)
-import Maybes
-import UniqSet
-import Util
+import UniqSet         ( unionUniqSets )
+import Usage           ( SYN_IE(UVar) )
+import Util            ( panic, assertPanic )
 \end{code}
 
 %************************************************************************
@@ -68,35 +59,36 @@ I've half-convinced myself we don't for case- and letrec bound ids
 but I might be wrong. (SLPJ, date unknown)
 
 \begin{code}
-type CoreExprWithFVs =  AnnCoreExpr Id Id FVInfo
+type CoreExprWithFVs =  AnnCoreExpr Id Id TyVar UVar FVInfo
 
 type TyVarCands = TyVarSet  -- for when we carry around lists of
 type IdCands   = IdSet     -- "candidate" TyVars/Ids.
-noTyVarCands    = emptyUniqSet
-noIdCands       = emptyUniqSet
-
-data FVInfo = FVInfo 
-               IdSet       -- Free ids
-               TyVarSet    -- Free tyvars
-               LeakInfo
-
-noFreeIds      = emptyUniqSet
-noFreeTyVars   = emptyUniqSet
-aFreeId i      = singletonUniqSet i
-aFreeTyVar t   = singletonUniqSet t
-is_among       = elementOfUniqSet
-combine               = unionUniqSets
-munge_id_ty  i = mkUniqSet (extractTyVarsFromTy (getIdUniType i))
+noTyVarCands    = emptyTyVarSet
+noIdCands       = emptyIdSet
+
+data FVInfo
+  = FVInfo  IdSet      -- Free ids
+           TyVarSet    -- Free tyvars
+           LeakInfo
+
+noFreeIds      = emptyIdSet
+noFreeTyVars   = emptyTyVarSet
+noFreeAnything = (noFreeIds, noFreeTyVars)
+aFreeId i      = unitIdSet i
+aFreeTyVar t   = unitTyVarSet t
+is_among       = elementOfIdSet
+munge_id_ty  i = tyVarsOfType (idType i)
+combine               = unionUniqSets -- used both for {Id,TyVar}Sets
 
 combineFVInfo (FVInfo fvs1 tfvs1 leak1) (FVInfo fvs2 tfvs2 leak2)
-  = FVInfo (fvs1  `combine` fvs2) 
-          (tfvs1 `combine` tfvs2) 
-          (leak1 `orLeak`        leak2)
+  = FVInfo (fvs1  `combine` fvs2)
+          (tfvs1 `combine` tfvs2)
+          (leak1 `orLeak`  leak2)
 \end{code}
 
-Leak-free-ness is based only on the value, not the type.
-In particular, nested collections of constructors are guaranteed leak free.
-Function applications are not, except for PAPs.
+Leak-free-ness is based only on the value, not the type.  In
+particular, nested collections of constructors are guaranteed leak
+free.  Function applications are not, except for PAPs.
 
 Applications of error gets (LeakFree bigArity) -- a hack!
 
@@ -119,12 +111,16 @@ orLeak (LeakFree n) (LeakFree m) = LeakFree (n `min` m)
 
 Main public interface:
 \begin{code}
-freeVars :: PlainCoreExpr -> CoreExprWithFVs
+freeVars :: CoreExpr -> CoreExprWithFVs
 
 freeVars expr = fvExpr noIdCands noTyVarCands expr
 \end{code}
 
+%************************************************************************
+%*                                                                     *
 \subsection{Free variables (and types)}
+%*                                                                     *
+%************************************************************************
 
 We do the free-variable stuff by passing around ``candidates lists''
 of @Ids@ and @TyVars@ that may be considered free.  This is useful,
@@ -135,16 +131,16 @@ put them on the candidates list.
 
 fvExpr :: IdCands          -- In-scope Ids
        -> TyVarCands       -- In-scope tyvars
-       -> PlainCoreExpr 
+       -> CoreExpr
        -> CoreExprWithFVs
 
-fvExpr id_cands tyvar_cands (CoVar v) 
+fvExpr id_cands tyvar_cands (Var v)
   = (FVInfo (if (v `is_among` id_cands)
             then aFreeId v
             else noFreeIds)
            noFreeTyVars
            leakiness,
-     AnnCoVar v)
+     AnnVar v)
   where
     leakiness
       | isBottomingId v = lEAK_FREE_BIG        -- Hack
@@ -152,101 +148,95 @@ fvExpr id_cands tyvar_cands (CoVar v)
                            Nothing    -> lEAK_FREE_0
                            Just arity -> LeakFree arity
 
-fvExpr id_cands tyvar_cands (CoLit k) 
-  = (FVInfo noFreeIds noFreeTyVars lEAK_FREE_0, AnnCoLit k)
+fvExpr id_cands tyvar_cands (Lit k)
+  = (FVInfo noFreeIds noFreeTyVars lEAK_FREE_0, AnnLit k)
 
-fvExpr id_cands tyvar_cands (CoCon c tys args)
-  = (FVInfo args_fvs tfvs lEAK_FREE_0, AnnCoCon c tys args)
+fvExpr id_cands tyvar_cands (Con c args)
+  = (FVInfo args_fvs tfvs lEAK_FREE_0, AnnCon c args)
   where
-    args_fvs = foldr (combine . freeAtom id_cands)  noFreeIds    args
-    tfvs     = foldr (combine . freeTy tyvar_cands) noFreeTyVars tys
+    (args_fvs, tfvs) = freeArgs id_cands tyvar_cands args
 
-fvExpr id_cands tyvar_cands (CoPrim op@(CCallOp _ _ _ _ res_ty) tys args)
-  = ASSERT (null tys)
-    (FVInfo args_fvs tfvs lEAK_FREE_0, AnnCoPrim op tys args)
+fvExpr id_cands tyvar_cands (Prim op args)
+  = (FVInfo args_fvs tfvs lEAK_FREE_0, AnnPrim op args)
   where
-    args_fvs = foldr (combine . freeAtom id_cands)  noFreeIds    args
-    tfvs     = foldr (combine . freeTy tyvar_cands) noFreeTyVars (res_ty:tys)
+    (args_fvs, tfvs) = freeArgs id_cands tyvar_cands args_to_use{-NB-}
+    args_to_use
+      = case op of
+         CCallOp _ _ _ _ res_ty -> TyArg res_ty : args
+         _                      -> args
 
-fvExpr id_cands tyvar_cands (CoPrim op tys args)
-  = (FVInfo args_fvs tfvs lEAK_FREE_0, AnnCoPrim op tys args)
-  where
-    args_fvs = foldr (combine . freeAtom id_cands)  noFreeIds    args
-    tfvs     = foldr (combine . freeTy tyvar_cands) noFreeTyVars tys
+-- this Lam stuff could probably be improved by rewriting (WDP 96/03)
 
-fvExpr id_cands tyvar_cands (CoLam binders body)
-  = (FVInfo (freeVarsOf body2   `minusUniqSet`  mkUniqSet binders)
-           (freeTyVarsOf body2 `combine` binder_ftvs)
+fvExpr id_cands tyvar_cands (Lam (UsageBinder uvar) body)
+  = panic "fvExpr:Lam UsageBinder"
+
+fvExpr id_cands tyvar_cands (Lam b@(ValBinder binder) body)
+  = (FVInfo (freeVarsOf body2   `minusIdSet` unitIdSet binder)
+           (freeTyVarsOf body2 `combine`    munge_id_ty binder)
            leakiness,
-     AnnCoLam binders body2)
+     AnnLam b body2)
   where
        -- We need to collect free tyvars from the binders
-    body2 = fvExpr (mkUniqSet binders `combine` id_cands) tyvar_cands body
-
-    binder_ftvs
-      = foldr (combine . munge_id_ty) noFreeTyVars binders
+    body2 = fvExpr (unitIdSet binder `combine` id_cands) tyvar_cands body
 
-    no_args   = length binders
     leakiness = case leakinessOf body2 of
-                 MightLeak  -> LeakFree  no_args
-                 LeakFree n -> LeakFree (n + no_args)
+                 MightLeak  -> LeakFree 1
+                 LeakFree n -> LeakFree (n + 1)
 
-fvExpr id_cands tyvar_cands (CoTyLam tyvar body)
+fvExpr id_cands tyvar_cands (Lam b@(TyBinder tyvar) body)
   = (FVInfo (freeVarsOf body2)
-           (freeTyVarsOf body2 `minusUniqSet` aFreeTyVar tyvar)
+           (freeTyVarsOf body2 `minusTyVarSet` aFreeTyVar tyvar)
            (leakinessOf body2),
-     AnnCoTyLam tyvar body2)
+     AnnLam b body2)
   where
     body2 = fvExpr id_cands (aFreeTyVar tyvar `combine` tyvar_cands) body
 
-fvExpr id_cands tyvar_cands (CoApp fun arg)
-  = (FVInfo (freeVarsOf fun2 `combine` fvs_arg)
-           (freeTyVarsOf fun2)
+-- ditto on rewriting this App stuff (WDP 96/03)
+
+fvExpr id_cands tyvar_cands (App fun arg)
+  = (FVInfo (freeVarsOf fun2   `combine` fvs_arg)
+           (freeTyVarsOf fun2 `combine` tfvs_arg)
            leakiness,
-     AnnCoApp fun2 arg)
+     AnnApp fun2 arg)
   where
     fun2 = fvExpr id_cands tyvar_cands fun
-    fvs_arg = freeAtom id_cands arg
+    fun2_leakiness = leakinessOf fun2
 
-    leakiness = case leakinessOf fun2 of
-                  LeakFree n | n>1 -> LeakFree (n-1)   -- Note > not >=
-                  other            -> MightLeak
+    (fvs_arg, tfvs_arg) = freeArgs id_cands tyvar_cands [arg]
 
-fvExpr id_cands tyvar_cands (CoTyApp expr ty)
-  = (FVInfo (freeVarsOf expr2)
-           (freeTyVarsOf expr2 `combine` tfvs_arg)
-           (leakinessOf expr2),
-     AnnCoTyApp expr2 ty)
-  where
-    expr2    = fvExpr id_cands tyvar_cands expr
-    tfvs_arg = freeTy tyvar_cands ty
+    leakiness = if (notValArg arg) then
+                   fun2_leakiness
+               else
+                   case fun2_leakiness of
+                      LeakFree n | n>1 -> LeakFree (n-1) -- Note > not >=
+                      other            -> MightLeak
 
-fvExpr id_cands tyvar_cands (CoCase expr alts)
+fvExpr id_cands tyvar_cands (Case expr alts)
   = (combineFVInfo expr_fvinfo alts_fvinfo,
-     AnnCoCase expr2 alts')
+     AnnCase expr2 alts')
   where
     expr2@(expr_fvinfo,_) = fvExpr id_cands tyvar_cands expr
     (alts_fvinfo, alts') = annotate_alts alts
 
-    annotate_alts (CoAlgAlts alts deflt)
-      = (fvinfo, AnnCoAlgAlts alts' deflt')
+    annotate_alts (AlgAlts alts deflt)
+      = (fvinfo, AnnAlgAlts alts' deflt')
       where
        (alts_fvinfo_s, alts') = unzip (map ann_boxed_alt alts)
        (deflt_fvinfo, deflt') = annotate_default deflt
-        fvinfo = foldr combineFVInfo deflt_fvinfo alts_fvinfo_s
+       fvinfo = foldr combineFVInfo deflt_fvinfo alts_fvinfo_s
 
        ann_boxed_alt (con, params, rhs)
-         = (FVInfo (freeVarsOf rhs' `minusUniqSet` mkUniqSet params)
+         = (FVInfo (freeVarsOf rhs' `minusIdSet` mkIdSet params)
                    (freeTyVarsOf rhs' `combine` param_ftvs)
                    (leakinessOf rhs'),
             (con, params, rhs'))
          where
-           rhs' = fvExpr (mkUniqSet params `combine` id_cands) tyvar_cands rhs
+           rhs' = fvExpr (mkIdSet params `combine` id_cands) tyvar_cands rhs
            param_ftvs = foldr (combine . munge_id_ty) noFreeTyVars params
                -- We need to collect free tyvars from the binders
 
-    annotate_alts (CoPrimAlts alts deflt)
-      = (fvinfo, AnnCoPrimAlts alts' deflt')
+    annotate_alts (PrimAlts alts deflt)
+      = (fvinfo, AnnPrimAlts alts' deflt')
       where
        (alts_fvinfo_s, alts') = unzip (map ann_unboxed_alt alts)
        (deflt_fvinfo, deflt') = annotate_default deflt
@@ -256,128 +246,94 @@ fvExpr id_cands tyvar_cands (CoCase expr alts)
          where
            rhs'@(rhs_info,_) = fvExpr id_cands tyvar_cands rhs
 
-#ifdef DPH
-    annotate_alts id_cands tyvar_cands (CoParAlgAlts tycon ctxt binders alts deflt)
-      = ((alts_fvs `minusUniqSet` (mkUniqSet binders)) `combine` deflt_fvs,
-        AnnCoParAlgAlts tycon ctxt binders alts' deflt')
-      where
-       (alts_fvs_sets,  alts') = unzip (map (ann_boxed_par_alt id_cands tyvar_cands) alts)
-       alts_fvs                = unionManyUniqSets alts_fvs_sets
-       (deflt_fvs, ???ToDo:DPH, deflt')        = annotate_default deflt
-
-       ann_boxed_par_alt id_cands tyvar_cands (con, rhs)
-         = (rhs_fvs, (con, rhs'))
-         where
-           rhs'     = fvExpr (mkUniqSet binders `combine` id_cands) tyvar_cands rhs
-           rhs_fvs  = freeVarsOf rhs'
-
-    annotate_alts id_cands tyvar_cands (CoParPrimAlts tycon ctxt alts deflt)
-      = (alts_fvs `combine` deflt_fvs,
-        AnnCoParPrimAlts tycon ctxt alts' deflt')
-      where
-       (alts_fvs_sets,  alts') = unzip (map (ann_unboxed_par_alt id_cands tyvar_cands) alts)
-       alts_fvs                = unionManyUniqSets alts_fvs_sets
-       (deflt_fvs, ??? ToDo:DPH, deflt')       = annotate_default deflt
-
-       ann_unboxed_par_alt id_cands tyvar_cands (lit, rhs)
-         = (rhs_fvs, (lit, rhs'))
-         where
-           rhs'     = fvExpr id_cands tyvar_cands rhs
-           rhs_fvs  = freeVarsOf rhs'
-#endif {- Data Parallel Haskell -}
+    annotate_default NoDefault = (FVInfo noFreeIds noFreeTyVars lEAK_FREE_BIG,
+                                   AnnNoDefault)
 
-    annotate_default CoNoDefault = (FVInfo noFreeIds noFreeTyVars lEAK_FREE_BIG, 
-                                   AnnCoNoDefault)
-
-    annotate_default (CoBindDefault binder rhs)
-      = (FVInfo (freeVarsOf   rhs' `minusUniqSet` aFreeId binder)
+    annotate_default (BindDefault binder rhs)
+      = (FVInfo (freeVarsOf   rhs' `minusIdSet` aFreeId binder)
                (freeTyVarsOf rhs' `combine` binder_ftvs)
                (leakinessOf rhs'),
-        AnnCoBindDefault binder rhs')
+        AnnBindDefault binder rhs')
       where
        rhs' = fvExpr (aFreeId binder `combine` id_cands) tyvar_cands rhs
        binder_ftvs = munge_id_ty binder
            -- We need to collect free tyvars from the binder
 
-fvExpr id_cands tyvar_cands (CoLet (CoNonRec binder rhs) body)
+fvExpr id_cands tyvar_cands (Let (NonRec binder rhs) body)
   = (FVInfo (freeVarsOf rhs'   `combine` body_fvs)
            (freeTyVarsOf rhs' `combine` freeTyVarsOf body2 `combine` binder_ftvs)
            (leakinessOf rhs' `orLeak` leakinessOf body2),
-     AnnCoLet (AnnCoNonRec binder rhs') body2)
+     AnnLet (AnnNonRec binder rhs') body2)
   where
     rhs'       = fvExpr id_cands tyvar_cands rhs
     body2      = fvExpr (aFreeId binder `combine` id_cands) tyvar_cands body
-    body_fvs   = freeVarsOf body2 `minusUniqSet` aFreeId binder
+    body_fvs   = freeVarsOf body2 `minusIdSet` aFreeId binder
     binder_ftvs = munge_id_ty binder
        -- We need to collect free tyvars from the binder
 
-fvExpr id_cands tyvar_cands (CoLet (CoRec binds) body)
+fvExpr id_cands tyvar_cands (Let (Rec binds) body)
   = (FVInfo (binds_fvs `combine` body_fvs)
            (rhss_tfvs `combine` freeTyVarsOf body2 `combine` binders_ftvs)
            (leakiness_of_rhss `orLeak` leakinessOf body2),
-     AnnCoLet (AnnCoRec (binders `zip` rhss')) body2)
+     AnnLet (AnnRec (binders `zip` rhss')) body2)
   where
     (binders, rhss)   = unzip binds
     new_id_cands      = binders_set `combine` id_cands
-    binders_set              = mkUniqSet binders
+    binders_set              = mkIdSet binders
     rhss'            = map (fvExpr new_id_cands tyvar_cands) rhss
 
     FVInfo rhss_fvs rhss_tfvs leakiness_of_rhss
        = foldr1 combineFVInfo [info | (info,_) <- rhss']
 
-    binds_fvs        = rhss_fvs `minusUniqSet` binders_set
+    binds_fvs        = rhss_fvs `minusIdSet` binders_set
     body2            = fvExpr new_id_cands tyvar_cands body
-    body_fvs         = freeVarsOf body2 `minusUniqSet` binders_set
+    body_fvs         = freeVarsOf body2 `minusIdSet` binders_set
     binders_ftvs      = foldr (combine . munge_id_ty) noFreeTyVars binders
        -- We need to collect free tyvars from the binders
 
-fvExpr id_cands tyvar_cands (CoSCC label expr)
-  = (fvinfo, AnnCoSCC label expr2)
+fvExpr id_cands tyvar_cands (SCC label expr)
+  = (fvinfo, AnnSCC label expr2)
   where
     expr2@(fvinfo,_) = fvExpr id_cands tyvar_cands expr
 
-#ifdef DPH
-fvExpr id_cands tyvar_cands e@(CoParCon c ctxt tys args)
-  = ((args_fvs, typeOfCoreExpr e), AnnCoParCon c ctxt tys args')
-  where
-    args'      = map (fvExpr id_cands tyvar_cands) args
-    args_fvs   = unionManyUniqSets [ fvs | ((fvs,_), _) <- args' ]
-
-fvExpr id_cands tyvar_cands e@(CoParComm ctxt expr comm)
-  = ((expr_fvs `combine` comm_fvs, tyOf expr2), AnnCoParComm ctxt expr2 comm')
+fvExpr id_cands tyvar_cands (Coerce c ty expr)
+  = (FVInfo (freeVarsOf   expr2)
+           (freeTyVarsOf expr2 `combine` tfvs)
+           (leakinessOf  expr2),
+     AnnCoerce c ty expr2)
   where
-    expr2            = fvExpr id_cands tyvar_cands expr
-    expr_fvs         = freeVarsOf expr2
-    (comm_fvs,comm') = free_stuff_comm id_cands tyvar_cands comm
-
-    free_stuff_comm id_cands tyvar_cands (CoParSend exprs)
-      = let exprs'    = map (fvExpr id_cands tyvar_cands) exprs                        in
-       let exprs_fvs = unionManyUniqSets [fvs | ((fvs,_), _) <- exprs' ]  in
-        (exprs_fvs,AnnCoParSend exprs')
-
-    free_stuff_comm id_cands tyvar_cands (CoParFetch exprs)
-      = let exprs'    = map (fvExpr id_cands tyvar_cands) exprs                        in
-       let exprs_fvs = unionManyUniqSets [fvs | ((fvs,_), _) <- exprs' ]  in
-        (exprs_fvs,AnnCoParFetch exprs')
-
-    free_stuff_comm id_cands tyvar_cands (CoToPodized)
-      = (emptyUniqSet, AnnCoToPodized)
-
-    free_stuff_comm id_cands tyvar_cands (CoFromPodized)
-      = (emptyUniqSet, AnnCoFromPodized)     
-#endif {- Data Parallel Haskell -}
+    expr2 = fvExpr id_cands tyvar_cands expr
+    tfvs  = freeTy tyvar_cands ty
 \end{code}
 
 \begin{code}
-freeAtom :: IdCands -> PlainCoreAtom ->  IdSet
+freeArgs :: IdCands -> TyVarCands
+        -> [CoreArg]
+        -> (IdSet, TyVarSet)
 
-freeAtom cands (CoLitAtom k) = noFreeIds
-freeAtom cands (CoVarAtom v) | v `is_among` cands = aFreeId v
-                            | otherwise          = noFreeIds
+freeArgs icands tcands [] = noFreeAnything
+freeArgs icands tcands (arg:args)
+  -- this code is written this funny way only for "efficiency" purposes
+  = let
+       free_first_arg@(arg_fvs, tfvs) = free_arg arg
+    in
+    if (null args) then
+       free_first_arg
+    else
+       case (freeArgs icands tcands args) of { (irest, trest) ->
+       (arg_fvs `combine` irest, tfvs `combine` trest) }
+  where
+    free_arg (LitArg   _) = noFreeAnything
+    free_arg (UsageArg _) = noFreeAnything
+    free_arg (TyArg   ty) = (noFreeIds, freeTy tcands ty)
+    free_arg (VarArg   v)
+      | v `is_among` icands = (aFreeId v, noFreeTyVars)
+      | otherwise          = noFreeAnything
 
-freeTy :: TyVarCands -> UniType -> TyVarSet
+---------
+freeTy :: TyVarCands -> Type -> TyVarSet
 
-freeTy cands ty = mkUniqSet (extractTyVarsFromTy ty) `intersectUniqSets` cands
+freeTy cands ty = tyVarsOfType ty `intersectTyVarSets` cands
 
 freeVarsOf :: CoreExprWithFVs -> IdSet
 freeVarsOf (FVInfo free_vars _ _, _) = free_vars
@@ -409,7 +365,7 @@ expression!
   The free vars attached to a let(rec) binder are the free vars of the
   rhs of the binding.  In the case of letrecs, this set excludes the
   binders themselves.
-\item  
+\item
   The free vars attached to a case alternative binder are the free
   vars of the alternative, excluding the alternative's binders.
 \end{itemize}
@@ -417,7 +373,7 @@ expression!
 There's a predicate carried in which tells what is a free-var
 candidate. It is passed the Id and a set of in-scope Ids.
 
-(Global) constructors used on the rhs in a CoCon are also treated as
+(Global) constructors used on the rhs in a Con are also treated as
 potential free-var candidates (though they will not be recorded in the
 in-scope set). The predicate must decide if they are to be recorded as
 free-vars.
@@ -426,8 +382,8 @@ As it happens this is only ever used by the Specialiser!
 
 \begin{code}
 type FVCoreBinder  = (Id, IdSet)
-type FVCoreExpr    = CoreExpr    FVCoreBinder Id
-type FVCoreBinding = CoreBinding FVCoreBinder Id
+type FVCoreExpr    = GenCoreExpr    FVCoreBinder Id TyVar UVar
+type FVCoreBinding = GenCoreBinding FVCoreBinder Id TyVar UVar
 
 type InterestingIdFun
   =  IdSet     -- Non-top-level in-scope variables
@@ -438,131 +394,127 @@ type InterestingIdFun
 \begin{code}
 addExprFVs :: InterestingIdFun -- "Interesting id" predicate
           -> IdSet             -- In scope ids
-          -> PlainCoreExpr
+          -> CoreExpr
           -> (FVCoreExpr, IdSet)
 
-addExprFVs fv_cand in_scope (CoVar v)
-  = (CoVar v, if fv_cand in_scope v
+addExprFVs fv_cand in_scope (Var v)
+  = (Var v, if fv_cand in_scope v
              then aFreeId v
              else noFreeIds)
 
-addExprFVs fv_cand in_scope (CoLit lit) = (CoLit lit, noFreeIds)
+addExprFVs fv_cand in_scope (Lit lit) = (Lit lit, noFreeIds)
 
-addExprFVs fv_cand in_scope (CoCon con tys args) 
-  = (CoCon con tys args,
-     if fv_cand in_scope con 
+addExprFVs fv_cand in_scope (Con con args)
+  = (Con con args,
+     if fv_cand in_scope con
      then aFreeId con
-     else noFreeIds
-       `combine`
-     unionManyUniqSets (map (fvsOfAtom fv_cand in_scope) args))
+     else noFreeIds `combine` fvsOfArgs fv_cand in_scope args)
 
-addExprFVs fv_cand in_scope (CoPrim op tys args) 
-  = (CoPrim op tys args,
-     unionManyUniqSets (map (fvsOfAtom fv_cand in_scope) args))
+addExprFVs fv_cand in_scope (Prim op args)
+  = (Prim op args, fvsOfArgs fv_cand in_scope args)
 
-addExprFVs fv_cand in_scope (CoLam binders body)
-  = (CoLam (binders `zip` (repeat lam_fvs)) new_body, lam_fvs)
+addExprFVs fv_cand in_scope (Lam binder body)
+  = (Lam new_binder new_body, lam_fvs)
   where
-    binder_set = mkUniqSet binders
-    new_in_scope = in_scope `combine` binder_set
+    (new_binder, binder_set)
+      = case binder of
+         TyBinder    t -> (TyBinder t, emptyIdSet)
+         UsageBinder u -> (UsageBinder u, emptyIdSet)
+          ValBinder   b -> (ValBinder (b, lam_fvs),
+                           unitIdSet b)
+
+    new_in_scope        = in_scope `combine` binder_set
     (new_body, body_fvs) = addExprFVs fv_cand new_in_scope body
-    lam_fvs = body_fvs `minusUniqSet` binder_set
-
-addExprFVs fv_cand in_scope (CoTyLam tyvar body)
-  = (CoTyLam tyvar body2, body_fvs)
-  where
-    (body2, body_fvs) = addExprFVs fv_cand in_scope body
+    lam_fvs             = body_fvs `minusIdSet` binder_set
 
-addExprFVs fv_cand in_scope (CoApp fun arg)
-  = (CoApp fun2 arg, fun_fvs `combine` fvsOfAtom fv_cand in_scope arg)
+addExprFVs fv_cand in_scope (App fun arg)
+  = (App fun2 arg, fun_fvs `combine` fvsOfArgs fv_cand in_scope [arg])
   where
     (fun2, fun_fvs) = addExprFVs fv_cand in_scope fun
 
-addExprFVs fv_cand in_scope (CoTyApp fun ty)
-  = (CoTyApp fun2 ty, fun_fvs)
-  where
-    (fun2, fun_fvs) = addExprFVs fv_cand in_scope fun
-
-addExprFVs fv_cand in_scope (CoCase scrut alts)
-  = (CoCase scrut' alts', scrut_fvs `combine` alts_fvs)
+addExprFVs fv_cand in_scope (Case scrut alts)
+  = (Case scrut' alts', scrut_fvs `combine` alts_fvs)
   where
     (scrut', scrut_fvs) = addExprFVs fv_cand in_scope scrut
 
     (alts', alts_fvs)
       = case alts of
-         CoAlgAlts alg_alts deflt -> (CoAlgAlts alg_alts' deflt', fvs)
+         AlgAlts alg_alts deflt -> (AlgAlts alg_alts' deflt', fvs)
            where
              (alg_alts', alt_fvs) = unzip (map do_alg_alt alg_alts)
              (deflt', deflt_fvs) = do_deflt deflt
-             fvs = unionManyUniqSets (deflt_fvs : alt_fvs)
+             fvs = unionManyIdSets (deflt_fvs : alt_fvs)
 
-         CoPrimAlts prim_alts deflt -> (CoPrimAlts prim_alts' deflt', fvs)
+         PrimAlts prim_alts deflt -> (PrimAlts prim_alts' deflt', fvs)
            where
              (prim_alts', alt_fvs) = unzip (map do_prim_alt prim_alts)
              (deflt', deflt_fvs) = do_deflt deflt
-             fvs = unionManyUniqSets (deflt_fvs : alt_fvs)
+             fvs = unionManyIdSets (deflt_fvs : alt_fvs)
 
-    do_alg_alt :: (Id, [Id], PlainCoreExpr)
+    do_alg_alt :: (Id, [Id], CoreExpr)
               -> ((Id, [FVCoreBinder], FVCoreExpr), IdSet)
 
     do_alg_alt (con, args, rhs) = ((con, args `zip` (repeat fvs), rhs'), fvs)
       where
        new_in_scope = in_scope `combine` arg_set
        (rhs', rhs_fvs) = addExprFVs fv_cand new_in_scope rhs
-       fvs = rhs_fvs `minusUniqSet` arg_set
-        arg_set = mkUniqSet args
+       fvs = rhs_fvs `minusIdSet` arg_set
+       arg_set = mkIdSet args
 
     do_prim_alt (lit, rhs) = ((lit, rhs'), rhs_fvs)
       where
        (rhs', rhs_fvs) = addExprFVs fv_cand in_scope rhs
 
-    do_deflt CoNoDefault = (CoNoDefault, noFreeIds)
-    do_deflt (CoBindDefault var rhs)
-      = (CoBindDefault (var,fvs) rhs', fvs)
+    do_deflt NoDefault = (NoDefault, noFreeIds)
+    do_deflt (BindDefault var rhs)
+      = (BindDefault (var,fvs) rhs', fvs)
       where
        new_in_scope = in_scope `combine` var_set
        (rhs', rhs_fvs) = addExprFVs fv_cand new_in_scope rhs
-       fvs = rhs_fvs `minusUniqSet` var_set
-        var_set = aFreeId var
+       fvs = rhs_fvs `minusIdSet` var_set
+       var_set = aFreeId var
 
-addExprFVs fv_cand in_scope (CoLet binds body)
-  = (CoLet binds' body2, fvs_binds `combine` (fvs_body `minusUniqSet` binder_set))
+addExprFVs fv_cand in_scope (Let binds body)
+  = (Let binds' body2, fvs_binds `combine` (fvs_body `minusIdSet` binder_set))
   where
     (binds', fvs_binds, new_in_scope, binder_set)
       = addBindingFVs fv_cand in_scope binds
 
     (body2, fvs_body)  = addExprFVs fv_cand new_in_scope body
 
-addExprFVs fv_cand in_scope (CoSCC label expr)
-  = (CoSCC label expr2, expr_fvs)
+addExprFVs fv_cand in_scope (SCC label expr)
+  = (SCC label expr2, expr_fvs)
   where
     (expr2, expr_fvs) = addExprFVs fv_cand in_scope expr
 
--- ToDo: DPH: add stuff here
+addExprFVs fv_cand in_scope (Coerce c ty expr)
+  = (Coerce c ty expr2, expr_fvs)
+  where
+    (expr2, expr_fvs) = addExprFVs fv_cand in_scope expr
 \end{code}
 
 \begin{code}
 addBindingFVs
            :: InterestingIdFun -- "Interesting id" predicate
            -> IdSet            -- In scope ids
-           -> PlainCoreBinding
+           -> CoreBinding
            -> (FVCoreBinding,
                IdSet,          -- Free vars of binding group
                IdSet,          -- Augmented in-scope Ids
                IdSet)          -- Set of Ids bound by this binding
 
-addBindingFVs fv_cand in_scope (CoNonRec binder rhs)
-  = (CoNonRec binder' rhs', fvs, new_in_scope, binder_set)
-  where 
+addBindingFVs fv_cand in_scope (NonRec binder rhs)
+  = (NonRec binder' rhs', fvs, new_in_scope, binder_set)
+  where
     ((binder', rhs'), fvs) = do_pair fv_cand in_scope binder_set (binder, rhs)
     new_in_scope = in_scope `combine` binder_set
     binder_set = aFreeId binder
 
-addBindingFVs fv_cand in_scope (CoRec pairs)
-  = (CoRec pairs', unionManyUniqSets fvs_s, new_in_scope, binder_set)
+addBindingFVs fv_cand in_scope (Rec pairs)
+  = (Rec pairs', unionManyIdSets fvs_s, new_in_scope, binder_set)
   where
     binders = [binder | (binder,_) <- pairs]
-    binder_set = mkUniqSet binders
+    binder_set = mkIdSet binders
     new_in_scope = in_scope `combine` binder_set
     (pairs', fvs_s) = unzip (map (do_pair fv_cand new_in_scope binder_set) pairs)
 \end{code}
@@ -570,7 +522,7 @@ addBindingFVs fv_cand in_scope (CoRec pairs)
 \begin{code}
 addTopBindsFVs
            :: InterestingIdFun -- "Interesting id" predicate
-           -> [PlainCoreBinding]
+           -> [CoreBinding]
            -> ([FVCoreBinding],
                IdSet)
 
@@ -584,26 +536,31 @@ addTopBindsFVs fv_cand (b:bs)
 \end{code}
 
 \begin{code}
-fvsOfAtom   :: InterestingIdFun        -- "Interesting id" predicate
+fvsOfArgs   :: InterestingIdFun        -- "Interesting id" predicate
            -> IdSet            -- In scope ids
-           -> PlainCoreAtom
+           -> [CoreArg]
            -> IdSet
 
-fvsOfAtom fv_cand in_scope (CoVarAtom v)
-  = if fv_cand in_scope v
-    then aFreeId v
-    else noFreeIds
-fvsOfAtom _ _ _ = noFreeIds -- if a literal...
+fvsOfArgs _ _ [] = noFreeIds
+
+fvsOfArgs fv_cand in_scope [VarArg v] -- this is only a short-cut...
+  = if (fv_cand in_scope v) then aFreeId v else noFreeIds
+fvsOfArgs _      _        [ _ ] = noFreeIds
+
+fvsOfArgs fv_cand in_scope args
+  = mkIdSet [ v | (VarArg v) <- args, fv_cand in_scope v ]
+    -- all other types of args are uninteresting here...
 
+----------
 do_pair        :: InterestingIdFun -- "Interesting id" predicate
        -> IdSet            -- In scope ids
        -> IdSet
-       -> (Id, PlainCoreExpr)
+       -> (Id, CoreExpr)
        -> ((FVCoreBinder, FVCoreExpr), IdSet)
 
 do_pair fv_cand in_scope binder_set (binder,rhs)
  = (((binder, fvs), rhs'), fvs)
  where
    (rhs', rhs_fvs) = addExprFVs fv_cand in_scope rhs
-   fvs = rhs_fvs `minusUniqSet` binder_set
+   fvs = rhs_fvs `minusIdSet` binder_set
 \end{code}