[project @ 2005-04-05 08:25:06 by simonpj]
[ghc-hetmet.git] / ghc / compiler / typecheck / TcExpr.lhs
index 6ea75a2..6d441b2 100644 (file)
@@ -4,66 +4,75 @@
 \section[TcExpr]{Typecheck an expression}
 
 \begin{code}
-module TcExpr ( tcCheckSigma, tcCheckRho, tcInferRho, tcMonoExpr ) where
+module TcExpr ( tcCheckSigma, tcCheckRho, tcInferRho, 
+               tcMonoExpr, tcExpr, tcSyntaxOp
+   ) where
 
 #include "HsVersions.h"
 
 #ifdef GHCI    /* Only if bootstrapped */
 import {-# SOURCE #-}  TcSplice( tcSpliceExpr, tcBracket )
+import HsSyn           ( nlHsVar )
 import Id              ( Id )
+import Name            ( isExternalName )
 import TcType          ( isTauTy )
-import TcEnv           ( tcMetaTy, checkWellStaged )
+import TcEnv           ( checkWellStaged )
+import HsSyn           ( nlHsApp )
 import qualified DsMeta
 #endif
 
-import HsSyn           ( HsExpr(..), HsLit(..), ArithSeqInfo(..), recBindFields,
-                         HsMatchContext(..) )
-import RnHsSyn         ( RenamedHsExpr, RenamedRecordBinds )
-import TcHsSyn         ( TcExpr, TcRecordBinds, hsLitType, mkHsDictApp, mkHsTyApp, (<$>) )
+import HsSyn           ( HsExpr(..), LHsExpr, HsLit(..), ArithSeqInfo(..), recBindFields,
+                         HsMatchContext(..), HsRecordBinds, mkHsApp )
+import TcHsSyn         ( hsLitType, (<$>) )
 import TcRnMonad
-import TcUnify         ( Expected(..), newHole, zapExpectedType, zapExpectedTo, tcSubExp, tcGen,
-                         unifyFunTy, zapToListTy, zapToPArrTy, zapToTupleTy )
+import TcUnify         ( Expected(..), tcInfer, zapExpectedType, zapExpectedTo, 
+                         tcSubExp, tcGen, tcSub,
+                         unifyFunTys, zapToListTy, zapToTyConApp )
 import BasicTypes      ( isMarkedStrict )
-import Inst            ( InstOrigin(..), 
-                         newOverloadedLit, newMethodFromName, newIPDict,
-                         newDicts, newMethodWithGivenTy, 
-                         instToId, tcInstCall, tcInstDataCon
-                       )
+import Inst            ( tcOverloadedLit, newMethodFromName, newIPDict,
+                         newDicts, newMethodWithGivenTy, tcInstStupidTheta, tcInstCall )
 import TcBinds         ( tcBindsAndThen )
-import TcEnv           ( tcLookup, tcLookupGlobalId, 
-                         tcLookupDataCon, tcLookupId, checkProcLevel
+import TcEnv           ( tcLookup, tcLookupId, checkProcLevel,
+                         tcLookupDataCon, tcLookupGlobalId
                        )
 import TcArrows                ( tcProc )
 import TcMatches       ( tcMatchesCase, tcMatchLambda, tcDoStmts, tcThingWithSig, TcMatchCtxt(..) )
 import TcHsType                ( tcHsSigType, UserTypeCtxt(..) )
-import TcPat           ( badFieldCon )
-import TcMType         ( tcInstTyVars, tcInstType, newTyVarTy, zonkTcType )
-import TcType          ( TcType, TcSigmaType, TcRhoType, TyVarDetails(VanillaTv),
+import TcPat           ( badFieldCon, refineTyVars )
+import TcMType         ( tcInstTyVars, tcInstType, newTyFlexiVarTy, zonkTcType )
+import TcType          ( Type, TcTyVar, TcType, TcSigmaType, TcRhoType, 
                          tcSplitFunTys, tcSplitTyConApp, mkTyVarTys,
-                         isSigmaTy, mkFunTy, mkFunTys,
-                         mkTyConApp, tyVarsOfTypes, isLinearPred,
-                         liftedTypeKind, openTypeKind, 
+                         isSigmaTy, mkFunTy, mkTyConApp, tyVarsOfTypes, isLinearPred,
                          tcSplitSigmaTy, tidyOpenType
                        )
-import FieldLabel      ( FieldLabel, fieldLabelName, fieldLabelType, fieldLabelTyCon )
+import Kind            ( openTypeKind, liftedTypeKind, argTypeKind )
+
 import Id              ( idType, recordSelectorFieldLabel, isRecordSelector )
-import DataCon         ( DataCon, dataConFieldLabels, dataConStrictMarks, dataConWrapId )
+import DataCon         ( DataCon, dataConFieldLabels, dataConStrictMarks, 
+                         dataConWrapId )
 import Name            ( Name )
-import TyCon           ( TyCon, tyConTyVars, tyConTheta, isAlgTyCon, tyConDataCons )
-import Subst           ( mkTopTyVarSubst, substTheta, substTy )
+import TyCon           ( TyCon, FieldLabel, tyConTyVars, tyConStupidTheta, 
+                         tyConDataCons, tyConFields )
+import Type            ( zipTopTvSubst, substTheta, substTy )
+import Var             ( tyVarKind )
 import VarSet          ( emptyVarSet, elemVarSet )
-import TysWiredIn      ( boolTy )
+import TysWiredIn      ( boolTy, parrTyCon, tupleTyCon )
 import PrelNames       ( enumFromName, enumFromThenName, 
                          enumFromToName, enumFromThenToName,
-                         enumFromToPName, enumFromThenToPName
+                         enumFromToPName, enumFromThenToPName, negateName
                        )
 import ListSetOps      ( minusList )
-import CmdLineOpts
+import DynFlags
+import StaticFlags     ( opt_NoMethodSharing )
 import HscTypes                ( TyThing(..) )
-
+import SrcLoc          ( Located(..), unLoc, getLoc )
 import Util
 import Outputable
 import FastString
+
+#ifdef DEBUG
+import TyCon           ( isAlgTyCon )
+#endif
 \end{code}
 
 %************************************************************************
@@ -74,9 +83,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_`
@@ -87,7 +96,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
@@ -99,44 +108,57 @@ 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)) = setSrcSpan loc $ do 
+                                 { (e,_,ty) <- tcId (OccurrenceOf name) name
+                                 ; return (L loc e, ty) }
+tcInferRho expr                        = tcInfer (tcMonoExpr expr)
+
+tcSyntaxOp :: InstOrigin -> HsExpr Name -> TcType -> TcM (HsExpr TcId)
+-- Typecheck a syntax operator, checking that it has the specified type
+-- The operator is always a variable at this stage (i.e. renamer output)
+tcSyntaxOp orig (HsVar op) ty = do { (expr', _, id_ty) <- tcId orig op
+                                  ; co_fn <- tcSub ty id_ty
+                                  ; returnM (co_fn <$> expr') }
+tcSyntaxOp orig other     ty = pprPanic "tcSyntaxOp" (ppr other)
 \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
+  = setSrcSpan loc (do { expr' <- tcExpr expr res_ty
+                      ; return (L loc expr') })
 
-tcMonoExpr (HsVar name) res_ty
-  = tcId name                  `thenM` \ (expr', id_ty) ->
-    tcSubExp res_ty id_ty      `thenM` \ co_fn ->
-    returnM (co_fn <$> expr')
+tcExpr :: HsExpr Name -> Expected TcRhoType -> TcM (HsExpr TcId)
+tcExpr (HsVar name) res_ty
+  = do { (expr', _, id_ty) <- tcId (OccurrenceOf name) name
+       ; co_fn <- tcSubExp res_ty id_ty
+       ; returnM (co_fn <$> expr') }
 
-tcMonoExpr (HsIPVar ip) res_ty
+tcExpr (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) ->
+    newTyFlexiVarTy 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')
@@ -150,13 +172,13 @@ tcMonoExpr (HsIPVar ip) res_ty
 %************************************************************************
 
 \begin{code}
-tcMonoExpr in_expr@(ExprWithTySig expr poly_ty) res_ty
- = addErrCtxt (exprSigCtxt in_expr)                    $
+tcExpr 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 <$> ExprWithTySigOut expr' poly_ty)
 
-tcMonoExpr (HsType ty) res_ty
+tcExpr (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)
@@ -173,25 +195,35 @@ 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
+tcExpr (HsPar expr)    res_ty  = tcMonoExpr expr res_ty        `thenM` \ expr' -> 
+                                 returnM (HsPar expr')
+tcExpr (HsSCC lbl expr) res_ty = tcMonoExpr expr res_ty        `thenM` \ expr' ->
+                                 returnM (HsSCC lbl expr')
+tcExpr (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
-       -- ToDo: use tcSyntaxName
 
-tcMonoExpr (HsLam match) res_ty
+tcExpr (HsLit lit) res_ty  = tcLit lit res_ty
+
+tcExpr (HsOverLit lit) res_ty  
+  = zapExpectedType res_ty liftedTypeKind              `thenM` \ res_ty' ->
+       -- Overloaded literals must have liftedTypeKind, because
+       -- we're instantiating an overloaded function here,
+       -- whereas res_ty might be openTypeKind. This was a bug in 6.2.2
+    tcOverloadedLit (LiteralOrigin lit) lit res_ty'    `thenM` \ lit' ->
+    returnM (HsOverLit lit')
+
+tcExpr (NegApp expr neg_expr) res_ty
+  = do { res_ty' <- zapExpectedType res_ty liftedTypeKind
+       ; neg_expr' <- tcSyntaxOp (OccurrenceOf negateName) neg_expr
+                                 (mkFunTy res_ty' res_ty')
+       ; expr' <- tcCheckRho expr res_ty'
+       ; return (NegApp expr' neg_expr') }
+
+tcExpr (HsLam match) res_ty
   = tcMatchLambda match res_ty                 `thenM` \ match' ->
     returnM (HsLam match')
 
-tcMonoExpr (HsApp e1 e2) res_ty 
+tcExpr (HsApp e1 e2) res_ty 
   = tcApp e1 [e2] res_ty
 \end{code}
 
@@ -206,9 +238,9 @@ a type error will occur if they aren't.
 -- or just
 --     op e
 
-tcMonoExpr in_expr@(SectionL arg1 op) res_ty
+tcExpr 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) ->
+    unifyFunTys 2 op_ty {- two args -}         `thenM` \ ([arg1_ty, arg2_ty], op_res_ty) ->
     tcArg op (arg1, arg1_ty, 1)                        `thenM` \ arg1' ->
     addErrCtxt (exprCtxt in_expr)              $
     tcSubExp res_ty (mkFunTy arg2_ty op_res_ty)        `thenM` \ co_fn ->
@@ -217,9 +249,9 @@ 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
+tcExpr 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) ->
+    unifyFunTys 2 op_ty {- two args -}         `thenM` \ ([arg1_ty, arg2_ty], op_res_ty) ->
     tcArg op (arg2, arg2_ty, 2)                        `thenM` \ arg2' ->
     addErrCtxt (exprCtxt in_expr)              $
     tcSubExp res_ty (mkFunTy arg1_ty op_res_ty)        `thenM` \ co_fn ->
@@ -227,9 +259,9 @@ tcMonoExpr in_expr@(SectionR op arg2) res_ty
 
 -- equivalent to (op e1) e2:
 
-tcMonoExpr in_expr@(OpApp arg1 op fix arg2) res_ty
+tcExpr 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) ->
+    unifyFunTys 2 op_ty {- two args -}         `thenM` \ ([arg1_ty, arg2_ty], op_res_ty) ->
     tcArg op (arg1, arg1_ty, 1)                        `thenM` \ arg1' ->
     tcArg op (arg2, arg2_ty, 2)                        `thenM` \ arg2' ->
     addErrCtxt (exprCtxt in_expr)              $
@@ -238,54 +270,49 @@ tcMonoExpr in_expr@(OpApp arg1 op fix arg2) res_ty
 \end{code}
 
 \begin{code}
-tcMonoExpr (HsLet binds expr) res_ty
+tcExpr (HsLet binds (L loc expr)) res_ty
   = tcBindsAndThen
-       HsLet
+       glue
        binds                   -- Bindings to check
-       (tcMonoExpr expr res_ty)
-
-tcMonoExpr in_expr@(HsCase scrut matches src_loc) res_ty
-  = addSrcLoc src_loc                  $
-    addErrCtxt (caseCtxt in_expr)      $
+       (setSrcSpan loc $ tcExpr expr res_ty)
+  where
+    glue bind expr = HsLet [bind] (L loc expr)
 
-       -- Typecheck the case alternatives first.
+tcExpr in_expr@(HsCase scrut matches) exp_ty
+  =    -- We used to typecheck the case alternatives first.
        -- The case patterns tend to give good type info to use
        -- when typechecking the scrutinee.  For example
        --      case (map f) of
        --        (x:xs) -> ...
        -- will report that map is applied to too few arguments
-
-    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)
-  where
+       --
+       -- But now, in the GADT world, we need to typecheck the scrutinee
+       -- first, to get type info that may be refined in the case alternatives
+    addErrCtxt (caseScrutCtxt scrut)
+              (tcInferRho scrut)       `thenM`    \ (scrut', scrut_ty) ->
+
+    addErrCtxt (caseCtxt in_expr)                      $
+    tcMatchesCase match_ctxt scrut_ty matches exp_ty   `thenM` \ matches' ->
+    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) (
-    tcCheckRho pred boolTy     )       `thenM`    \ pred' ->
+tcExpr (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
-    tcDoStmts do_or_lc stmts method_names res_ty'      `thenM` \ (stmts', methods') ->
-    returnM (HsDo do_or_lc stmts' methods' res_ty' src_loc)
+tcExpr (HsDo do_or_lc stmts body _) res_ty
+  = tcDoStmts do_or_lc stmts body res_ty
 
-tcMonoExpr in_expr@(ExplicitList _ exprs) res_ty       -- Non-empty list
+tcExpr 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,24 +321,30 @@ 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
-  = zapToPArrTy res_ty                `thenM` \ elt_ty ->  
-    mappM (tc_elt elt_ty) exprs              `thenM` \ exprs' ->
-    returnM (ExplicitPArr elt_ty exprs')
+tcExpr in_expr@(ExplicitPArr _ exprs) res_ty   -- maybe empty
+  = do { [elt_ty] <- zapToTyConApp parrTyCon res_ty
+       ; exprs' <- mappM (tc_elt elt_ty) exprs 
+       ; return (ExplicitPArr elt_ty exprs') }
   where
     tc_elt elt_ty expr
-      = addErrCtxt (parrCtxt expr) $
-       tcCheckRho expr elt_ty
+      = addErrCtxt (parrCtxt expr) (tcCheckRho expr elt_ty)
 
-tcMonoExpr (ExplicitTuple exprs boxity) res_ty
-  = zapToTupleTy boxity (length exprs) res_ty  `thenM` \ arg_tys ->
-    tcCheckRhos exprs arg_tys                  `thenM` \ exprs' ->
-    returnM (ExplicitTuple exprs' boxity)
+tcExpr (ExplicitTuple exprs boxity) res_ty
+  = do { arg_tys <- zapToTyConApp (tupleTyCon boxity (length exprs)) res_ty
+       ; exprs' <-  tcCheckRhos exprs arg_tys
+       ; return (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)
+tcExpr (HsProc pat cmd) res_ty
+  = tcProc pat cmd res_ty                      `thenM` \ (pat', cmd') ->
+    returnM (HsProc pat' cmd')
+
+tcExpr e@(HsArrApp _ _ _ _ _) _
+  = failWithTc (vcat [ptext SLIT("The arrow command"), nest 2 (ppr e), 
+                      ptext SLIT("was found where an expression was expected")])
+
+tcExpr e@(HsArrForm _ _ _) _
+  = failWithTc (vcat [ptext SLIT("The arrow command"), nest 2 (ppr e), 
+                      ptext SLIT("was found where an expression was expected")])
 \end{code}
 
 %************************************************************************
@@ -321,9 +354,9 @@ tcMonoExpr (HsProc pat cmd loc) res_ty
 %************************************************************************
 
 \begin{code}
-tcMonoExpr expr@(RecordCon con_name rbinds) res_ty
+tcExpr expr@(RecordCon con@(L loc con_name) _ rbinds) res_ty
   = addErrCtxt (recordConCtxt expr)            $
-    tcId con_name                      `thenM` \ (con_expr, con_tau) ->
+    addLocM (tcId (OccurrenceOf con_name)) con `thenM` \ (con_expr, _, con_tau) ->
     let
        (_, record_ty)   = tcSplitFunTys con_tau
        (tycon, ty_args) = tcSplitTyConApp record_ty
@@ -348,7 +381,7 @@ tcMonoExpr expr@(RecordCon con_name rbinds) res_ty
        -- Check for missing fields
     checkMissingFields data_con rbinds         `thenM_` 
 
-    returnM (RecordConOut data_con con_expr rbinds')
+    returnM (RecordCon (L loc (dataConWrapId data_con)) con_expr rbinds')
 
 -- The main complication with RecordUpd is that we need to explicitly
 -- handle the *non-updated* fields.  Consider:
@@ -376,21 +409,21 @@ tcMonoExpr expr@(RecordCon con_name rbinds) res_ty
 --
 -- All this is done in STEP 4 below.
 
-tcMonoExpr expr@(RecordUpd record_expr rbinds) res_ty
+tcExpr 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 tcLookupGlobalId field_names         `thenM` \ 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, sel_id) <- field_names `zip` sel_ids,
+       bad_guys = [ setSrcSpan loc $ addErrTc (notSelector field_name) 
+                  | (L loc field_name, sel_id) <- field_names `zip` sel_ids,
                     not (isRecordSelector sel_id)      -- Excludes class ops
                   ]
     in
@@ -401,18 +434,17 @@ tcMonoExpr expr@(RecordUpd record_expr rbinds) res_ty
     let
                -- It's OK to use the non-tc splitters here (for a selector)
        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
+       (tycon, _)   = recordSelectorFieldLabel sel_id  -- We've failed already if
+       data_cons    = tyConDataCons tycon              -- it's not a field label
        tycon_tyvars = tyConTyVars tycon                -- The data cons use the same type vars
     in
-    tcInstTyVars VanillaTv tycon_tyvars                `thenM` \ (_, result_inst_tys, inst_env) ->
+    tcInstTyVars tycon_tyvars          `thenM` \ (_, result_inst_tys, inst_env) ->
 
        -- STEP 2
        -- Check that at least one constructor has all the named fields
        -- i.e. has an empty set of bad fields returned by badFields
     checkTc (any (null . badFields rbinds) data_cons)
-           (badFieldsUpd rbinds)               `thenM_`
+           (badFieldsUpd rbinds)       `thenM_`
 
        -- STEP 3
        -- Typecheck the update bindings.
@@ -431,7 +463,7 @@ tcMonoExpr expr@(RecordUpd record_expr rbinds) res_ty
        -- WARNING: this code assumes that all data_cons in a common tycon
        -- have FieldLabels abstracted over the same tyvars.
     let
-       upd_field_lbls      = map recordSelectorFieldLabel (recBindFields rbinds')
+       upd_field_lbls      = recBindFields rbinds
        con_field_lbls_s    = map dataConFieldLabels data_cons
 
                -- A constructor is only relevant to this process if
@@ -440,13 +472,15 @@ tcMonoExpr expr@(RecordUpd record_expr rbinds) res_ty
        is_relevant con_field_lbls = all (`elem` con_field_lbls) upd_field_lbls
 
        non_upd_field_lbls  = concat relevant_field_lbls_s `minusList` upd_field_lbls
-       common_tyvars       = tyVarsOfTypes (map fieldLabelType non_upd_field_lbls)
+       common_tyvars       = tyVarsOfTypes [ty | (fld,ty,_) <- tyConFields tycon,
+                                                 fld `elem` non_upd_field_lbls]
+       is_common_tv tv = tv `elemVarSet` common_tyvars
 
-       mk_inst_ty (tyvar, result_inst_ty) 
-         | tyvar `elemVarSet` common_tyvars = returnM result_inst_ty   -- Same as result type
-         | otherwise                        = newTyVarTy liftedTypeKind        -- Fresh type
+       mk_inst_ty tv result_inst_ty 
+         | is_common_tv tv = returnM result_inst_ty            -- Same as result type
+         | otherwise       = newTyFlexiVarTy (tyVarKind tv)    -- Fresh type, of correct kind
     in
-    mappM mk_inst_ty (zip tycon_tyvars result_inst_tys)        `thenM` \ inst_tys ->
+    zipWithM mk_inst_ty tycon_tyvars result_inst_tys   `thenM` \ inst_tys ->
 
        -- STEP 5
        -- Typecheck the expression to be updated
@@ -463,13 +497,13 @@ tcMonoExpr expr@(RecordUpd record_expr rbinds) res_ty
        -- What dictionaries do we need?  
        -- We just take the context of the type constructor
     let
-       theta' = substTheta inst_env (tyConTheta tycon)
+       theta' = substTheta inst_env (tyConStupidTheta tycon)
     in
     newDicts RecordUpdOrigin theta'    `thenM` \ dicts ->
     extendLIEs dicts                   `thenM_`
 
        -- Phew!
-    returnM (RecordUpdOut record_expr' record_ty result_record_ty rbinds') 
+    returnM (RecordUpd record_expr' rbinds' record_ty result_record_ty) 
 \end{code}
 
 
@@ -482,16 +516,16 @@ tcMonoExpr expr@(RecordUpd record_expr rbinds) res_ty
 %************************************************************************
 
 \begin{code}
-tcMonoExpr (ArithSeqIn seq@(From expr)) res_ty
+tcExpr (ArithSeq _ 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 (ArithSeq (HsVar enum_from) (From expr'))
 
-tcMonoExpr in_expr@(ArithSeqIn seq@(FromThen expr1 expr2)) res_ty
+tcExpr in_expr@(ArithSeq _ seq@(FromThen expr1 expr2)) res_ty
   = addErrCtxt (arithSeqCtxt in_expr) $ 
     zapToListTy  res_ty                                `thenM`    \ elt_ty ->  
     tcCheckRho expr1 elt_ty                            `thenM`    \ expr1' ->
@@ -499,10 +533,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 (ArithSeq (HsVar enum_from_then) (FromThen expr1' expr2'))
 
 
-tcMonoExpr in_expr@(ArithSeqIn seq@(FromTo expr1 expr2)) res_ty
+tcExpr in_expr@(ArithSeq _ seq@(FromTo expr1 expr2)) res_ty
   = addErrCtxt (arithSeqCtxt in_expr) $
     zapToListTy  res_ty                                `thenM`    \ elt_ty ->  
     tcCheckRho expr1 elt_ty                            `thenM`    \ expr1' ->
@@ -510,9 +544,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 (ArithSeq (HsVar enum_from_to) (FromTo expr1' expr2'))
 
-tcMonoExpr in_expr@(ArithSeqIn seq@(FromThenTo expr1 expr2 expr3)) res_ty
+tcExpr in_expr@(ArithSeq _ seq@(FromThenTo expr1 expr2 expr3)) res_ty
   = addErrCtxt  (arithSeqCtxt in_expr) $
     zapToListTy  res_ty                                `thenM`    \ elt_ty ->  
     tcCheckRho expr1 elt_ty                            `thenM`    \ expr1' ->
@@ -521,30 +555,30 @@ 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 (ArithSeq (HsVar eft) (FromThenTo expr1' expr2' expr3'))
 
-tcMonoExpr in_expr@(PArrSeqIn seq@(FromTo expr1 expr2)) res_ty
+tcExpr in_expr@(PArrSeq _ seq@(FromTo expr1 expr2)) res_ty
   = addErrCtxt (parrSeqCtxt in_expr) $
-    zapToPArrTy  res_ty                                `thenM`    \ elt_ty ->  
+    zapToTyConApp parrTyCon res_ty                             `thenM`    \ [elt_ty] ->  
     tcCheckRho expr1 elt_ty                            `thenM`    \ expr1' ->
     tcCheckRho expr2 elt_ty                            `thenM`    \ expr2' ->
     newMethodFromName (PArrSeqOrigin seq) 
                      elt_ty enumFromToPName            `thenM` \ enum_from_to ->
 
-    returnM (PArrSeqOut (HsVar enum_from_to) (FromTo expr1' expr2'))
+    returnM (PArrSeq (HsVar enum_from_to) (FromTo expr1' expr2'))
 
-tcMonoExpr in_expr@(PArrSeqIn seq@(FromThenTo expr1 expr2 expr3)) res_ty
+tcExpr in_expr@(PArrSeq _ seq@(FromThenTo expr1 expr2 expr3)) res_ty
   = addErrCtxt  (parrSeqCtxt in_expr) $
-    zapToPArrTy  res_ty                                `thenM`    \ elt_ty ->  
+    zapToTyConApp parrTyCon res_ty                             `thenM`    \ [elt_ty] ->  
     tcCheckRho expr1 elt_ty                            `thenM`    \ expr1' ->
     tcCheckRho expr2 elt_ty                            `thenM`    \ expr2' ->
     tcCheckRho expr3 elt_ty                            `thenM`    \ expr3' ->
     newMethodFromName (PArrSeqOrigin seq)
                      elt_ty enumFromThenToPName        `thenM` \ eft ->
 
-    returnM (PArrSeqOut (HsVar eft) (FromThenTo expr1' expr2' expr3'))
+    returnM (PArrSeq (HsVar eft) (FromThenTo expr1' expr2' expr3'))
 
-tcMonoExpr (PArrSeqIn _) _ 
+tcExpr (PArrSeq _ _) _ 
   = panic "TcExpr.tcMonoExpr: Infinite parallel array!"
     -- the parser shouldn't have generated it and the renamer shouldn't have
     -- let it through
@@ -560,9 +594,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)
+tcExpr (HsSpliceE splice) res_ty = tcSpliceExpr splice res_ty
+tcExpr (HsBracket brack)  res_ty = do  { e <- tcBracket brack res_ty
+                                       ; return (unLoc e) }
 #endif /* GHCI */
 \end{code}
 
@@ -574,7 +608,7 @@ tcMonoExpr (HsBracket brack loc) res_ty = addSrcLoc loc (tcBracket brack res_ty)
 %************************************************************************
 
 \begin{code}
-tcMonoExpr other _ = pprPanic "tcMonoExpr" (ppr other)
+tcExpr other _ = pprPanic "tcMonoExpr" (ppr other)
 \end{code}
 
 
@@ -586,59 +620,108 @@ 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
-  =    -- First type-check the function
-    tcInferRho fun                             `thenM` \ (fun', fun_ty) ->
-
-    addErrCtxt (wrongArgsCtxt "too many" fun args) (
-       traceTc (text "tcApp" <+> (ppr fun $$ ppr fun_ty))      `thenM_`
-       split_fun_ty fun_ty (length args)
-    )                                          `thenM` \ (expected_arg_tys, actual_result_ty) ->
-
-       -- Unify with expected result before (was: after) type-checking the args
-       -- so that the info from res_ty (was: args) percolates to args (was actual_result_ty).
-       -- This is when we might detect a too-few args situation.
-       -- (One can think of cases when the opposite order would give
-       -- a better error message.)
-       -- [March 2003: I'm experimenting with putting this first.  Here's an 
-       --              example where it actually makes a real difference
-       --    class C t a b | t a -> b
-       --    instance C Char a Bool
-       --
-       --    data P t a = forall b. (C t a b) => MkP b
-       --    data Q t   = MkQ (forall a. P t a)
-    
-       --    f1, f2 :: Q Char;
-       --    f1 = MkQ (MkP True)
-       --    f2 = MkQ (MkP True :: forall a. P Char a)
-       --
-       -- With the change, f1 will type-check, because the 'Char' info from
-       -- the signature is propagated into MkQ's argument. With the check
-       -- in the other order, the extra signature in f2 is reqd.]
-
-    addErrCtxtM (checkArgsCtxt fun args res_ty actual_result_ty)
-               (tcSubExp res_ty actual_result_ty)      `thenM` \ co_fn ->
-
-       -- Now typecheck the args
-    mappM (tcArg fun)
-         (zip3 args expected_arg_tys [1..])    `thenM` \ args' ->
-
-    returnM (co_fn <$> foldl HsApp fun' args') 
-
+  = do { (fun', fun_tvs, fun_tau) <- tcFun fun         -- Type-check the function
+
+       -- Extract its argument types
+       ; (expected_arg_tys, actual_res_ty)
+             <- addErrCtxt (wrongArgsCtxt "too many" fun args) $ do
+                { traceTc (text "tcApp" <+> (ppr fun $$ ppr fun_tau))
+                ; unifyFunTys (length args) fun_tau }
+
+
+       ; case res_ty of
+           Check _ -> do       -- Connect to result type first
+                               -- See Note [Push result type in]
+               { co_fn    <- tcResult fun args res_ty actual_res_ty
+               ; the_app' <- tcArgs fun fun' args expected_arg_tys
+               ; traceTc (text "tcApp: check" <+> vcat [ppr fun <+> ppr args,
+                                                        ppr the_app', ppr actual_res_ty])
+               ; returnM (co_fn <$> the_app') }
+
+           Infer _ -> do       -- Type check args first, then
+                               -- refine result type, then do tcResult
+               { the_app'       <- tcArgs fun fun' args expected_arg_tys
+               ; subst          <- refineTyVars fun_tvs
+               ; let actual_res_ty' = substTy subst actual_res_ty
+               ; co_fn          <- tcResult fun args res_ty actual_res_ty'
+               ; traceTc (text "tcApp: infer" <+> vcat [ppr fun <+> ppr args, ppr the_app',
+                                                        ppr actual_res_ty, ppr actual_res_ty'])
+               ; returnM (co_fn <$> the_app') }
+       }
+
+--     Note [Push result type in]
+--
+-- Unify with expected result before (was: after) type-checking the args
+-- so that the info from res_ty (was: args) percolates to args (was actual_res_ty).
+-- This is when we might detect a too-few args situation.
+-- (One can think of cases when the opposite order would give
+-- a better error message.)
+-- [March 2003: I'm experimenting with putting this first.  Here's an 
+--             example where it actually makes a real difference
+--    class C t a b | t a -> b
+--    instance C Char a Bool
+--
+--    data P t a = forall b. (C t a b) => MkP b
+--    data Q t   = MkQ (forall a. P t a)
 
+--    f1, f2 :: Q Char;
+--    f1 = MkQ (MkP True)
+--    f2 = MkQ (MkP True :: forall a. P Char a)
+--
+-- With the change, f1 will type-check, because the 'Char' info from
+-- the signature is propagated into MkQ's argument. With the check
+-- in the other order, the extra signature in f2 is reqd.]
+
+----------------
+tcFun :: LHsExpr Name -> TcM (LHsExpr TcId, [TcTyVar], TcRhoType)
+-- Instantiate the function, returning the type variables used
+-- If the function isn't simple, infer its type, and return no 
+-- type variables
+tcFun (L loc (HsVar f)) = setSrcSpan loc $ do
+                         { (fun', tvs, fun_tau) <- tcId (OccurrenceOf f) f
+                         ; return (L loc fun', tvs, fun_tau) }
+tcFun fun = do { (fun', fun_tau) <- tcInfer (tcMonoExpr fun)
+              ; return (fun', [], fun_tau) }
+
+----------------
+tcArgs :: LHsExpr Name                         -- The function (for error messages)
+       -> LHsExpr TcId                         -- The function (to build into result)
+       -> [LHsExpr Name] -> [TcSigmaType]      -- Actual arguments and expected arg types
+       -> TcM (HsExpr TcId)                    -- Resulting application
+
+tcArgs fun fun' args expected_arg_tys
+  = do         { args' <- mappM (tcArg fun) (zip3 args expected_arg_tys [1..])
+       ; return (unLoc (foldl mkHsApp fun' args')) }
+
+tcArg :: LHsExpr Name                          -- The function (for error messages)
+       -> (LHsExpr Name, TcSigmaType, Int)     -- Actual argument and expected arg type
+       -> TcM (LHsExpr TcId)                   -- Resulting argument
+tcArg fun (arg, ty, arg_no) = addErrCtxt (funAppCtxt fun arg arg_no)
+                                        (tcCheckSigma arg ty)
+
+----------------
+tcResult fun args res_ty actual_res_ty
+  = addErrCtxtM (checkArgsCtxt fun args res_ty actual_res_ty)
+               (tcSubExp res_ty actual_res_ty)
+
+----------------
 -- If an error happens we try to figure out whether the
 -- function has been given too many or too few arguments,
 -- and say so.
 -- The ~(Check...) is because in the Infer case the tcSubExp 
 -- definitely won't fail, so we can be certain we're in the Check branch
-checkArgsCtxt fun args ~(Check expected_res_ty) actual_res_ty tidy_env
+checkArgsCtxt fun args (Infer _) actual_res_ty tidy_env
+  = return (tidy_env, ptext SLIT("Urk infer"))
+
+checkArgsCtxt fun args (Check expected_res_ty) actual_res_ty tidy_env
   = zonkTcType expected_res_ty   `thenM` \ exp_ty' ->
     zonkTcType actual_res_ty     `thenM` \ act_ty' ->
     let
@@ -655,31 +738,6 @@ checkArgsCtxt fun args ~(Check expected_res_ty) actual_res_ty tidy_env
              | otherwise                   = appCtxt fun args
     in
     returnM (env2, message)
-
-
-split_fun_ty :: TcRhoType      -- The type of the function
-            -> Int             -- Number of arguments
-            -> TcM ([TcType],  -- Function argument types
-                    TcType)    -- Function result types
-
-split_fun_ty fun_ty 0 
-  = returnM ([], fun_ty)
-
-split_fun_ty fun_ty n
-  =    -- Expect the function to have type A->B
-    unifyFunTy fun_ty          `thenM` \ (arg_ty, res_ty) ->
-    split_fun_ty res_ty (n-1)  `thenM` \ (arg_tys, final_res_ty) ->
-    returnM (arg_ty:arg_tys, final_res_ty)
-\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 the_fun (arg, expected_arg_ty, arg_no)
-  = addErrCtxt (funAppCtxt the_fun arg arg_no) $
-    tcCheckSigma arg expected_arg_ty
 \end{code}
 
 
@@ -712,85 +770,107 @@ This gets a bit less sharing, but
        b) perhaps fewer separated lambdas
 
 \begin{code}
-tcId :: Name -> TcM (TcExpr, 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` \ thing ->
+tcId :: InstOrigin -> Name -> TcM (HsExpr TcId, [TcTyVar], TcRhoType)
+       -- Return the type variables at which the function
+       -- is instantiated, as well as the translated variable and its type
+
+tcId orig id_name      -- Look up the Id and instantiate its type
+  = tcLookup id_name   `thenM` \ thing ->
     case thing of {
-       AGlobal (ADataCon data_con)  -> inst_data_con data_con 
-    ;  AGlobal (AnId id)            -> loop (HsVar id) (idType id)
+       AGlobal (ADataCon con)  -- Similar, but instantiate the stupid theta too
+         -> do { (expr, tvs, tau) <- instantiate (dataConWrapId con)
+               ; tcInstStupidTheta con (mkTyVarTys tvs)
+               -- Remember to chuck in the constraints from the "silly context"
+               ; return (expr, tvs, tau) }
+
+    ;  AGlobal (AnId id) -> instantiate 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 $$ ppr thing)
+    ;  ATcId id th_level proc_level 
+         -> do { checkProcLevel id proc_level
+               ; tc_local_id id th_level }
+
+    ;  other -> failWithTc (ppr other <+> ptext SLIT("used where a value identifer was expected"))
     }
   where
 
 #ifndef GHCI
-    tc_local_id id th_bind_lvl proc_lvl                        -- Non-TH case
-       = checkProcLevel id proc_lvl    `thenM_`
-         loop (HsVar id) (idType id)
+    tc_local_id id th_bind_lvl                 -- Non-TH case
+       = instantiate id
 
 #else /* GHCI and TH is on */
-    tc_local_id id th_bind_lvl proc_lvl                        -- TH case
-       = checkProcLevel id proc_lvl    `thenM_`
-
-       -- Check for cross-stage lifting
+    tc_local_id id th_bind_lvl                 -- TH case
+       =       -- Check for cross-stage lifting
          getStage                              `thenM` \ use_stage -> 
          case use_stage of
              Brack use_lvl ps_var lie_var
                | use_lvl > th_bind_lvl 
-               ->      -- E.g. \x -> [| h x |]
-               -- We must behave as if the reference to x was
-
-               --      h $(lift x)     
-               -- We use 'x' itself as the splice proxy, used by 
-               -- the desugarer to stitch it all back together.
-               -- If 'x' occurs many times we may get many identical
-               -- bindings of the same splice proxy, but that doesn't
-               -- matter, although it's a mite untidy.
-               let
-                   id_ty = idType id
-               in
-               checkTc (isTauTy id_ty) (polySpliceErr id)      `thenM_` 
-                   -- If x is polymorphic, its occurrence sites might
-                   -- have different instantiations, so we can't use plain
-                   -- 'x' as the splice proxy name.  I don't know how to 
-                   -- solve this, and it's probably unimportant, so I'm
-                   -- just going to flag an error for now
-
-               setLIEVar lie_var       (
-               newMethodFromName orig id_ty DsMeta.liftName    `thenM` \ lift ->
-                       -- Put the 'lift' constraint into the right LIE
-       
-               -- Update the pending splices
-               readMutVar ps_var                       `thenM` \ ps ->
-               writeMutVar ps_var ((name, HsApp (HsVar lift) (HsVar id)) : ps) `thenM_`
-       
-               returnM (HsVar id, id_ty))
+               -> if isExternalName id_name then       
+                       -- Top-level identifiers in this module,
+                       -- (which have External Names)
+                       -- are just like the imported case:
+                       -- no need for the 'lifting' treatment
+                       -- E.g.  this is fine:
+                       --   f x = x
+                       --   g y = [| f 3 |]
+                       -- But we do need to put f into the keep-alive
+                       -- set, because after desugaring the code will
+                       -- only mention f's *name*, not f itself.
+                       keepAliveTc id_name     `thenM_` 
+                       instantiate id
+
+                  else -- Nested identifiers, such as 'x' in
+                       -- E.g. \x -> [| h x |]
+                       -- We must behave as if the reference to x was
+                       --      h $(lift x)     
+                       -- We use 'x' itself as the splice proxy, used by 
+                       -- the desugarer to stitch it all back together.
+                       -- If 'x' occurs many times we may get many identical
+                       -- bindings of the same splice proxy, but that doesn't
+                       -- matter, although it's a mite untidy.
+                  let
+                      id_ty = idType id
+                  in
+                  checkTc (isTauTy id_ty)      (polySpliceErr id)      `thenM_` 
+                      -- If x is polymorphic, its occurrence sites might
+                      -- have different instantiations, so we can't use plain
+                      -- 'x' as the splice proxy name.  I don't know how to 
+                      -- solve this, and it's probably unimportant, so I'm
+                      -- just going to flag an error for now
+   
+                  setLIEVar lie_var    (
+                  newMethodFromName orig id_ty DsMeta.liftName `thenM` \ lift ->
+                          -- Put the 'lift' constraint into the right LIE
+          
+                  -- Update the pending splices
+                  readMutVar ps_var                    `thenM` \ ps ->
+                  writeMutVar ps_var ((id_name, nlHsApp (nlHsVar lift) (nlHsVar id)) : ps)     `thenM_`
+          
+                  returnM (HsVar id, [], id_ty))
 
              other -> 
                checkWellStaged (quotes (ppr id)) th_bind_lvl use_stage `thenM_`
-               loop (HsVar id) (idType id)
+               instantiate id
 #endif /* GHCI */
 
-    loop (HsVar fun_id) fun_ty
+    instantiate :: TcId -> TcM (HsExpr TcId, [TcTyVar], TcRhoType)
+    instantiate fun_id = loop (HsVar fun_id) [] (idType fun_id)
+
+    loop (HsVar fun_id) tvs fun_ty
        | want_method_inst fun_ty
-       = tcInstType VanillaTv fun_ty           `thenM` \ (tyvars, theta, tau) ->
+       = tcInstType fun_ty             `thenM` \ (tyvars, theta, tau) ->
          newMethodWithGivenTy orig fun_id 
                (mkTyVarTys tyvars) theta tau   `thenM` \ meth_id ->
-         loop (HsVar meth_id) tau
+         loop (HsVar meth_id) (tvs ++ tyvars) tau
 
-    loop fun fun_ty 
+    loop fun tvs fun_ty 
        | isSigmaTy fun_ty
-       = tcInstCall orig fun_ty        `thenM` \ (inst_fn, tau) ->
-         loop (inst_fn <$> fun) tau
+       = tcInstCall orig fun_ty        `thenM` \ (inst_fn, new_tvs, tau) ->
+         loop (inst_fn <$> fun) (tvs ++ new_tvs) tau
 
        | otherwise
-       = returnM (fun, fun_ty)
+       = returnM (fun, tvs, fun_ty)
 
        --      Hack Alert (want_method_inst)!
        -- If   f :: (%x :: T) => Int -> Int
@@ -805,20 +885,6 @@ tcId name  -- Look up the Id and instantiate its type
        | otherwise           = case tcSplitSigmaTy fun_ty of
                                  (_,[],_)    -> False  -- Not overloaded
                                  (_,theta,_) -> not (any isLinearPred theta)
-
-
-       -- We treat data constructors differently, because we have to generate
-       -- constraints for their silly theta, which no longer appears in
-       -- the type of dataConWrapId (see note on "stupid context" in DataCon.lhs
-       -- It's dual to TcPat.tcConstructor
-    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), 
-                mkFunTys arg_tys result_ty)
-
-    orig = OccurrenceOf name
 \end{code}
 
 %************************************************************************
@@ -848,39 +914,39 @@ 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
+    tenv = zipTopTvSubst (tyConTyVars tycon) ty_args
 
-    do_bind (field_lbl_name, rhs)
-      = addErrCtxt (fieldCtxt field_lbl_name)  $
-           tcLookupId field_lbl_name           `thenM` \ sel_id ->
+    do_bind (L loc field_lbl, rhs)
+      = addErrCtxt (fieldCtxt field_lbl)       $
        let
-           field_lbl = recordSelectorFieldLabel sel_id
-           field_ty  = substTy tenv (fieldLabelType field_lbl)
+           field_ty  = tyConFieldType tycon field_lbl
+           field_ty' = substTy tenv field_ty
        in
+       tcCheckSigma rhs field_ty'              `thenM` \ rhs' ->
+        tcLookupId field_lbl                   `thenM` \ sel_id ->
        ASSERT( isRecordSelector sel_id )
+       returnM (L loc sel_id, rhs')
+
+tyConFieldType :: TyCon -> FieldLabel -> Type
+tyConFieldType tycon field_lbl
+  = case [ty | (f,ty,_) <- tyConFields tycon, f == field_lbl] of
+       (ty:other) -> ASSERT( null other) ty
                -- This lookup and assertion will surely succeed, because
                -- we check that the fields are indeed record selectors
                -- before calling tcRecordBinds
-       ASSERT2( fieldLabelTyCon field_lbl == tycon, ppr field_lbl )
-               -- The caller of tcRecordBinds has already checked
-               -- that all the fields come from the same type
-
-       tcCheckSigma rhs field_ty               `thenM` \ rhs' ->
-
-       returnM (sel_id, rhs')
 
 badFields rbinds data_con
   = filter (not . (`elem` field_names)) (recBindFields rbinds)
   where
-    field_names = map fieldLabelName (dataConFieldLabels data_con)
+    field_names = 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
@@ -902,12 +968,12 @@ checkMissingFields data_con rbinds
     missing_s_fields
        = [ fl | (fl, str) <- field_info,
                 isMarkedStrict str,
-                not (fieldLabelName fl `elem` field_names_used)
+                not (fl `elem` field_names_used)
          ]
     missing_ns_fields
        = [ fl | (fl, str) <- field_info,
                 not (isMarkedStrict str),
-                not (fieldLabelName fl `elem` field_names_used)
+                not (fl `elem` field_names_used)
          ]
 
     field_names_used = recBindFields rbinds
@@ -927,7 +993,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)
@@ -946,7 +1012,7 @@ tcCheckRhos (expr:exprs) (ty:tys)
 Overloaded literals.
 
 \begin{code}
-tcLit :: HsLit -> Expected TcRhoType -> TcM TcExpr
+tcLit :: HsLit -> Expected TcRhoType -> TcM (HsExpr TcId)
 tcLit lit res_ty 
   = zapExpectedTo res_ty (hsLitType lit)               `thenM_`
     returnM (HsLit lit)
@@ -973,10 +1039,6 @@ caseCtxt expr
 caseScrutCtxt expr
   = hang (ptext SLIT("In the scrutinee of a case expression:")) 4 (ppr expr)
 
-exprSigCtxt expr
-  = hang (ptext SLIT("In the type signature of the expression:"))
-        4 (ppr expr)
-
 exprCtxt expr
   = hang (ptext SLIT("In the expression:")) 4 (ppr expr)
 
@@ -1000,7 +1062,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
+    the_app = foldl mkHsApp fun args   -- Used in error messages
 
 badFieldsUpd rbinds
   = hang (ptext SLIT("No constructor has all these fields:"))
@@ -1034,7 +1096,7 @@ wrongArgsCtxt too_many_or_few fun args
                    <+> 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