[project @ 2003-06-02 13:27:53 by simonpj]
[ghc-hetmet.git] / ghc / compiler / typecheck / TcIfaceSig.lhs
index 40cc5df..5295fec 100644 (file)
@@ -4,52 +4,44 @@
 \section[TcIfaceSig]{Type checking of type signatures in interface files}
 
 \begin{code}
-module TcIfaceSig ( tcInterfaceSigs ) where
+module TcIfaceSig ( tcInterfaceSigs,
+                   tcCoreExpr,
+                   tcCoreLamBndrs,
+                   tcCoreBinds ) where
 
 #include "HsVersions.h"
 
-import HsSyn           ( HsDecl(..), IfaceSig(..) )
-import TcMonad
-import TcMonoType      ( tcHsType, tcHsTypeKind, 
-                               -- NB: all the tyars in interface files are kinded,
-                               -- so tcHsType will do the Right Thing without
-                               -- having to mess about with zonking
-                         tcExtendTyVarScope
-                       )
-import TcEnv           ( ValueEnv, tcExtendTyVarEnv, 
-                         tcExtendGlobalValEnv, tcSetValueEnv,
-                         tcLookupTyConByKey, tcLookupValueMaybe,
-                         explicitLookupValue, badCon, badPrimOp
-                       )
-import TcType          ( TcKind, kindToTcKind )
-
-import RnHsSyn         ( RenamedHsDecl )
+import HsSyn           ( CoreDecl(..), TyClDecl(..), HsTupCon(..) )
+import TcHsSyn         ( TypecheckedCoreBind )
+import TcRnTypes
+import TcRnMonad
+import TcMonoType      ( tcIfaceType, kcHsSigType )
+import TcEnv           ( tcExtendTyVarEnv, tcExtendGlobalValEnv, tcLookupGlobalId,
+                         tcLookupDataCon )
+
+import RnHsSyn         ( RenamedCoreDecl, RenamedTyClDecl )
 import HsCore
-import HsDecls         ( HsIdInfo(..), HsStrictnessInfo(..) )
-import CallConv                ( cCallConv )
-import Const           ( Con(..), Literal(..) )
+import Literal         ( Literal(..) )
 import CoreSyn
-import CoreUtils       ( coreExprType )
+import CoreUtils       ( exprType )
 import CoreUnfold
-import WwLib           ( mkWrapper )
-import PrimOp          ( PrimOp(..) )
+import CoreLint                ( lintUnfolding )
+import WorkWrap                ( mkWrapper )
 
-import Id              ( Id, mkImportedId, mkUserId,
-                         isPrimitiveId_maybe, isDataConId_maybe
-                       )
+import Id              ( Id, mkVanillaGlobal, mkLocalId )
+import MkId            ( mkFCallId )
 import IdInfo
-import DataCon         ( dataConSig, dataConArgTys )
-import SpecEnv         ( addToSpecEnv )
-import Type            ( mkSynTy, mkTyVarTys, splitAlgTyConApp )
+import TyCon           ( tyConDataCons, tyConTyVars )
+import DataCon         ( DataCon, dataConWorkId, dataConExistentialTyVars, dataConArgTys )
+import Type            ( mkTyVarTys, splitTyConApp )
+import TysWiredIn      ( tupleCon )
 import Var             ( mkTyVar, tyVarKind )
-import VarEnv
-import Name            ( Name, NamedThing(..) )
-import Unique          ( rationalTyConKey )
-import TysWiredIn      ( integerTy, stringTy )
-import ErrUtils                ( pprBagOfErrors )
-import Maybes          ( maybeToBool, MaybeErr(..) )
+import Name            ( Name )
+import UniqSupply      ( initUs_ )
 import Outputable      
-import Util            ( zipWithEqual )
+import Util            ( zipWithEqual, dropList, equalLength )
+import HscTypes                ( typeEnvIds )
+import CmdLineOpts     ( DynFlag(..) )
 \end{code}
 
 Ultimately, type signatures in interfaces will have pragmatic
@@ -60,271 +52,256 @@ As always, we do not have to worry about user-pragmas in interface
 signatures.
 
 \begin{code}
