[project @ 2003-12-30 20:24:21 by simonpj]
[ghc-hetmet.git] / ghc / compiler / typecheck / TcExpr.lhs
index f889697..cf94f27 100644 (file)
@@ -10,16 +10,16 @@ module TcExpr ( tcCheckSigma, tcCheckRho, tcInferRho, tcMonoExpr ) where
 
 #ifdef GHCI    /* Only if bootstrapped */
 import {-# SOURCE #-}  TcSplice( tcSpliceExpr, tcBracket )
-import HsSyn           ( HsReify(..), ReifyFlavour(..) )
+import Id              ( Id )
 import TcType          ( isTauTy )
-import TcEnv           ( bracketOK, tcMetaTy, checkWellStaged )
-import Name            ( isExternalName )
+import TcEnv           ( checkWellStaged )
+import HsSyn           ( nlHsApp )
 import qualified DsMeta
 #endif
 
-import HsSyn           ( HsExpr(..), HsLit(..), ArithSeqInfo(..), recBindFields )
-import RnHsSyn         ( RenamedHsExpr, RenamedRecordBinds )
-import TcHsSyn         ( TcExpr, TcRecordBinds, hsLitType, mkHsDictApp, mkHsTyApp, mkHsLet, (<$>) )
+import HsSyn           ( HsExpr(..), LHsExpr, HsLit(..), ArithSeqInfo(..), recBindFields,
+                         HsMatchContext(..), HsRecordBinds, mkHsApp, nlHsVar )
+import TcHsSyn         ( hsLitType, mkHsDictApp, mkHsTyApp, (<$>) )
 import TcRnMonad
 import TcUnify         ( Expected(..), newHole, zapExpectedType, zapExpectedTo, tcSubExp, tcGen,
                          unifyFunTy, zapToListTy, zapToPArrTy, zapToTupleTy )
@@ -30,43 +30,45 @@ import Inst         ( InstOrigin(..),
                          instToId, tcInstCall, tcInstDataCon
                        )
 import TcBinds         ( tcBindsAndThen )
-import TcEnv           ( tcLookupClass, tcLookupGlobal_maybe, tcLookup,
-                         tcLookupTyCon, tcLookupDataCon, tcLookupId, checkProcLevel
+import TcEnv           ( tcLookup, tcLookupId, checkProcLevel,
+                         tcLookupDataCon, tcLookupGlobalId
                        )
 import TcArrows                ( tcProc )
-import TcMatches       ( tcMatchesCase, tcMatchLambda, tcDoStmts, tcThingWithSig )
-import TcMonoType      ( tcHsSigType, UserTypeCtxt(..) )
+import TcMatches       ( tcMatchesCase, tcMatchLambda, tcDoStmts, tcThingWithSig, TcMatchCtxt(..) )
+import TcHsType                ( tcHsSigType, UserTypeCtxt(..) )
 import TcPat           ( badFieldCon )
-import TcMType         ( tcInstTyVars, tcInstType, newTyVarTy, newTyVarTys, zonkTcType )
+import TcMType         ( tcInstTyVars, tcInstType, newTyVarTy, zonkTcType )
 import TcType          ( TcType, TcSigmaType, TcRhoType, TyVarDetails(VanillaTv),
                          tcSplitFunTys, tcSplitTyConApp, mkTyVarTys,
                          isSigmaTy, mkFunTy, mkFunTys,
-                         mkTyConApp, mkClassPred, 
-                         tyVarsOfTypes, isLinearPred,
-                         liftedTypeKind, openTypeKind, 
+                         mkTyConApp, tyVarsOfTypes, isLinearPred,
                          tcSplitSigmaTy, tidyOpenType
                        )
+import Kind            ( openTypeKind, liftedTypeKind, argTypeKind )
+
 import FieldLabel      ( FieldLabel, fieldLabelName, fieldLabelType, fieldLabelTyCon )
-import Id              ( Id, idType, recordSelectorFieldLabel, isRecordSelector )
-import DataCon         ( DataCon, dataConFieldLabels, dataConSig, dataConStrictMarks, dataConWrapId )
+import Id              ( idType, recordSelectorFieldLabel, isRecordSelector )
+import DataCon         ( DataCon, dataConFieldLabels, dataConStrictMarks, dataConWrapId )
 import Name            ( Name )
-import TyCon           ( TyCon, tyConTyVars, tyConTheta, isAlgTyCon, tyConDataCons )
+import TyCon           ( TyCon, tyConTyVars, tyConTheta, tyConDataCons )
 import Subst           ( mkTopTyVarSubst, substTheta, substTy )
 import VarSet          ( emptyVarSet, elemVarSet )
 import TysWiredIn      ( boolTy )
-import PrelNames       ( cCallableClassName, cReturnableClassName, 
-                         enumFromName, enumFromThenName, 
+import PrelNames       ( enumFromName, enumFromThenName, 
                          enumFromToName, enumFromThenToName,
-                         enumFromToPName, enumFromThenToPName,
-                         ioTyConName
+                         enumFromToPName, enumFromThenToPName
                        )
 import ListSetOps      ( minusList )
 import CmdLineOpts
 import HscTypes                ( TyThing(..) )
-
+import SrcLoc          ( Located(..), unLoc, getLoc )
 import Util
 import Outputable
 import FastString
+
+#ifdef DEBUG
+import TyCon           ( isAlgTyCon )
+#endif
 \end{code}
 
 %************************************************************************
@@ -77,9 +79,9 @@ import FastString
 
 \begin{code}
 -- tcCheckSigma does type *checking*; it's passed the expected type of the result
-tcCheckSigma :: RenamedHsExpr          -- Expession to type check
+tcCheckSigma :: LHsExpr Name           -- Expession to type check
                     -> TcSigmaType             -- Expected type (could be a polytpye)
-                    -> TcM TcExpr              -- Generalised expr with expected type
+                    -> TcM (LHsExpr TcId)      -- Generalised expr with expected type
 
 tcCheckSigma expr expected_ty 
   = traceTc (text "tcExpr" <+> (ppr expected_ty $$ ppr expr)) `thenM_`
@@ -90,7 +92,7 @@ tc_expr' expr sigma_ty
   = tcGen sigma_ty emptyVarSet (
        \ rho_ty -> tcCheckRho expr rho_ty
     )                          `thenM` \ (gen_fn, expr') ->
-    returnM (gen_fn <$> expr')
+    returnM (L (getLoc expr') (gen_fn <$> unLoc expr'))
 
 tc_expr' expr rho_ty   -- Monomorphic case
   = tcCheckRho expr rho_ty
@@ -102,44 +104,51 @@ The expression can return a higher-ranked type, such as
 so we must create a hole to pass in as the expected tyvar.
 
 \begin{code}
-tcCheckRho :: RenamedHsExpr -> TcRhoType -> TcM TcExpr
+tcCheckRho :: LHsExpr Name -> TcRhoType -> TcM (LHsExpr TcId)
 tcCheckRho expr rho_ty = tcMonoExpr expr (Check rho_ty)
 
-tcInferRho :: RenamedHsExpr -> TcM (TcExpr, TcRhoType)
-tcInferRho (HsVar name) = tcId name
-tcInferRho expr         = newHole                      `thenM` \ hole ->
-                         tcMonoExpr expr (Infer hole)  `thenM` \ expr' ->
-                         readMutVar hole               `thenM` \ rho_ty ->
-                         returnM (expr', rho_ty) 
+tcInferRho :: LHsExpr Name -> TcM (LHsExpr TcId, TcRhoType)
+tcInferRho (L loc (HsVar name)) = addSrcSpan loc $ 
+                                 do { (e,ty) <- tcId name; return (L loc e, ty)}
+tcInferRho expr                        = newHole                       `thenM` \ hole ->
+                                 tcMonoExpr expr (Infer hole)  `thenM` \ expr' ->
+                                 readMutVar hole               `thenM` \ rho_ty ->
+                                 returnM (expr', rho_ty) 
 \end{code}
 
 
 
 %************************************************************************
 %*                                                                     *
-\subsection{The TAUT rules for variables}
+\subsection{The TAUT rules for variables}TcExpr
 %*                                                                     *
 %************************************************************************
 
 \begin{code}
-tcMonoExpr :: RenamedHsExpr            -- Expession to type check
+tcMonoExpr :: LHsExpr Name             -- Expession to type check
           -> Expected TcRhoType        -- Expected type (could be a type variable)
                                        -- Definitely no foralls at the top
                                        -- Can be a 'hole'.
-          -> TcM TcExpr
+          -> TcM (LHsExpr TcId)
+
+tcMonoExpr (L loc expr) res_ty
+  = addSrcSpan loc (do { expr' <- tc_expr expr res_ty
+                      ; return (L loc expr') })
 
-tcMonoExpr (HsVar name) res_ty
+tc_expr :: HsExpr Name -> Expected TcRhoType -> TcM (HsExpr TcId)
+tc_expr (HsVar name) res_ty
   = tcId name                  `thenM` \ (expr', id_ty) ->
     tcSubExp res_ty id_ty      `thenM` \ co_fn ->
     returnM (co_fn <$> expr')
 
-tcMonoExpr (HsIPVar ip) res_ty
+tc_expr (HsIPVar ip) res_ty
   =    -- Implicit parameters must have a *tau-type* not a 
        -- type scheme.  We enforce this by creating a fresh
        -- type variable as its type.  (Because res_ty may not
        -- be a tau-type.)
-    newTyVarTy openTypeKind            `thenM` \ ip_ty ->
-    newIPDict (IPOcc ip) ip ip_ty      `thenM` \ (ip', inst) ->
+    newTyVarTy argTypeKind             `thenM` \ ip_ty ->
+       -- argTypeKind: it can't be an unboxed tuple
+    newIPDict (IPOccOrigin ip) ip ip_ty `thenM` \ (ip', inst) ->
     extendLIE inst                     `thenM_`
     tcSubExp res_ty ip_ty              `thenM` \ co_fn ->
     returnM (co_fn <$> HsIPVar ip')
@@ -153,13 +162,14 @@ tcMonoExpr (HsIPVar ip) res_ty
 %************************************************************************
 
 \begin{code}
-tcMonoExpr in_expr@(ExprWithTySig expr poly_ty) res_ty
- = addErrCtxt (exprSigCtxt in_expr)                    $
+tc_expr in_expr@(ExprWithTySig expr poly_ty) res_ty
+ = addErrCtxt (exprCtxt in_expr)                       $
    tcHsSigType ExprSigCtxt poly_ty                     `thenM` \ sig_tc_ty ->
    tcThingWithSig sig_tc_ty (tcCheckRho expr) res_ty   `thenM` \ (co_fn, expr') ->
-   returnM (co_fn <$> expr')
+   returnM (co_fn <$> unLoc expr')
+       -- ToDo: nasty unLoc
 
-tcMonoExpr (HsType ty) res_ty
+tc_expr (HsType ty) res_ty
   = failWithTc (text "Can't handle type argument:" <+> ppr ty)
        -- This is the syntax for type applications that I was planning
        -- but there are difficulties (e.g. what order for type args)
@@ -176,25 +186,29 @@ tcMonoExpr (HsType ty) res_ty
 %************************************************************************
 
 \begin{code}
-tcMonoExpr (HsLit lit)     res_ty  = tcLit lit res_ty
-tcMonoExpr (HsOverLit lit) res_ty  = zapExpectedType res_ty    `thenM` \ res_ty' ->
-                                    newOverloadedLit (LiteralOrigin lit) lit res_ty'
-tcMonoExpr (HsPar expr)    res_ty  = tcMonoExpr expr res_ty    `thenM` \ expr' -> 
-                                    returnM (HsPar expr')
-tcMonoExpr (HsSCC lbl expr) res_ty = tcMonoExpr expr res_ty    `thenM` \ expr' ->
-                                    returnM (HsSCC lbl expr')
-
-tcMonoExpr (HsCoreAnn lbl expr) res_ty = tcMonoExpr expr res_ty `thenM` \ expr' ->  -- hdaume: core annotation
+tc_expr (HsPar expr)    res_ty  = tcMonoExpr expr res_ty       `thenM` \ expr' -> 
+                                 returnM (HsPar expr')
+tc_expr (HsSCC lbl expr) res_ty = tcMonoExpr expr res_ty       `thenM` \ expr' ->
+                                 returnM (HsSCC lbl expr')
+tc_expr (HsCoreAnn lbl expr) res_ty = tcMonoExpr expr res_ty `thenM` \ expr' ->  -- hdaume: core annotation
                                          returnM (HsCoreAnn lbl expr')
-tcMonoExpr (NegApp expr neg_name) res_ty
-  = tcMonoExpr (HsApp (HsVar neg_name) expr) res_ty
+
+tc_expr (HsLit lit) res_ty  = tcLit lit res_ty
+
+tc_expr (HsOverLit lit) res_ty  
+  = zapExpectedType res_ty liftedTypeKind              `thenM` \ res_ty' ->
+    newOverloadedLit (LiteralOrigin lit) lit res_ty'   `thenM` \ lit_expr ->
+    returnM (unLoc lit_expr)   -- ToDo: nasty unLoc
+
+tc_expr (NegApp expr neg_name) res_ty
+  = tc_expr (HsApp (nlHsVar neg_name) expr) res_ty
        -- ToDo: use tcSyntaxName
 
-tcMonoExpr (HsLam match) res_ty
+tc_expr (HsLam match) res_ty
   = tcMatchLambda match res_ty                 `thenM` \ match' ->
     returnM (HsLam match')
 
-tcMonoExpr (HsApp e1 e2) res_ty 
+tc_expr (HsApp e1 e2) res_ty 
   = tcApp e1 [e2] res_ty
 \end{code}
 
@@ -209,7 +223,7 @@ a type error will occur if they aren't.
 -- or just
 --     op e
 
-tcMonoExpr in_expr@(SectionL arg1 op) res_ty
+tc_expr in_expr@(SectionL arg1 op) res_ty
   = tcInferRho op                              `thenM` \ (op', op_ty) ->
     split_fun_ty op_ty 2 {- two args -}                `thenM` \ ([arg1_ty, arg2_ty], op_res_ty) ->
     tcArg op (arg1, arg1_ty, 1)                        `thenM` \ arg1' ->
@@ -220,7 +234,7 @@ tcMonoExpr in_expr@(SectionL arg1 op) res_ty
 -- Right sections, equivalent to \ x -> x op expr, or
 --     \ x -> op x expr
 
-tcMonoExpr in_expr@(SectionR op arg2) res_ty
+tc_expr in_expr@(SectionR op arg2) res_ty
   = tcInferRho op                              `thenM` \ (op', op_ty) ->
     split_fun_ty op_ty 2 {- two args -}                `thenM` \ ([arg1_ty, arg2_ty], op_res_ty) ->
     tcArg op (arg2, arg2_ty, 2)                        `thenM` \ arg2' ->
@@ -230,7 +244,7 @@ tcMonoExpr in_expr@(SectionR op arg2) res_ty
 
 -- equivalent to (op e1) e2:
 
-tcMonoExpr in_expr@(OpApp arg1 op fix arg2) res_ty
+tc_expr in_expr@(OpApp arg1 op fix arg2) res_ty
   = tcInferRho op                              `thenM` \ (op', op_ty) ->
     split_fun_ty op_ty 2 {- two args -}                `thenM` \ ([arg1_ty, arg2_ty], op_res_ty) ->
     tcArg op (arg1, arg1_ty, 1)                        `thenM` \ arg1' ->
@@ -241,15 +255,16 @@ tcMonoExpr in_expr@(OpApp arg1 op fix arg2) res_ty
 \end{code}
 
 \begin{code}
-tcMonoExpr (HsLet binds expr) res_ty
+tc_expr (HsLet binds (L loc expr)) res_ty
   = tcBindsAndThen
-       HsLet
+       glue
        binds                   -- Bindings to check
-       (tcMonoExpr expr res_ty)
+       (tc_expr expr res_ty)
+  where
+    glue bind expr = HsLet [bind] (L loc expr)
 
-tcMonoExpr in_expr@(HsCase scrut matches src_loc) res_ty
-  = addSrcLoc src_loc                  $
-    addErrCtxt (caseCtxt in_expr)      $
+tc_expr in_expr@(HsCase scrut matches) res_ty
+  = addErrCtxt (caseCtxt in_expr)      $
 
        -- Typecheck the case alternatives first.
        -- The case patterns tend to give good type info to use
@@ -258,34 +273,35 @@ tcMonoExpr in_expr@(HsCase scrut matches src_loc) res_ty
        --        (x:xs) -> ...
        -- will report that map is applied to too few arguments
 
-    tcMatchesCase matches res_ty       `thenM`    \ (scrut_ty, matches') ->
+    tcMatchesCase match_ctxt matches res_ty    `thenM`    \ (scrut_ty, matches') ->
 
     addErrCtxt (caseScrutCtxt scrut)   (
       tcCheckRho scrut scrut_ty
     )                                  `thenM`    \ scrut' ->
 
-    returnM (HsCase scrut' matches' src_loc)
+    returnM (HsCase scrut' matches')
+  where
+    match_ctxt = MC { mc_what = CaseAlt,
+                     mc_body = tcMonoExpr }
 
-tcMonoExpr (HsIf pred b1 b2 src_loc) res_ty
-  = addSrcLoc src_loc  $
-    addErrCtxt (predCtxt pred) (
+tc_expr (HsIf pred b1 b2) res_ty
+  = addErrCtxt (predCtxt pred) (
     tcCheckRho pred boolTy     )       `thenM`    \ pred' ->
 
-    zapExpectedType res_ty             `thenM`    \ res_ty' ->
+    zapExpectedType res_ty openTypeKind        `thenM`    \ res_ty' ->
        -- C.f. the call to zapToType in TcMatches.tcMatches
 
     tcCheckRho b1 res_ty'              `thenM`    \ b1' ->
     tcCheckRho b2 res_ty'              `thenM`    \ b2' ->
-    returnM (HsIf pred' b1' b2' src_loc)
+    returnM (HsIf pred' b1' b2')
 
-tcMonoExpr (HsDo do_or_lc stmts method_names _ src_loc) res_ty
-  = addSrcLoc src_loc                                  $
-    zapExpectedType res_ty                             `thenM` \ res_ty' ->
-       -- All comprehensions yield a monotype
+tc_expr (HsDo do_or_lc stmts method_names _) res_ty
+  = zapExpectedType res_ty liftedTypeKind              `thenM` \ res_ty' ->
+       -- All comprehensions yield a monotype of kind *
     tcDoStmts do_or_lc stmts method_names res_ty'      `thenM` \ (stmts', methods') ->
-    returnM (HsDo do_or_lc stmts' methods' res_ty' src_loc)
+    returnM (HsDo do_or_lc stmts' methods' res_ty')
 
-tcMonoExpr in_expr@(ExplicitList _ exprs) res_ty       -- Non-empty list
+tc_expr in_expr@(ExplicitList _ exprs) res_ty  -- Non-empty list
   = zapToListTy res_ty                `thenM` \ elt_ty ->  
     mappM (tc_elt elt_ty) exprs              `thenM` \ exprs' ->
     returnM (ExplicitList elt_ty exprs')
@@ -294,7 +310,7 @@ tcMonoExpr in_expr@(ExplicitList _ exprs) res_ty    -- Non-empty list
       = addErrCtxt (listCtxt expr) $
        tcCheckRho expr elt_ty
 
-tcMonoExpr in_expr@(ExplicitPArr _ exprs) res_ty       -- maybe empty
+tc_expr in_expr@(ExplicitPArr _ exprs) res_ty  -- maybe empty
   = zapToPArrTy res_ty                `thenM` \ elt_ty ->  
     mappM (tc_elt elt_ty) exprs              `thenM` \ exprs' ->
     returnM (ExplicitPArr elt_ty exprs')
@@ -303,81 +319,16 @@ tcMonoExpr in_expr@(ExplicitPArr _ exprs) res_ty  -- maybe empty
       = addErrCtxt (parrCtxt expr) $
        tcCheckRho expr elt_ty
 
-tcMonoExpr (ExplicitTuple exprs boxity) res_ty
+tc_expr (ExplicitTuple exprs boxity) res_ty
   = zapToTupleTy boxity (length exprs) res_ty  `thenM` \ arg_tys ->
     tcCheckRhos exprs arg_tys                  `thenM` \ exprs' ->
     returnM (ExplicitTuple exprs' boxity)
 
-tcMonoExpr (HsProc pat cmd loc) res_ty
-  = addSrcLoc loc $
-    tcProc pat cmd res_ty                      `thenM` \ (pat', cmd') ->
-    returnM (HsProc pat' cmd' loc)
+tc_expr (HsProc pat cmd) res_ty
+  = tcProc pat cmd res_ty                      `thenM` \ (pat', cmd') ->
+    returnM (HsProc pat' cmd')
 \end{code}
 
-
-%************************************************************************
-%*                                                                     *
-               Foreign calls
-%*                                                                     *
-%************************************************************************
-
-The interesting thing about @ccall@ is that it is just a template
-which we instantiate by filling in details about the types of its
-argument and result (ie minimal typechecking is performed).  So, the
-basic story is that we allocate a load of type variables (to hold the
-arg/result types); unify them with the args/result; and store them for
-later use.
-
-\begin{code}
-tcMonoExpr e0@(HsCCall lbl args may_gc is_casm ignored_fake_result_ty) res_ty
-
-  = getDOpts                           `thenM` \ dflags ->
-
-    checkTc (not (is_casm && dopt_HscLang dflags /= HscC)) 
-        (vcat [text "_casm_ is only supported when compiling via C (-fvia-C).",
-               text "Either compile with -fvia-C, or, better, rewrite your code",
-               text "to use the foreign function interface.  _casm_s are deprecated",
-               text "and support for them may one day disappear."])
-                                       `thenM_`
-
-    -- Get the callable and returnable classes.
-    tcLookupClass cCallableClassName   `thenM` \ cCallableClass ->
-    tcLookupClass cReturnableClassName `thenM` \ cReturnableClass ->
-    tcLookupTyCon ioTyConName          `thenM` \ ioTyCon ->
-    let
-       new_arg_dict (arg, arg_ty)
-         = newDicts (CCallOrigin (unpackFS lbl) (Just arg))
-                    [mkClassPred cCallableClass [arg_ty]]      `thenM` \ arg_dicts ->
-           returnM arg_dicts   -- Actually a singleton bag
-
-       result_origin = CCallOrigin (unpackFS lbl) Nothing {- Not an arg -}
-    in
-
-       -- Arguments
-    let tv_idxs | null args  = []
-               | otherwise  = [1..length args]
-    in
-    newTyVarTys (length tv_idxs) openTypeKind          `thenM` \ arg_tys ->
-    tcCheckRhos args arg_tys                           `thenM` \ args' ->
-
-       -- The argument types can be unlifted or lifted; the result
-       -- type must, however, be lifted since it's an argument to the IO
-       -- type constructor.
-    newTyVarTy liftedTypeKind                  `thenM` \ result_ty ->
-    let
-       io_result_ty = mkTyConApp ioTyCon [result_ty]
-    in
-    zapExpectedTo res_ty io_result_ty  `thenM_`
-
-       -- Construct the extra insts, which encode the
-       -- constraints on the argument and result types.
-    mappM new_arg_dict (zipEqual "tcMonoExpr:CCall" args arg_tys)      `thenM` \ ccarg_dicts_s ->
-    newDicts result_origin [mkClassPred cReturnableClass [result_ty]]  `thenM` \ ccres_dict ->
-    extendLIEs (ccres_dict ++ concat ccarg_dicts_s)                    `thenM_`
-    returnM (HsCCall lbl args' may_gc is_casm io_result_ty)
-\end{code}
-
-
 %************************************************************************
 %*                                                                     *
                Record construction and update
@@ -385,9 +336,9 @@ tcMonoExpr e0@(HsCCall lbl args may_gc is_casm ignored_fake_result_ty) res_ty
 %************************************************************************
 
 \begin{code}
-tcMonoExpr expr@(RecordCon con_name rbinds) res_ty
+tc_expr expr@(RecordCon con@(L _ con_name) rbinds) res_ty
   = addErrCtxt (recordConCtxt expr)            $
-    tcId con_name                      `thenM` \ (con_expr, con_tau) ->
+    addLocM tcId con                   `thenM` \ (con_expr, con_tau) ->
     let
        (_, record_ty)   = tcSplitFunTys con_tau
        (tycon, ty_args) = tcSplitTyConApp record_ty
@@ -412,7 +363,8 @@ tcMonoExpr expr@(RecordCon con_name rbinds) res_ty
        -- Check for missing fields
     checkMissingFields data_con rbinds         `thenM_` 
 
-    returnM (RecordConOut data_con con_expr rbinds')
+    getSrcSpanM                                        `thenM` \ loc ->
+    returnM (RecordConOut data_con (L loc con_expr) rbinds')
 
 -- The main complication with RecordUpd is that we need to explicitly
 -- handle the *non-updated* fields.  Consider:
@@ -440,23 +392,23 @@ tcMonoExpr expr@(RecordCon con_name rbinds) res_ty
 --
 -- All this is done in STEP 4 below.
 
-tcMonoExpr expr@(RecordUpd record_expr rbinds) res_ty
+tc_expr expr@(RecordUpd record_expr rbinds) res_ty
   = addErrCtxt (recordUpdCtxt  expr)           $
 
        -- STEP 0
        -- Check that the field names are really field names
     ASSERT( notNull rbinds )
     let 
-       field_names = recBindFields rbinds
+       field_names = map fst rbinds
     in
-    mappM tcLookupGlobal_maybe field_names             `thenM` \ maybe_sel_ids ->
+    mappM (tcLookupGlobalId.unLoc) field_names `thenM` \ sel_ids ->
+       -- The renamer has already checked that they
+       -- are all in scope
     let
-       bad_guys = [ addErrTc (notSelector field_name) 
-                  | (field_name, maybe_sel_id) <- field_names `zip` maybe_sel_ids,
-                    not (is_selector maybe_sel_id)
+       bad_guys = [ addSrcSpan loc $ addErrTc (notSelector field_name) 
+                  | (L loc field_name, sel_id) <- field_names `zip` sel_ids,
+                    not (isRecordSelector sel_id)      -- Excludes class ops
                   ]
-       is_selector (Just (AnId sel_id)) = isRecordSelector sel_id      -- Excludes class ops
-       is_selector other                = False        
     in
     checkM (null bad_guys) (sequenceM bad_guys `thenM_` failM) `thenM_`
     
@@ -464,7 +416,7 @@ tcMonoExpr expr@(RecordUpd record_expr rbinds) res_ty
        -- Figure out the tycon and data cons from the first field name
     let
                -- It's OK to use the non-tc splitters here (for a selector)
-       (Just (AnId sel_id) : _) = maybe_sel_ids
+       sel_id : _   = sel_ids
        field_lbl    = recordSelectorFieldLabel sel_id  -- We've failed already if
        tycon        = fieldLabelTyCon field_lbl        -- it's not a field label
        data_cons    = tyConDataCons tycon
@@ -546,16 +498,16 @@ tcMonoExpr expr@(RecordUpd record_expr rbinds) res_ty
 %************************************************************************
 
 \begin{code}
-tcMonoExpr (ArithSeqIn seq@(From expr)) res_ty
+tc_expr (ArithSeqIn seq@(From expr)) res_ty
   = zapToListTy res_ty                                 `thenM` \ elt_ty ->  
     tcCheckRho expr elt_ty                     `thenM` \ expr' ->
 
     newMethodFromName (ArithSeqOrigin seq) 
                      elt_ty enumFromName       `thenM` \ enum_from ->
 
-    returnM (ArithSeqOut (HsVar enum_from) (From expr'))
+    returnM (ArithSeqOut (nlHsVar enum_from) (From expr'))
 
-tcMonoExpr in_expr@(ArithSeqIn seq@(FromThen expr1 expr2)) res_ty
+tc_expr in_expr@(ArithSeqIn seq@(FromThen expr1 expr2)) res_ty
   = addErrCtxt (arithSeqCtxt in_expr) $ 
     zapToListTy  res_ty                                `thenM`    \ elt_ty ->  
     tcCheckRho expr1 elt_ty                            `thenM`    \ expr1' ->
@@ -563,10 +515,10 @@ tcMonoExpr in_expr@(ArithSeqIn seq@(FromThen expr1 expr2)) res_ty
     newMethodFromName (ArithSeqOrigin seq) 
                      elt_ty enumFromThenName           `thenM` \ enum_from_then ->
 
-    returnM (ArithSeqOut (HsVar enum_from_then) (FromThen expr1' expr2'))
+    returnM (ArithSeqOut (nlHsVar enum_from_then) (FromThen expr1' expr2'))
 
 
-tcMonoExpr in_expr@(ArithSeqIn seq@(FromTo expr1 expr2)) res_ty
+tc_expr in_expr@(ArithSeqIn seq@(FromTo expr1 expr2)) res_ty
   = addErrCtxt (arithSeqCtxt in_expr) $
     zapToListTy  res_ty                                `thenM`    \ elt_ty ->  
     tcCheckRho expr1 elt_ty                            `thenM`    \ expr1' ->
@@ -574,9 +526,9 @@ tcMonoExpr in_expr@(ArithSeqIn seq@(FromTo expr1 expr2)) res_ty
     newMethodFromName (ArithSeqOrigin seq) 
                      elt_ty enumFromToName             `thenM` \ enum_from_to ->
 
-    returnM (ArithSeqOut (HsVar enum_from_to) (FromTo expr1' expr2'))
+    returnM (ArithSeqOut (nlHsVar enum_from_to) (FromTo expr1' expr2'))
 
-tcMonoExpr in_expr@(ArithSeqIn seq@(FromThenTo expr1 expr2 expr3)) res_ty
+tc_expr in_expr@(ArithSeqIn seq@(FromThenTo expr1 expr2 expr3)) res_ty
   = addErrCtxt  (arithSeqCtxt in_expr) $
     zapToListTy  res_ty                                `thenM`    \ elt_ty ->  
     tcCheckRho expr1 elt_ty                            `thenM`    \ expr1' ->
@@ -585,9 +537,9 @@ tcMonoExpr in_expr@(ArithSeqIn seq@(FromThenTo expr1 expr2 expr3)) res_ty
     newMethodFromName (ArithSeqOrigin seq) 
                      elt_ty enumFromThenToName         `thenM` \ eft ->
 
-    returnM (ArithSeqOut (HsVar eft) (FromThenTo expr1' expr2' expr3'))
+    returnM (ArithSeqOut (nlHsVar eft) (FromThenTo expr1' expr2' expr3'))
 
-tcMonoExpr in_expr@(PArrSeqIn seq@(FromTo expr1 expr2)) res_ty
+tc_expr in_expr@(PArrSeqIn seq@(FromTo expr1 expr2)) res_ty
   = addErrCtxt (parrSeqCtxt in_expr) $
     zapToPArrTy  res_ty                                `thenM`    \ elt_ty ->  
     tcCheckRho expr1 elt_ty                            `thenM`    \ expr1' ->
@@ -595,9 +547,9 @@ tcMonoExpr in_expr@(PArrSeqIn seq@(FromTo expr1 expr2)) res_ty
     newMethodFromName (PArrSeqOrigin seq) 
                      elt_ty enumFromToPName            `thenM` \ enum_from_to ->
 
-    returnM (PArrSeqOut (HsVar enum_from_to) (FromTo expr1' expr2'))
+    returnM (PArrSeqOut (nlHsVar enum_from_to) (FromTo expr1' expr2'))
 
-tcMonoExpr in_expr@(PArrSeqIn seq@(FromThenTo expr1 expr2 expr3)) res_ty
+tc_expr in_expr@(PArrSeqIn seq@(FromThenTo expr1 expr2 expr3)) res_ty
   = addErrCtxt  (parrSeqCtxt in_expr) $
     zapToPArrTy  res_ty                                `thenM`    \ elt_ty ->  
     tcCheckRho expr1 elt_ty                            `thenM`    \ expr1' ->
@@ -606,9 +558,9 @@ tcMonoExpr in_expr@(PArrSeqIn seq@(FromThenTo expr1 expr2 expr3)) res_ty
     newMethodFromName (PArrSeqOrigin seq)
                      elt_ty enumFromThenToPName        `thenM` \ eft ->
 
-    returnM (PArrSeqOut (HsVar eft) (FromThenTo expr1' expr2' expr3'))
+    returnM (PArrSeqOut (nlHsVar eft) (FromThenTo expr1' expr2' expr3'))
 
-tcMonoExpr (PArrSeqIn _) _ 
+tc_expr (PArrSeqIn _) _ 
   = panic "TcExpr.tcMonoExpr: Infinite parallel array!"
     -- the parser shouldn't have generated it and the renamer shouldn't have
     -- let it through
@@ -624,20 +576,9 @@ tcMonoExpr (PArrSeqIn _) _
 \begin{code}
 #ifdef GHCI    /* Only if bootstrapped */
        -- Rename excludes these cases otherwise
-
-tcMonoExpr (HsSplice n expr loc) res_ty = addSrcLoc loc (tcSpliceExpr n expr res_ty)
-tcMonoExpr (HsBracket brack loc) res_ty = addSrcLoc loc (tcBracket brack res_ty)
-
-tcMonoExpr (HsReify (Reify flavour name)) res_ty
-  = addErrCtxt (ptext SLIT("At the reification of") <+> ppr name)      $
-    tcMetaTy  tycon_name               `thenM` \ reify_ty ->
-    zapExpectedTo res_ty reify_ty      `thenM_`
-    returnM (HsReify (ReifyOut flavour name))
-  where
-    tycon_name = case flavour of
-                  ReifyDecl -> DsMeta.decQTyConName
-                  ReifyType -> DsMeta.typeQTyConName
-                  ReifyFixity -> pprPanic "tcMonoExpr: cant do reifyFixity yet" (ppr name)
+tc_expr (HsSpliceE splice) res_ty = tcSpliceExpr splice res_ty
+tc_expr (HsBracket brack)  res_ty = do { e <- tcBracket brack res_ty
+                                       ; return (unLoc e) }
 #endif /* GHCI */
 \end{code}
 
@@ -649,7 +590,7 @@ tcMonoExpr (HsReify (Reify flavour name)) res_ty
 %************************************************************************
 
 \begin{code}
-tcMonoExpr other _ = pprPanic "tcMonoExpr" (ppr other)
+tc_expr other _ = pprPanic "tcMonoExpr" (ppr other)
 \end{code}
 
 
@@ -661,11 +602,11 @@ tcMonoExpr other _ = pprPanic "tcMonoExpr" (ppr other)
 
 \begin{code}
 
-tcApp :: RenamedHsExpr -> [RenamedHsExpr]      -- Function and args
+tcApp :: LHsExpr Name -> [LHsExpr Name]        -- Function and args
       -> Expected TcRhoType                    -- Expected result type of application
-      -> TcM TcExpr                            -- Translated fun and args
+      -> TcM (HsExpr TcId)                             -- Translated fun and args
 
-tcApp (HsApp e1 e2) args res_ty 
+tcApp (L _ (HsApp e1 e2)) args res_ty 
   = tcApp e1 (e2:args) res_ty          -- Accumulate the arguments
 
 tcApp fun args res_ty
@@ -705,7 +646,7 @@ tcApp fun args res_ty
     mappM (tcArg fun)
          (zip3 args expected_arg_tys [1..])    `thenM` \ args' ->
 
-    returnM (co_fn <$> foldl HsApp fun' args') 
+    returnM (co_fn <$> unLoc (foldl mkHsApp fun' args'))
 
 
 -- If an error happens we try to figure out whether the
@@ -748,9 +689,9 @@ split_fun_ty fun_ty n
 \end{code}
 
 \begin{code}
-tcArg :: RenamedHsExpr                         -- The function (for error messages)
-      -> (RenamedHsExpr, TcSigmaType, Int)     -- Actual argument and expected arg type
-      -> TcM TcExpr                            -- Resulting argument and LIE
+tcArg :: LHsExpr Name                          -- The function (for error messages)
+      -> (LHsExpr Name, TcSigmaType, Int)      -- Actual argument and expected arg type
+      -> TcM (LHsExpr TcId)                    -- Resulting argument
 
 tcArg the_fun (arg, expected_arg_ty, arg_no)
   = addErrCtxt (funAppCtxt the_fun arg arg_no) $
@@ -787,20 +728,20 @@ This gets a bit less sharing, but
        b) perhaps fewer separated lambdas
 
 \begin{code}
-tcId :: Name -> TcM (TcExpr, TcRhoType)
+tcId :: Name -> TcM (HsExpr TcId, TcRhoType)
 tcId name      -- Look up the Id and instantiate its type
   =    -- First check whether it's a DataCon
        -- Reason: we must not forget to chuck in the
        --         constraints from their "silly context"
-    tcLookup name              `thenM` \ maybe_thing ->
-    case maybe_thing of {
+    tcLookup name              `thenM` \ thing ->
+    case thing of {
        AGlobal (ADataCon data_con)  -> inst_data_con data_con 
     ;  AGlobal (AnId id)            -> loop (HsVar id) (idType id)
                -- A global cannot possibly be ill-staged
                -- nor does it need the 'lifting' treatment
 
     ;  ATcId id th_level proc_level -> tc_local_id id th_level proc_level
-    ;  other                        -> pprPanic "tcId" (ppr name)
+    ;  other                        -> pprPanic "tcId" (ppr name $$ ppr thing)
     }
   where
 
@@ -843,7 +784,7 @@ tcId name   -- Look up the Id and instantiate its type
        
                -- Update the pending splices
                readMutVar ps_var                       `thenM` \ ps ->
-               writeMutVar ps_var ((name, HsApp (HsVar lift) (HsVar id)) : ps) `thenM_`
+               writeMutVar ps_var ((name, nlHsApp (nlHsVar lift) (nlHsVar id)) : ps)   `thenM_`
        
                returnM (HsVar id, id_ty))
 
@@ -889,9 +830,11 @@ tcId name  -- Look up the Id and instantiate its type
     inst_data_con data_con
       = tcInstDataCon orig data_con    `thenM` \ (ty_args, ex_dicts, arg_tys, result_ty, _) ->
        extendLIEs ex_dicts             `thenM_`
-       returnM (mkHsDictApp (mkHsTyApp (HsVar (dataConWrapId data_con)) ty_args) 
-                            (map instToId ex_dicts), 
+       getSrcSpanM                     `thenM` \ loc ->
+       returnM (unLoc (mkHsDictApp (mkHsTyApp (L loc (HsVar (dataConWrapId data_con))) ty_args) 
+                            (map instToId ex_dicts)), 
                 mkFunTys arg_tys result_ty)
+       -- ToDo: nasty loc/unloc stuff here
 
     orig = OccurrenceOf name
 \end{code}
@@ -923,17 +866,17 @@ This extends OK when the field types are universally quantified.
 tcRecordBinds
        :: TyCon                -- Type constructor for the record
        -> [TcType]             -- Args of this type constructor
-       -> RenamedRecordBinds
-       -> TcM TcRecordBinds
+       -> HsRecordBinds Name
+       -> TcM (HsRecordBinds TcId)
 
 tcRecordBinds tycon ty_args rbinds
   = mappM do_bind rbinds
   where
     tenv = mkTopTyVarSubst (tyConTyVars tycon) ty_args
 
-    do_bind (field_lbl_name, rhs)
+    do_bind (L loc field_lbl_name, rhs)
       = addErrCtxt (fieldCtxt field_lbl_name)  $
-           tcLookupId field_lbl_name           `thenM` \ sel_id ->
+        tcLookupId field_lbl_name              `thenM` \ sel_id ->
        let
            field_lbl = recordSelectorFieldLabel sel_id
            field_ty  = substTy tenv (fieldLabelType field_lbl)
@@ -948,14 +891,14 @@ tcRecordBinds tycon ty_args rbinds
 
        tcCheckSigma rhs field_ty               `thenM` \ rhs' ->
 
-       returnM (sel_id, rhs')
+       returnM (L loc sel_id, rhs')
 
 badFields rbinds data_con
   = filter (not . (`elem` field_names)) (recBindFields rbinds)
   where
     field_names = map fieldLabelName (dataConFieldLabels data_con)
 
-checkMissingFields :: DataCon -> RenamedRecordBinds -> TcM ()
+checkMissingFields :: DataCon -> HsRecordBinds Name -> TcM ()
 checkMissingFields data_con rbinds
   | null field_labels  -- Not declared as a record;
                        -- But C{} is still valid if no strict fields
@@ -992,10 +935,7 @@ checkMissingFields data_con rbinds
                          field_labels
                          field_strs
 
-    field_strs = dropList ex_theta (dataConStrictMarks data_con)
-       -- The 'drop' is because dataConStrictMarks
-       -- includes the existential dictionaries
-    (_, _, _, ex_theta, _, _) = dataConSig data_con
+    field_strs = dataConStrictMarks data_con
 \end{code}
 
 %************************************************************************
@@ -1005,7 +945,7 @@ checkMissingFields data_con rbinds
 %************************************************************************
 
 \begin{code}
-tcCheckRhos :: [RenamedHsExpr] -> [TcType] -> TcM [TcExpr]
+tcCheckRhos :: [LHsExpr Name] -> [TcType] -> TcM [LHsExpr TcId]
 
 tcCheckRhos [] [] = returnM []
 tcCheckRhos (expr:exprs) (ty:tys)
@@ -1024,15 +964,7 @@ tcCheckRhos (expr:exprs) (ty:tys)
 Overloaded literals.
 
 \begin{code}
-tcLit :: HsLit -> Expected TcRhoType -> TcM TcExpr
-tcLit (HsLitLit s _) res_ty
-  = zapExpectedType res_ty                             `thenM` \ res_ty' ->
-    tcLookupClass cCallableClassName                   `thenM` \ cCallableClass ->
-    newDicts (LitLitOrigin (unpackFS s))
-            [mkClassPred cCallableClass [res_ty']]     `thenM` \ dicts ->
-    extendLIEs dicts                                   `thenM_`
-    returnM (HsLit (HsLitLit s res_ty'))
-
+tcLit :: HsLit -> Expected TcRhoType -> TcM (HsExpr TcId)
 tcLit lit res_ty 
   = zapExpectedTo res_ty (hsLitType lit)               `thenM_`
     returnM (HsLit lit)
@@ -1059,10 +991,6 @@ caseCtxt expr
 caseScrutCtxt expr
   = hang (ptext SLIT("In the scrutinee of a case expression:")) 4 (ppr expr)
 
-exprSigCtxt expr
-  = hang (ptext SLIT("When checking the type signature of the expression:"))
-        4 (ppr expr)
-
 exprCtxt expr
   = hang (ptext SLIT("In the expression:")) 4 (ppr expr)
 
@@ -1086,12 +1014,7 @@ predCtxt expr
 appCtxt fun args
   = ptext SLIT("In the application") <+> quotes (ppr the_app)
   where
-    the_app = foldl HsApp fun args     -- Used in error messages
-
-lurkingRank2Err fun fun_ty
-  = hang (hsep [ptext SLIT("Illegal use of"), quotes (ppr fun)])
-        4 (vcat [ptext SLIT("It is applied to too few arguments"),  
-                 ptext SLIT("so that the result type has for-alls in it:") <+> ppr fun_ty])
+    the_app = foldl mkHsApp fun args   -- Used in error messages
 
 badFieldsUpd rbinds
   = hang (ptext SLIT("No constructor has all these fields:"))
@@ -1119,15 +1042,17 @@ missingFields con fields
   = ptext SLIT("Fields of") <+> quotes (ppr con) <+> ptext SLIT("not initialised:") 
        <+> pprWithCommas ppr fields
 
-polySpliceErr :: Id -> SDoc
-polySpliceErr id
-  = ptext SLIT("Can't splice the polymorphic local variable") <+> quotes (ppr id)
-
 wrongArgsCtxt too_many_or_few fun args
   = hang (ptext SLIT("Probable cause:") <+> quotes (ppr fun)
                    <+> ptext SLIT("is applied to") <+> text too_many_or_few 
                    <+> ptext SLIT("arguments in the call"))
         4 (parens (ppr the_app))
   where
-    the_app = foldl HsApp fun args     -- Used in error messages
+    the_app = foldl mkHsApp fun args   -- Used in error messages
+
+#ifdef GHCI
+polySpliceErr :: Id -> SDoc
+polySpliceErr id
+  = ptext SLIT("Can't splice the polymorphic local variable") <+> quotes (ppr id)
+#endif
 \end{code}