-tcInterfaceSigs :: ValueEnv            -- Envt to use when checking unfoldings
-               -> [RenamedHsDecl]      -- Ignore non-sig-decls in these decls
-               -> TcM s [Id]
+tcInterfaceSigs :: [RenamedTyClDecl]   -- Ignore non-sig-decls in these decls
+               -> TcM TcGblEnv
                
-
-tcInterfaceSigs unf_env (SigD (IfaceSig name ty id_infos src_loc) : rest)
-  = tcAddSrcLoc src_loc (
-    tcAddErrCtxt (ifaceSigCtxt name) (
-       tcHsType ty                                             `thenTc` \ sigma_ty ->
-       tcIdInfo unf_env name sigma_ty noIdInfo id_infos        `thenTc` \ id_info ->
-       returnTc (mkImportedId name sigma_ty id_info)
-    ))                                         `thenTc` \ sig_id ->
-    tcInterfaceSigs unf_env rest               `thenTc` \ sig_ids ->
-    returnTc (sig_id : sig_ids)
-
-tcInterfaceSigs unf_env (other_decl : rest) = tcInterfaceSigs unf_env rest
-
-tcInterfaceSigs unf_env [] = returnTc []
+-- May 2003: 
+--     NOTE 1: careful about the side-effected EPS
+--             in the two tcExtendGlobalValueEnv calls
+--     NOTE 2: no point in tying the knot with fixM; all
+--             the important knot-tying comes via the PCS global variable
+
+tcInterfaceSigs decls = 
+  zapEnv (fixM (tc_interface_sigs decls)) `thenM` \ (_,sig_ids) ->
+       -- The zapEnv dramatically trims the environment, solely
+       -- to plug the space leak that would otherwise be caused
+       -- by a rich environment bound into lots of lazy thunks
+       -- The thunks are the lazily-typechecked IdInfo of the 
+       -- imported things.
+
+  tcExtendGlobalValEnv sig_ids getGblEnv  `thenM` \ gbl_env ->
+  returnM gbl_env
+       -- We tie a knot so that the Ids read out of interfaces are in scope
+       --   when we read their pragmas.
+       -- What we rely on is that pragmas are typechecked lazily; if
+       --   any type errors are found (ie there's an inconsistency)
+       --   we silently discard the pragma
+       --
+       -- NOTE ALSO: the knot is in two parts:
+       --      * Ids defined in this module are added to the typechecker envt
+       --        which is knot-tied by the fixM.
+       --      * Imported Ids are side-effected into the PCS by the 
+       --        tcExtendGlobalValueEnv, so they will be seen there provided
+       --        we don't look them up too early. 
+       --      In both cases, we must defer lookups until after the knot is tied
+       --
+       -- We used to have a much bigger loop (in TcRnDriver), so that the 
+       -- interface pragmas could mention variables bound in this module 
+       -- (by mutual recn), but
+       --     (a) the knot is tiresomely big, and 
+       --     (b) it black-holes when we have Template Haskell
+       --
+       -- For (b) consider: f = $(...h....)
+       -- where h is imported, and calls f via an hi-boot file.  
+       -- This is bad!  But it is not seen as a staging error, because h
+       -- is indeed imported.  We don't want the type-checker to black-hole 
+       -- when simplifying and compiling the splice!
+       --
+       -- Simple solution: discard any unfolding that mentions a variable
+       -- bound in this module (and hence not yet processed).
+       -- The discarding happens when forkM finds a type error.
+
+tc_interface_sigs decls ~(unf_env, _)
+  = sequenceM [do_one d | d@(IfaceSig {}) <- decls]    `thenM` \ sig_ids ->
+    tcExtendGlobalValEnv sig_ids getGblEnv             `thenM` \ gbl_env ->
+    returnM (gbl_env, sig_ids)
+  where
+    in_scope_vars = typeEnvIds (tcg_type_env unf_env)
+       -- When we have hi-boot files, an unfolding might refer to
+       -- something defined in this module, so we must build a
+       -- suitable in-scope set.  This thunk will only be poked
+       -- if -dcore-lint is on.
+
+    do_one IfaceSig {tcdName   = name,     tcdType = ty, 
+                    tcdIdInfo = id_infos, tcdLoc  = src_loc}
+      = addSrcLoc src_loc                      $       
+       addErrCtxt (ifaceSigCtxt name)          $
+       tcIfaceType ty                          `thenM` \ sigma_ty ->
+       tcIdInfo unf_env in_scope_vars name 
+                sigma_ty id_infos              `thenM` \ id_info ->
+       returnM (mkVanillaGlobal name sigma_ty id_info)
 \end{code}
 
 \begin{code}
-tcIdInfo unf_env name ty info info_ins
-  = foldlTc tcPrag noIdInfo info_ins
+tcIdInfo unf_env in_scope_vars name ty info_ins
+  = setGblEnv unf_env $
+       -- Use the knot-tied environment for the IdInfo
+       -- In particular: typechecking unfoldings and worker names
+    foldlM tcPrag init_info info_ins 
   where
-    tcPrag info (HsArity arity) = returnTc (arity `setArityInfo` info)
-    tcPrag info (HsUpdate upd)  = returnTc (upd   `setUpdateInfo` info)
-    tcPrag info (HsNoCafRefs)   = returnTc (NoCafRefs `setCafInfo` info)
-
-    tcPrag info (HsUnfold inline_prag maybe_expr)
-       = (case maybe_expr of
-               Just expr -> tcPragExpr unf_env name expr
-               Nothing   -> returnNF_Tc Nothing
-         )                                     `thenNF_Tc` \ maybe_expr' ->
-         let
-               -- maybe_expr doesn't get looked at if the unfolding
-               -- is never inspected; so the typecheck doesn't even happen
-               unfold_info = case maybe_expr' of
-                               Nothing    -> NoUnfolding
-                               Just expr' -> mkUnfolding expr' 
-               info1 = unfold_info `setUnfoldingInfo` info
-
-               info2 = inline_prag `setInlinePragInfo` info1
-         in
-         returnTc info2
+    -- Set the CgInfo to something sensible but uninformative before
+    -- we start; default assumption is that it has CAFs
+    init_info = vanillaIdInfo
 
-    tcPrag info (HsStrictness strict)
-       = tcStrictness unf_env ty info strict
+    tcPrag info HsNoCafRefs         = returnM (info `setCafInfo`   NoCafRefs)
+    tcPrag info (HsArity arity)     = returnM (info `setArityInfo` arity)
+    tcPrag info (HsStrictness str)  = returnM (info `setAllStrictnessInfo` Just str)
+    tcPrag info (HsWorker nm arity) = tcWorkerInfo ty info nm arity
 
-    tcPrag info (HsSpecialise tyvars tys rhs)
-       = tcExtendTyVarScope tyvars             $ \ tyvars' ->
-         mapAndUnzipTc tcHsTypeKind tys        `thenTc` \ (kinds, tys') -> 
-               -- Assume that the kinds match the kinds of the 
-               -- type variables of the function; this is, after all, an
-               -- interface file generated by the compiler!
-
-         tcPragExpr unf_env name rhs   `thenNF_Tc` \ maybe_rhs' ->
+    tcPrag info (HsUnfold inline_prag expr)
+       = tcPragExpr name in_scope_vars expr    `thenM` \ maybe_expr' ->
          let
-               -- If spec_env isn't looked at, none of this 
-               -- actually takes place
-           spec_env  = specInfo info
-           spec_env' = case maybe_rhs' of
-                         Nothing -> spec_env
-                         Just rhs' -> case addToSpecEnv True {- overlap ok -} spec_env tyvars' tys' rhs' of
-                                         Succeeded spec_env' -> spec_env'
-                                         Failed err          -> pprTrace "tcIdInfo: bad specialisation"
-                                                                         (ppr name <+> ppr err) $
-                                                                spec_env
+               -- maybe_expr' doesn't get looked at if the unfolding
+               -- is never inspected; so the typecheck doesn't even happen
+               unfold_info = case maybe_expr' of
+                               Nothing    -> noUnfolding
+                               Just expr' -> mkTopUnfolding expr' 
          in
-         returnTc (spec_env' `setSpecInfo` info)
+         returnM (info `setUnfoldingInfoLazily` unfold_info
+                       `setInlinePragInfo`      inline_prag)
 \end{code}
 
 \begin{code}
-tcStrictness unf_env ty info (HsStrictnessInfo (demands, bot_result) maybe_worker)
-  = tcWorker unf_env maybe_worker              `thenNF_Tc` \ maybe_worker_id ->
-    uniqSMToTcM (mkWrapper ty demands)         `thenNF_Tc` \ wrap_fn ->
-    let
-       -- Watch out! We can't pull on maybe_worker_id too eagerly!
-       info' = case maybe_worker_id of
-                       Just worker_id -> setUnfoldingInfo (mkUnfolding (wrap_fn worker_id)) $
-                                         setInlinePragInfo IWantToBeINLINEd info
-
-                       Nothing        -> info
-
-       has_worker = maybeToBool maybe_worker_id
-    in
-    returnTc (StrictnessInfo demands bot_result has_worker  `setStrictnessInfo` info')
-\end{code}
-
-\begin{code}
-tcWorker unf_env Nothing = returnNF_Tc Nothing
+tcWorkerInfo ty info wkr_name arity
+  = forkM doc (tcVar wkr_name) `thenM` \ maybe_wkr_id ->
+       -- Watch out! We can't pull on unf_env too eagerly!
+       -- Hence the forkM
+
+       -- We return without testing maybe_wkr_id, but as soon as info is
+       -- looked at we will test it.  That's ok, because its outside the
+       -- knot; and there seems no big reason to further defer the
+       -- tcVar lookup.  (Contrast with tcPragExpr, where postponing walking
+       -- over the unfolding until it's actually used does seem worth while.)
+    newUniqueSupply            `thenM` \ us ->
+    returnM (case maybe_wkr_id of
+       Nothing     -> info
+       Just wkr_id -> info `setUnfoldingInfoLazily`  mk_unfolding us wkr_id
+                           `setWorkerInfo`           HasWorker wkr_id arity)
 
-tcWorker unf_env (Just (worker_name,_))
-  = returnNF_Tc (trace_maybe maybe_worker_id)
   where
-    maybe_worker_id = explicitLookupValue unf_env worker_name
+    doc = text "worker for" <+> ppr wkr_name
+
+    mk_unfolding us wkr_id = mkTopUnfolding (initUs_ us (mkWrapper ty strict_sig) wkr_id)
 
-       -- The trace is so we can see what's getting dropped
-    trace_maybe Nothing  = pprTrace "tcWorker failed:" (ppr worker_name) Nothing
-    trace_maybe (Just x) = Just x
+       -- We are relying here on strictness info always appearing 
+       -- before worker info,  fingers crossed ....
+    strict_sig = case newStrictnessInfo info of
+                  Just sig -> sig
+                  Nothing  -> pprPanic "Worker info but no strictness for" (ppr wkr_name)
 \end{code}
 
 For unfoldings we try to do the job lazily, so that we never type check
 an unfolding that isn't going to be looked at.
 
 \begin{code}
-tcPragExpr :: ValueEnv -> Name -> UfExpr Name -> NF_TcM s (Maybe CoreExpr)
-tcPragExpr unf_env name core_expr
-  = forkNF_Tc (
-       recoverNF_Tc no_unfolding (
-               tcSetValueEnv unf_env $
-               tcCoreExpr core_expr    `thenTc` \ core_expr' ->
-               returnTc (Just core_expr')
-    ))                 
+tcPragExpr :: Name -> [Id] -> UfExpr Name -> TcM (Maybe CoreExpr)
+tcPragExpr name in_scope_vars expr
+  = forkM doc $
+    tcCoreExpr expr            `thenM` \ core_expr' ->
+
+               -- Check for type consistency in the unfolding
+    ifOptM Opt_DoCoreLinting (
+       getSrcLocM              `thenM` \ src_loc -> 
+       case lintUnfolding src_loc in_scope_vars core_expr' of
+         Nothing       -> returnM ()
+         Just fail_msg -> failWithTc ((doc <+> text "Failed Lint") $$ fail_msg)
+    )                          `thenM_`
+
+   returnM core_expr'  
   where
-       -- The trace tells what wasn't available, for the benefit of
-       -- compiler hackers who want to improve it!
-    no_unfolding = getErrsTc           `thenNF_Tc` \ (warns,errs) ->
-                  returnNF_Tc (pprTrace "tcUnfolding failed with:" 
-                                        (hang (ppr name) 4 (pprBagOfErrors errs))
-                                        Nothing)
+    doc = text "unfolding of" <+> ppr name
 \end{code}
 
 
 Variables in unfoldings
 ~~~~~~~~~~~~~~~~~~~~~~~
-****** Inside here we use only the Global environment, even for locally bound variables.
-****** Why? Because we know all the types and want to bind them to real Ids.
 
 \begin{code}
-tcVar :: Name -> TcM s Id
-tcVar name
-  = tcLookupValueMaybe name    `thenNF_Tc` \ maybe_id ->
-    case maybe_id of {
-       Just id -> returnTc id;
-       Nothing -> failWithTc (noDecl name)
-    }
-
-noDecl name = hsep [ptext SLIT("Warning: no binding for"), ppr name]
+tcVar :: Name -> TcM Id
+  -- Inside here we use only the Global environment, even for locally bound variables.
+  -- Why? Because we know all the types and want to bind them to real Ids.
+tcVar name = tcLookupGlobalId name
 \end{code}
 
 UfCore expressions.
 
 \begin{code}
-tcCoreExpr :: UfExpr Name -> TcM s CoreExpr
+tcCoreExpr :: UfExpr Name -> TcM CoreExpr
 
 tcCoreExpr (UfType ty)
-  = tcHsTypeKind ty    `thenTc` \ (_, ty') ->
+  = tcIfaceType ty             `thenM` \ ty' ->
        -- It might not be of kind type
-    returnTc (Type ty')
+    returnM (Type ty')
 
 tcCoreExpr (UfVar name)
-  = tcVar name         `thenTc` \ id ->
-    returnTc (Var id)
+  = tcVar name         `thenM` \ id ->
+    returnM (Var id)
+
+tcCoreExpr (UfLit lit)
+  = returnM (Lit lit)
+
+-- The dreaded lit-lits are also similar, except here the type
+-- is read in explicitly rather than being implicit
+tcCoreExpr (UfLitLit lit ty)
+  = tcIfaceType ty             `thenM` \ ty' ->
+    returnM (Lit (MachLitLit lit ty'))
 
-tcCoreExpr (UfCon con args) 
-  = tcUfCon con                        `thenTc` \ con' ->
-    mapTc tcCoreExpr args      `thenTc` \ args' ->
-    returnTc (Con con' args')
+tcCoreExpr (UfFCall cc ty)
+  = tcIfaceType ty     `thenM` \ ty' ->
+    newUnique          `thenM` \ u ->
+    returnM (Var (mkFCallId u cc ty'))
 
-tcCoreExpr (UfTuple name args) 
-  = tcUfDataCon name           `thenTc` \ con ->
-    mapTc tcCoreExpr args      `thenTc` \ args' ->
+tcCoreExpr (UfTuple (HsTupCon boxity arity) args) 
+  = mappM tcCoreExpr args      `thenM` \ args' ->
     let
        -- Put the missing type arguments back in
-       con_args = map (Type . coreExprType) args' ++ args'
+       con_args = map (Type . exprType) args' ++ args'
     in
-    returnTc (Con con con_args)
+    returnM (mkApps (Var con_id) con_args)
+  where
+    con_id = dataConWorkId (tupleCon boxity arity)
+    
 
 tcCoreExpr (UfLam bndr body)
   = tcCoreLamBndr bndr                 $ \ bndr' ->
-    tcCoreExpr body            `thenTc` \ body' ->
-    returnTc (Lam bndr' body')
+    tcCoreExpr body            `thenM` \ body' ->
+    returnM (Lam bndr' body')
 
 tcCoreExpr (UfApp fun arg)
-  = tcCoreExpr fun             `thenTc` \ fun' ->
-    tcCoreExpr arg             `thenTc` \ arg' ->
-    returnTc (App fun' arg')
+  = tcCoreExpr fun             `thenM` \ fun' ->
+    tcCoreExpr arg             `thenM` \ arg' ->
+    returnM (App fun' arg')
 
 tcCoreExpr (UfCase scrut case_bndr alts) 
-  = tcCoreExpr scrut                                   `thenTc` \ scrut' ->
+  = tcCoreExpr scrut                                   `thenM` \ scrut' ->
     let
-       scrut_ty = coreExprType scrut'
-       case_bndr' = mkUserId case_bndr scrut_ty
+       scrut_ty = exprType scrut'
+       case_bndr' = mkLocalId case_bndr scrut_ty
     in
     tcExtendGlobalValEnv [case_bndr']  $
-    mapTc (tcCoreAlt scrut_ty) alts    `thenTc` \ alts' ->
-    returnTc (Case scrut' case_bndr' alts')
+    mappM (tcCoreAlt scrut_ty) alts    `thenM` \ alts' ->
+    returnM (Case scrut' case_bndr' alts')
 
 tcCoreExpr (UfLet (UfNonRec bndr rhs) body)
-  = tcCoreExpr rhs             `thenTc` \ rhs' ->
+  = tcCoreExpr rhs             `thenM` \ rhs' ->
     tcCoreValBndr bndr                 $ \ bndr' ->
-    tcCoreExpr body            `thenTc` \ body' ->
-    returnTc (Let (NonRec bndr' rhs') body')
+    tcCoreExpr body            `thenM` \ body' ->
+    returnM (Let (NonRec bndr' rhs') body')
 
 tcCoreExpr (UfLet (UfRec pairs) body)
   = tcCoreValBndrs bndrs       $ \ bndrs' ->
-    mapTc tcCoreExpr rhss      `thenTc` \ rhss' ->
-    tcCoreExpr body            `thenTc` \ body' ->
-    returnTc (Let (Rec (bndrs' `zip` rhss')) body')
+    mappM tcCoreExpr rhss      `thenM` \ rhss' ->
+    tcCoreExpr body            `thenM` \ body' ->
+    returnM (Let (Rec (bndrs' `zip` rhss')) body')
   where
     (bndrs, rhss) = unzip pairs
 
 tcCoreExpr (UfNote note expr) 
-  = tcCoreExpr expr            `thenTc` \ expr' ->
+  = tcCoreExpr expr            `thenM` \ expr' ->
     case note of
-       UfCoerce to_ty -> tcHsType to_ty        `thenTc` \ to_ty' ->
-                         returnTc (Note (Coerce to_ty' (coreExprType expr')) expr')
-       UfInlineCall   -> returnTc (Note InlineCall expr')
-       UfSCC cc       -> returnTc (Note (SCC cc) expr')
-
-tcCoreNote (UfSCC cc)   = returnTc (SCC cc)
-tcCoreNote UfInlineCall = returnTc InlineCall 
-
-
--- rationalTy isn't built in so, we have to construct it
--- (the "ty" part of the incoming literal is simply bottom)
-tcUfCon (UfLitCon (NoRepRational lit _)) 
-  = tcLookupTyConByKey rationalTyConKey        `thenNF_Tc` \ rational_tycon ->
-    let
-       rational_ty  = mkSynTy rational_tycon []
-    in
-    returnTc (Literal (NoRepRational lit rational_ty)) 
-
--- Similarly for integers and strings, except that they are wired in
-tcUfCon (UfLitCon (NoRepInteger lit _)) 
-  = returnTc (Literal (NoRepInteger lit integerTy))
-tcUfCon (UfLitCon (NoRepStr lit _))
-  = returnTc (Literal (NoRepStr lit stringTy))
-
-tcUfCon (UfLitCon other_lit)
-  = returnTc (Literal other_lit)
-
--- The dreaded lit-lits are also similar, except here the type
--- is read in explicitly rather than being implicit
-tcUfCon (UfLitLitCon lit ty)
-  = tcHsType ty                `thenTc` \ ty' ->
-    returnTc (Literal (MachLitLit lit ty'))
-
-tcUfCon (UfDataCon name) = tcUfDataCon name
-
-tcUfCon (UfPrimOp name)
-  = tcVar name         `thenTc` \ op_id ->
-    case isPrimitiveId_maybe op_id of
-       Just op -> returnTc (PrimOp op)
-       Nothing -> failWithTc (badPrimOp name)
-
-tcUfCon (UfCCallOp str is_dyn casm gc)
-  = case is_dyn of
-       True  -> 
-          tcGetUnique `thenNF_Tc` \ u ->
-         returnTc (PrimOp (CCallOp (Right u) casm gc cCallConv))
-       False -> returnTc (PrimOp (CCallOp (Left str) casm gc cCallConv))
-
-tcUfDataCon name
-  = tcVar name         `thenTc` \ con_id ->
-    case isDataConId_maybe con_id of
-       Just con -> returnTc (DataCon con)
-       Nothing  -> failWithTc (badCon name)
+       UfCoerce to_ty -> tcIfaceType to_ty     `thenM` \ to_ty' ->
+                         returnM (Note (Coerce to_ty'
+                                                 (exprType expr')) expr')
+       UfInlineCall   -> returnM (Note InlineCall expr')
+       UfInlineMe     -> returnM (Note InlineMe   expr')
+       UfSCC cc       -> returnM (Note (SCC cc)   expr')
 \end{code}
 
 \begin{code}
 tcCoreLamBndr (UfValBinder name ty) thing_inside
-  = tcHsType ty                        `thenTc` \ ty' ->
+  = tcIfaceType ty             `thenM` \ ty' ->
     let
-       id = mkUserId name ty'
+       id = mkLocalId name ty'
     in
     tcExtendGlobalValEnv [id] $
     thing_inside id
@@ -335,18 +312,24 @@ tcCoreLamBndr (UfTyBinder name kind) thing_inside
     in
     tcExtendTyVarEnv [tyvar] (thing_inside tyvar)
     
+tcCoreLamBndrs []     thing_inside = thing_inside []
+tcCoreLamBndrs (b:bs) thing_inside
+  = tcCoreLamBndr b    $ \ b' ->
+    tcCoreLamBndrs bs  $ \ bs' ->
+    thing_inside (b':bs')
+
 tcCoreValBndr (UfValBinder name ty) thing_inside
-  = tcHsType ty                        `thenTc` \ ty' ->
+  = tcIfaceType ty                     `thenM` \ ty' ->
     let
-       id = mkUserId name ty'
+       id = mkLocalId name ty'
     in
     tcExtendGlobalValEnv [id] $
     thing_inside id
     
 tcCoreValBndrs bndrs thing_inside              -- Expect them all to be ValBinders
-  = mapTc tcHsType tys                 `thenTc` \ tys' ->
+  = mappM tcIfaceType tys              `thenM` \ tys' ->
     let
-       ids = zipWithEqual "tcCoreValBndr" mkUserId names tys'
+       ids = zipWithEqual "tcCoreValBndr" mkLocalId names tys'
     in
     tcExtendGlobalValEnv ids $
     thing_inside ids
@@ -358,54 +341,95 @@ tcCoreValBndrs bndrs thing_inside         -- Expect them all to be ValBinders
 \begin{code}
 tcCoreAlt scrut_ty (UfDefault, names, rhs)
   = ASSERT( null names )
-    tcCoreExpr rhs             `thenTc` \ rhs' ->
-    returnTc (DEFAULT, [], rhs')
+    tcCoreExpr rhs             `thenM` \ rhs' ->
+    returnM (DEFAULT, [], rhs')
   
-tcCoreAlt scrut_ty (UfLitCon lit, names, rhs)
+tcCoreAlt scrut_ty (UfLitAlt lit, names, rhs)
   = ASSERT( null names )
-    tcCoreExpr rhs             `thenTc` \ rhs' ->
-    returnTc (Literal lit, [], rhs')
+    tcCoreExpr rhs             `thenM` \ rhs' ->
+    returnM (LitAlt lit, [], rhs')
 
-tcCoreAlt scrut_ty (UfLitLitCon str ty, names, rhs)
+tcCoreAlt scrut_ty (UfLitLitAlt str ty, names, rhs)
   = ASSERT( null names )
-    tcCoreExpr rhs             `thenTc` \ rhs' ->
-    tcHsType ty                        `thenTc` \ ty' ->
-    returnTc (Literal (MachLitLit str ty'), [], rhs')
+    tcCoreExpr rhs             `thenM` \ rhs' ->
+    tcIfaceType ty             `thenM` \ ty' ->
+    returnM (LitAlt (MachLitLit str ty'), [], rhs')
 
 -- A case alternative is made quite a bit more complicated
 -- by the fact that we omit type annotations because we can
 -- work them out.  True enough, but its not that easy!
-tcCoreAlt scrut_ty (UfDataCon con_name, names, rhs)
-  = tcVar con_name             `thenTc` \ con_id ->
+tcCoreAlt scrut_ty alt@(con, names, rhs)
+  = tcConAlt con       `thenM` \ con ->
     let
-       con                     = case isDataConId_maybe con_id of
-                                       Just con -> con
-                                       Nothing  -> pprPanic "tcCoreAlt" (ppr con_id)
-
-       (main_tyvars, _, ex_tyvars, _, _, _) = dataConSig con
-
-       (tycon, inst_tys, cons) = splitAlgTyConApp scrut_ty
-       ex_tyvars'              = [mkTyVar name (tyVarKind tv) | (name,tv) <- names `zip` ex_tyvars] 
-       ex_tys'                 = mkTyVarTys ex_tyvars'
-       arg_tys                 = dataConArgTys con (inst_tys ++ ex_tys')
-       id_names                = drop (length ex_tyvars) names
+       ex_tyvars         = dataConExistentialTyVars con
+       (tycon, inst_tys) = splitTyConApp scrut_ty      -- NB: not tcSplitTyConApp
+                                                       -- We are looking at Core here
+       main_tyvars       = tyConTyVars tycon
+       ex_tyvars'        = [mkTyVar name (tyVarKind tv) | (name,tv) <- names `zip` ex_tyvars] 
+       ex_tys'           = mkTyVarTys ex_tyvars'
+       arg_tys           = dataConArgTys con (inst_tys ++ ex_tys')
+       id_names          = dropList ex_tyvars names
        arg_ids
 #ifdef DEBUG
-               | length id_names /= length arg_tys
-               = pprPanic "tcCoreAlts" (ppr (con_name, names, rhs) $$
+               | not (equalLength id_names arg_tys)
+               = pprPanic "tcCoreAlts" (ppr (con, names, rhs) $$
                                         (ppr main_tyvars <+> ppr ex_tyvars) $$
                                         ppr arg_tys)
                | otherwise
 #endif
-               = zipWithEqual "tcCoreAlts" mkUserId id_names arg_tys
+               = zipWithEqual "tcCoreAlts" mkLocalId id_names arg_tys
     in
-    ASSERT( con `elem` cons && length inst_tys == length main_tyvars )
+    ASSERT( con `elem` tyConDataCons tycon && equalLength inst_tys main_tyvars )
     tcExtendTyVarEnv ex_tyvars'                        $
     tcExtendGlobalValEnv arg_ids               $
-    tcCoreExpr rhs                                     `thenTc` \ rhs' ->
-    returnTc (DataCon con, ex_tyvars' ++ arg_ids, rhs')
+    tcCoreExpr rhs                                     `thenM` \ rhs' ->
+    returnM (DataAlt con, ex_tyvars' ++ arg_ids, rhs')
+
+
+tcConAlt :: UfConAlt Name -> TcM DataCon
+tcConAlt (UfTupleAlt (HsTupCon boxity arity))
+  = returnM (tupleCon boxity arity)
+
+tcConAlt (UfDataAlt con_name)  -- When reading interface files
+                               -- the con_name will be the real name of
+                               -- the data con
+  = tcLookupDataCon con_name
 \end{code}
 
+%************************************************************************
+%*                                                                     *
+\subsection{Core decls}
+%*                                                                     *
+%************************************************************************
+
+
+\begin{code}
+tcCoreBinds :: [RenamedCoreDecl] -> TcM [TypecheckedCoreBind]
+-- We don't assume the bindings are in dependency order
+-- So first build the environment, then check the RHSs
+tcCoreBinds ls = mappM tcCoreBinder ls         `thenM` \ bndrs ->
+                tcExtendGlobalValEnv bndrs     $
+                mappM (tcCoreBind bndrs) ls
+
+tcCoreBinder (CoreDecl nm ty _ _)
+ = kcHsSigType ty      `thenM_`
+   tcIfaceType ty      `thenM` \ ty' ->
+   returnM (mkLocalId nm ty')
+
+tcCoreBind bndrs (CoreDecl nm _ rhs loc)
+ = tcVar nm            `thenM` \ id ->
+   tcCoreExpr rhs      `thenM` \ rhs' ->
+   let
+       mb_err = lintUnfolding loc bndrs rhs'
+   in
+   (case mb_err of
+       Just err -> addErr err
+       Nothing  -> returnM ()) `thenM_`
+
+   returnM (id, rhs')
+\end{code}
+
+
 \begin{code}
 ifaceSigCtxt sig_name
   = hsep [ptext SLIT("In an interface-file signature for"), ppr sig_name]