[project @ 2000-09-28 13:04:14 by simonpj]
[ghc-hetmet.git] / ghc / compiler / typecheck / TcPat.lhs
index 0bf3c31..9a44d8d 100644 (file)
 %
-% (c) The GRASP/AQUA Project, Glasgow University, 1992-1995
+% (c) The GRASP/AQUA Project, Glasgow University, 1992-1998
 %
 \section[TcPat]{Typechecking patterns}
 
 \begin{code}
+module TcPat ( tcPat, tcPatBndr_NoSigs, simpleHsLitTy, badFieldCon, polyPatSig ) where
+
 #include "HsVersions.h"
 
-module TcPat (
-       tcPat
-#ifdef DPH
-       , tcPats
-#endif
-    ) where
-
-import TcMonad         -- typechecking monad machinery
-import TcMonadFns      ( newOpenTyVarTy, newPolyTyVarTy,
-                         newPolyTyVarTys, copyTyVars, newMethod,
-                         newOverloadedLit
-                       )
-import AbsSyn          -- the stuff being typechecked
-
-import AbsPrel         ( charPrimTy, intPrimTy, floatPrimTy,
-                         doublePrimTy, charTy, stringTy, mkListTy,
-                         mkTupleTy, addrTy, addrPrimTy, --OLD: eqStringId
-                         PrimOp
-                         IF_ATTACK_PRAGMAS(COMMA tagOf_PrimOp)
-                         IF_ATTACK_PRAGMAS(COMMA pprPrimOp)
-#ifdef DPH
-                         ,mkProcessorTy, toDomainId
-#endif {- Data Parallel Haskell -}
+import HsSyn           ( InPat(..), OutPat(..), HsLit(..), HsOverLit(..), HsExpr(..) )
+import RnHsSyn         ( RenamedPat )
+import TcHsSyn         ( TcPat, TcId )
+
+import TcMonad
+import Inst            ( InstOrigin(..),
+                         emptyLIE, plusLIE, LIE,
+                         newMethod, newOverloadedLit, newDicts, newClassDicts
                        )
-import AbsUniType      ( instantiateTauTy, applyTyCon, InstTyEnv(..)
-                         IF_ATTACK_PRAGMAS(COMMA instantiateTy)
+import Name            ( Name, getOccName, getSrcLoc )
+import FieldLabel      ( fieldLabelName )
+import TcEnv           ( tcLookupValue, tcLookupClassByKey,
+                         tcLookupValueByKey, newLocalId, badCon
                        )
-import CmdLineOpts     ( GlobalSwitch(..) )
-import Id              ( mkInstId, getIdUniType, getDataConSig,
-                         getInstantiatedDataConSig, Id, DataCon(..)
+import TcType          ( TcType, TcTyVar, tcInstTyVars, newTyVarTy )
+import TcMonoType      ( tcHsSigType )
+import TcUnify                 ( unifyTauTy, unifyListTy, unifyTupleTy )
+
+import CmdLineOpts     ( opt_IrrefutableTuples )
+import DataCon         ( dataConSig, dataConFieldLabels, 
+                         dataConSourceArity
                        )
-import Inst
-import E               ( lookupE_Binder, lookupE_Value,
-                         lookupE_ClassOpByKey, E,
-                         LVE(..), TCE(..), UniqFM, CE(..)
-                       -- TCE and CE for pragmas only
+import Id              ( isDataConWrapId_maybe )
+import Type            ( isTauTy, mkTyConApp, mkClassPred, boxedTypeKind )
+import Subst           ( substTy, substClasses )
+import TysPrim         ( charPrimTy, intPrimTy, floatPrimTy,
+                         doublePrimTy, addrPrimTy
                        )
-import Errors          ( dataConArityErr, Error(..), UnifyErrContext(..)
+import TysWiredIn      ( charTy, stringTy, intTy, integerTy )
+import PrelNames       ( eqClassOpKey, geClassOpKey, 
+                         cCallableClassKey, eqStringIdKey,
                        )
-import LIE             ( nullLIE, plusLIE, mkLIE, LIE )
-import Unify
-import Unique          -- some ClassKey stuff
-import Util
-
-#ifdef DPH
-import TcParQuals
-#endif {- Data Parallel Haskell -}
+import BasicTypes      ( isBoxed )
+import Bag
+import Outputable
+\end{code}
+
+
+%************************************************************************
+%*                                                                     *
+\subsection{Variable patterns}
+%*                                                                     *
+%************************************************************************
+
+\begin{code}
+-- This is the right function to pass to tcPat when there are no signatures
+tcPatBndr_NoSigs binder_name pat_ty
+  =    -- Need to make a new, monomorphic, Id
+       -- The binder_name is already being used for the polymorphic Id
+     newLocalId (getOccName binder_name) pat_ty loc    `thenNF_Tc` \ bndr_id ->
+     returnTc bndr_id
+ where
+   loc = getSrcLoc binder_name
 \end{code}
 
-The E passed in already contains bindings for all the variables in
-the pattern, usually to fresh type variables (but maybe not, if there
-were type signatures present).
+
+%************************************************************************
+%*                                                                     *
+\subsection{Typechecking patterns}
+%*                                                                     *
+%************************************************************************
 
 \begin{code}
-tcPat :: E -> RenamedPat -> TcM (TypecheckedPat, LIE, UniType)
+tcPat :: (Name -> TcType -> TcM s TcId)        -- How to construct a suitable (monomorphic)
+                                       -- Id for variables found in the pattern
+                                       -- The TcType is the expected type, see note below
+      -> RenamedPat
+
+      -> TcType                -- Expected type derived from the context
+                       --      In the case of a function with a rank-2 signature,
+                       --      this type might be a forall type.
+                       --      INVARIANT: if it is, the foralls will always be visible,
+                       --      not hidden inside a mutable type variable
+
+      -> TcM s (TcPat, 
+               LIE,                    -- Required by n+k and literal pats
+               Bag TcTyVar,    -- TyVars bound by the pattern
+                                       --      These are just the existentially-bound ones.
+                                       --      Any tyvars bound by *type signatures* in the
+                                       --      patterns are brought into scope before we begin.
+               Bag (Name, TcId),       -- Ids bound by the pattern, along with the Name under
+                                       --      which it occurs in the pattern
+                                       --      The two aren't the same because we conjure up a new
+                                       --      local name for each variable.
+               LIE)                    -- Dicts or methods [see below] bound by the pattern
+                                       --      from existential constructor patterns
 \end{code}
 
+
 %************************************************************************
 %*                                                                     *
 \subsection{Variables, wildcards, lazy pats, as-pats}
@@ -70,27 +104,36 @@ tcPat :: E -> RenamedPat -> TcM (TypecheckedPat, LIE, UniType)
 %************************************************************************
 
 \begin{code}
-tcPat e (VarPatIn name)
-  = let
-       id = lookupE_Binder e name
-    in
-    returnTc (VarPat id, nullLIE, getIdUniType id)
+tcPat tc_bndr (VarPatIn name) pat_ty
+  = tc_bndr name pat_ty                `thenTc` \ bndr_id ->
+    returnTc (VarPat bndr_id, emptyLIE, emptyBag, unitBag (name, bndr_id), emptyLIE)
 
-tcPat e (LazyPatIn pat)
-  = tcPat e pat                `thenTc` \ (pat', lie, ty) ->
-    returnTc (LazyPat pat', lie, ty)
+tcPat tc_bndr (LazyPatIn pat) pat_ty
+  = tcPat tc_bndr pat pat_ty           `thenTc` \ (pat', lie_req, tvs, ids, lie_avail) ->
+    returnTc (LazyPat pat', lie_req, tvs, ids, lie_avail)
 
-tcPat e pat_in@(AsPatIn name pat)
-  = let
-       id = lookupE_Binder e name
-    in
-    tcPat e pat                                `thenTc` \ (pat', lie, ty) ->
-    unifyTauTy (getIdUniType id) ty (PatCtxt pat_in) `thenTc_`
-    returnTc (AsPat id pat', lie, ty)
+tcPat tc_bndr pat_in@(AsPatIn name pat) pat_ty
+  = tc_bndr name pat_ty                        `thenTc` \ bndr_id ->
+    tcPat tc_bndr pat pat_ty           `thenTc` \ (pat', lie_req, tvs, ids, lie_avail) ->
+    tcAddErrCtxt (patCtxt pat_in)      $
+    returnTc (AsPat bndr_id pat', lie_req, 
+             tvs, (name, bndr_id) `consBag` ids, lie_avail)
+
+tcPat tc_bndr WildPatIn pat_ty
+  = returnTc (WildPat pat_ty, emptyLIE, emptyBag, emptyBag, emptyLIE)
+
+tcPat tc_bndr (ParPatIn parend_pat) pat_ty
+  = tcPat tc_bndr parend_pat pat_ty
+
+tcPat tc_bndr (SigPatIn pat sig) pat_ty
+  = tcHsSigType sig                                    `thenTc` \ sig_ty ->
 
-tcPat e (WildPatIn)
-  = newOpenTyVarTy    `thenNF_Tc` \ tyvar_ty ->
-    returnTc (WildPat tyvar_ty, nullLIE, tyvar_ty)
+       -- Check that the signature isn't a polymorphic one, which
+       -- we don't permit (at present, anyway)
+    checkTc (isTauTy sig_ty) (polyPatSig sig_ty)       `thenTc_`
+
+    unifyTauTy pat_ty sig_ty   `thenTc_`
+    tcPat tc_bndr pat sig_ty
 \end{code}
 
 %************************************************************************
@@ -100,290 +143,303 @@ tcPat e (WildPatIn)
 %************************************************************************
 
 \begin{code}
-tcPat e pat_in@(ListPatIn pats)
-  = tcPats e pats      `thenTc`    \ (pats', lie, tys) ->
-    newPolyTyVarTy     `thenNF_Tc` \ tyvar_ty ->
-
-    unifyTauTyList (tyvar_ty:tys) (PatCtxt pat_in) `thenTc_`
-
-    returnTc (ListPat tyvar_ty pats', lie, mkListTy tyvar_ty)
+tcPat tc_bndr pat_in@(ListPatIn pats) pat_ty
+  = tcAddErrCtxt (patCtxt pat_in)              $
+    unifyListTy pat_ty                         `thenTc` \ elem_ty ->
+    tcPats tc_bndr pats (repeat elem_ty)       `thenTc` \ (pats', lie_req, tvs, ids, lie_avail) ->
+    returnTc (ListPat elem_ty pats', lie_req, tvs, ids, lie_avail)
 
-tcPat e pat_in@(TuplePatIn pats)
-  = let
-       arity = length pats
-    in
-    tcPats e pats   `thenTc` \ (pats', lie, tys) ->
-
-       -- We have to unify with fresh polymorphic type variables, to
-       -- make sure we record that the tuples can only contain boxed
-       -- types.
-    newPolyTyVarTys arity   `thenNF_Tc` \ tyvar_tys ->
+tcPat tc_bndr pat_in@(TuplePatIn pats boxity) pat_ty
+  = tcAddErrCtxt (patCtxt pat_in)      $
 
-    unifyTauTyLists tyvar_tys tys (PatCtxt pat_in) `thenTc_`
+    unifyTupleTy boxity arity pat_ty           `thenTc` \ arg_tys ->
+    tcPats tc_bndr pats arg_tys                `thenTc` \ (pats', lie_req, tvs, ids, lie_avail) ->
 
        -- possibly do the "make all tuple-pats irrefutable" test:
-    getSwitchCheckerTc `thenNF_Tc` \ sw_chkr ->
     let
-       unmangled_result = TuplePat pats'
+       unmangled_result = TuplePat pats' boxity
 
        -- Under flag control turn a pattern (x,y,z) into ~(x,y,z)
        -- so that we can experiment with lazy tuple-matching.
        -- This is a pretty odd place to make the switch, but
        -- it was easy to do.
-       possibly_mangled_result
-         = if sw_chkr IrrefutableTuples
-           then LazyPat unmangled_result
-           else unmangled_result
 
-       -- ToDo: IrrefutableEverything
+       possibly_mangled_result
+         | opt_IrrefutableTuples && isBoxed boxity = LazyPat unmangled_result
+         | otherwise                               = unmangled_result
     in
-    returnTc (possibly_mangled_result, lie, mkTupleTy arity tys)
+    returnTc (possibly_mangled_result, lie_req, tvs, ids, lie_avail)
+  where
+    arity = length pats
 \end{code}
 
 %************************************************************************
 %*                                                                     *
 \subsection{Other constructors}
 %*                                                                     *
-%************************************************************************
 
-Constructor patterns are a little fun:
-\begin{itemize}
-\item
-typecheck the arguments
-\item
-look up the constructor
-\item
-specialise its type (ignore the translation this produces)
-\item
-check that the context produced by this specialisation is empty
-\item
-get the arguments out of the function type produced from specialising
-\item
-unify them with the types of the patterns
-\item
-back substitute with the type of the result of the constructor
-\end{itemize}
-
-ToDo: exploit new representation of constructors to make this more
-efficient?
+%************************************************************************
 
 \begin{code}
-tcPat e pat_in@(ConPatIn name pats)
-  = let
-       con_id = lookupE_Value e name
-    in
-    tcPats e pats `thenTc` \ (pats', lie, tys) ->
+tcPat tc_bndr pat@(ConPatIn name arg_pats) pat_ty
+  = tcConPat tc_bndr pat name arg_pats pat_ty
+
+tcPat tc_bndr pat@(ConOpPatIn pat1 op _ pat2) pat_ty
+  = tcConPat tc_bndr pat op [pat1, pat2] pat_ty
+\end{code}
+
 
-    matchConArgTys con_id tys (\ ty -> PatCtxt pat_in) `thenTc` \ data_ty ->
+%************************************************************************
+%*                                                                     *
+\subsection{Records}
+%*                                                                     *
+%************************************************************************
 
-    returnTc (ConPat con_id data_ty pats', lie, data_ty)
+\begin{code}
+tcPat tc_bndr pat@(RecPatIn name rpats) pat_ty
+  = tcAddErrCtxt (patCtxt pat) $
 
-tcPat e pat_in@(ConOpPatIn pat1 op pat2) -- & in binary-op form...
-  = let
-       con_id = lookupE_Value e op
+       -- Check the constructor itself
+    tcConstructor pat name pat_ty      `thenTc` \ (data_con, ex_tvs, dicts, lie_avail1, arg_tys) ->
+    let
+       -- Don't use zipEqual! If the constructor isn't really a record, then
+       -- dataConFieldLabels will be empty (and each field in the pattern
+       -- will generate an error below).
+       field_tys = zip (map fieldLabelName (dataConFieldLabels data_con))
+                       arg_tys
     in
-    tcPats e [pat1, pat2]   `thenTc`   \ ([pat1',pat2'], lie, tys) ->
-        -- ToDo: there exists a less ugly way, no doubt...
 
-    matchConArgTys con_id tys (\ ty -> PatCtxt pat_in) `thenTc` \ data_ty ->
+       -- Check the fields
+    tc_fields field_tys rpats          `thenTc` \ (rpats', lie_req, tvs, ids, lie_avail2) ->
+
+    returnTc (RecPat data_con pat_ty ex_tvs dicts rpats',
+             lie_req,
+             listToBag ex_tvs `unionBags` tvs,
+             ids,
+             lie_avail1 `plusLIE` lie_avail2)
 
-    returnTc (ConOpPat pat1' con_id pat2' data_ty, lie, data_ty)
+  where
+    tc_fields field_tys []
+      = returnTc ([], emptyLIE, emptyBag, emptyBag, emptyLIE)
+
+    tc_fields field_tys ((field_label, rhs_pat, pun_flag) : rpats)
+      =        tc_fields field_tys rpats       `thenTc` \ (rpats', lie_req1, tvs1, ids1, lie_avail1) ->
+
+       (case [ty | (f,ty) <- field_tys, f == field_label] of
+
+               -- No matching field; chances are this field label comes from some
+               -- other record type (or maybe none).  As well as reporting an
+               -- error we still want to typecheck the pattern, principally to
+               -- make sure that all the variables it binds are put into the
+               -- environment, else the type checker crashes later:
+               --      f (R { foo = (a,b) }) = a+b
+               -- If foo isn't one of R's fields, we don't want to crash when
+               -- typechecking the "a+b".
+          [] -> addErrTc (badFieldCon name field_label)        `thenNF_Tc_` 
+                newTyVarTy boxedTypeKind                       `thenNF_Tc_` 
+                returnTc (error "Bogus selector Id", pat_ty)
+
+               -- The normal case, when the field comes from the right constructor
+          (pat_ty : extras) -> 
+               ASSERT( null extras )
+               tcLookupValue field_label                       `thenNF_Tc` \ sel_id ->
+               returnTc (sel_id, pat_ty)
+       )                                                       `thenTc` \ (sel_id, pat_ty) ->
+
+       tcPat tc_bndr rhs_pat pat_ty    `thenTc` \ (rhs_pat', lie_req2, tvs2, ids2, lie_avail2) ->
+
+       returnTc ((sel_id, rhs_pat', pun_flag) : rpats',
+                 lie_req1 `plusLIE` lie_req2,
+                 tvs1 `unionBags` tvs2,
+                 ids1 `unionBags` ids2,
+                 lie_avail1 `plusLIE` lie_avail2)
 \end{code}
 
 %************************************************************************
 %*                                                                     *
-\subsection{Non-overloaded literals}
+\subsection{Literals}
 %*                                                                     *
 %************************************************************************
 
 \begin{code}
-tcPat e (LitPatIn lit@(CharLit str))
-  = returnTc (LitPat lit charTy, nullLIE, charTy)
+tcPat tc_bndr (LitPatIn lit@(HsLitLit s _)) pat_ty 
+       -- cf tcExpr on LitLits
+  = tcLookupClassByKey cCallableClassKey               `thenNF_Tc` \ cCallableClass ->
+    newDicts (LitLitOrigin (_UNPK_ s))
+            [mkClassPred cCallableClass [pat_ty]]      `thenNF_Tc` \ (dicts, _) ->
+    returnTc (LitPat (HsLitLit s pat_ty) pat_ty, dicts, emptyBag, emptyBag, emptyLIE)
+
+tcPat tc_bndr pat@(LitPatIn lit@(HsString _)) pat_ty
+  = unifyTauTy pat_ty stringTy                 `thenTc_` 
+    tcLookupValueByKey eqStringIdKey           `thenNF_Tc` \ eq_id ->
+    returnTc (NPat lit stringTy (HsVar eq_id `HsApp` HsLit lit), 
+             emptyLIE, emptyBag, emptyBag, emptyLIE)
+
+tcPat tc_bndr (LitPatIn simple_lit) pat_ty
+  = unifyTauTy pat_ty (simpleHsLitTy simple_lit)               `thenTc_` 
+    returnTc (LitPat simple_lit pat_ty, emptyLIE, emptyBag, emptyBag, emptyLIE)
+
+tcPat tc_bndr pat@(NPatIn over_lit) pat_ty
+  = newOverloadedLit (PatOrigin pat) over_lit pat_ty   `thenNF_Tc` \ (over_lit_expr, lie1) ->
+    tcLookupValueByKey eqClassOpKey                    `thenNF_Tc` \ eq_sel_id ->
+    newMethod origin eq_sel_id [pat_ty]                        `thenNF_Tc` \ (lie2, eq_id) ->
+
+    returnTc (NPat lit' pat_ty (HsApp (HsVar eq_id) over_lit_expr),
+             lie1 `plusLIE` lie2,
+             emptyBag, emptyBag, emptyLIE)
+  where
+    origin = PatOrigin pat
+    lit' = case over_lit of
+               HsIntegral i   _ -> HsInteger i
+               HsFractional f _ -> HsRat f pat_ty
+\end{code}
 
-tcPat e (LitPatIn lit@(StringLit str))
-  = getSrcLocTc                                `thenNF_Tc` \ loc ->
-    let
-       origin = LiteralOrigin lit loc
-       eq_id  = lookupE_ClassOpByKey e eqClassKey  SLIT("==")
-    in
-    newMethod origin eq_id [stringTy]  `thenNF_Tc` \ eq ->
-    let
-       comp_op = App (Var (mkInstId eq)) (Lit lit)
-    in
-    returnTc (NPat lit stringTy comp_op, mkLIE [eq], stringTy)
+%************************************************************************
+%*                                                                     *
+\subsection{n+k patterns}
+%*                                                                     *
+%************************************************************************
 
-{- OLD:
-tcPat e (LitPatIn lit@(StringLit str))
-  = returnTc (NPat lit stringTy comp_op, nullLIE, stringTy)
+\begin{code}
+tcPat tc_bndr pat@(NPlusKPatIn name lit@(HsIntegral i _) minus) pat_ty
+  = tc_bndr name pat_ty                                `thenTc` \ bndr_id ->
+    tcLookupValue minus                                `thenNF_Tc` \ minus_sel_id ->
+    tcLookupValueByKey geClassOpKey            `thenNF_Tc` \ ge_sel_id ->
+    newOverloadedLit origin lit pat_ty         `thenNF_Tc` \ (over_lit_expr, lie1) ->
+    newMethod origin ge_sel_id    [pat_ty]     `thenNF_Tc` \ (lie2, ge_id) ->
+    newMethod origin minus_sel_id [pat_ty]     `thenNF_Tc` \ (lie3, minus_id) ->
+
+    returnTc (NPlusKPat bndr_id i pat_ty
+                       (SectionR (HsVar ge_id) over_lit_expr)
+                       (SectionR (HsVar minus_id) over_lit_expr),
+             lie1 `plusLIE` lie2 `plusLIE` lie3,
+             emptyBag, unitBag (name, bndr_id), emptyLIE)
   where
-    comp_op   = App (Var eqStringId) (Lit lit)
--}
-
-tcPat e (LitPatIn lit@(IntPrimLit _))
-  = returnTc (LitPat lit intPrimTy, nullLIE, intPrimTy)
-tcPat e (LitPatIn lit@(CharPrimLit _))
-  = returnTc (LitPat lit charPrimTy, nullLIE, charPrimTy)
-tcPat e (LitPatIn lit@(StringPrimLit _))
-  = returnTc (LitPat lit addrPrimTy, nullLIE, addrPrimTy)
-tcPat e (LitPatIn lit@(FloatPrimLit _))
-  = returnTc (LitPat lit floatPrimTy, nullLIE, floatPrimTy)
-tcPat e (LitPatIn lit@(DoublePrimLit _))
-  = returnTc (LitPat lit doublePrimTy, nullLIE, doublePrimTy)
+    origin = PatOrigin pat
 \end{code}
 
 %************************************************************************
 %*                                                                     *
-\subsection{Overloaded patterns: int literals and \tr{n+k} patterns}
+\subsection{Lists of patterns}
 %*                                                                     *
 %************************************************************************
 
+Helper functions
+
 \begin{code}
-tcPat e (LitPatIn lit@(IntLit i))
-  = getSrcLocTc                                `thenNF_Tc` \ loc ->
-    let
-       origin = LiteralOrigin lit loc
-    in
-    newPolyTyVarTy                     `thenNF_Tc` \ tyvar_ty ->
-    let
-       from_int     = lookupE_ClassOpByKey e numClassKey SLIT("fromInt")
-       from_integer = lookupE_ClassOpByKey e numClassKey SLIT("fromInteger")
-       eq_id        = lookupE_ClassOpByKey e eqClassKey  SLIT("==")
-    in
-    newOverloadedLit origin
-                    (OverloadedIntegral i from_int from_integer)
-                    tyvar_ty           `thenNF_Tc` \ over_lit ->
+tcPats :: (Name -> TcType -> TcM s TcId)       -- How to deal with variables
+       -> [RenamedPat] -> [TcType]             -- Excess 'expected types' discarded
+       -> TcM s ([TcPat], 
+                LIE,                           -- Required by n+k and literal pats
+                Bag TcTyVar,
+                Bag (Name, TcId),      -- Ids bound by the pattern
+                LIE)                           -- Dicts bound by the pattern
+
+tcPats tc_bndr [] tys = returnTc ([], emptyLIE, emptyBag, emptyBag, emptyLIE)
+
+tcPats tc_bndr (ty:tys) (pat:pats)
+  = tcPat tc_bndr ty pat               `thenTc` \ (pat',  lie_req1, tvs1, ids1, lie_avail1) ->
+    tcPats tc_bndr tys pats    `thenTc` \ (pats', lie_req2, tvs2, ids2, lie_avail2) ->
+
+    returnTc (pat':pats', lie_req1 `plusLIE` lie_req2,
+             tvs1 `unionBags` tvs2, ids1 `unionBags` ids2, 
+             lie_avail1 `plusLIE` lie_avail2)
+\end{code}
 
-    newMethod origin eq_id [tyvar_ty]  `thenNF_Tc` \ eq ->
+------------------------------------------------------
+\begin{code}
+simpleHsLitTy :: HsLit -> TcType
+simpleHsLitTy (HsCharPrim c)   = charPrimTy
+simpleHsLitTy (HsStringPrim s) = addrPrimTy
+simpleHsLitTy (HsInt i)               = intTy
+simpleHsLitTy (HsInteger i)    = integerTy
+simpleHsLitTy (HsIntPrim i)    = intPrimTy
+simpleHsLitTy (HsFloatPrim f)  = floatPrimTy
+simpleHsLitTy (HsDoublePrim d) = doublePrimTy
+simpleHsLitTy (HsChar c)       = charTy
+simpleHsLitTy (HsString str)   = stringTy
+\end{code}
 
-    returnTc (NPat lit tyvar_ty (App (Var (mkInstId eq))
-                                    (Var (mkInstId over_lit))),
-             mkLIE [over_lit, eq],
-             tyvar_ty)
 
-tcPat e (LitPatIn lit@(FracLit f))
-  = getSrcLocTc                                `thenNF_Tc` \ loc ->
-    let
-       origin = LiteralOrigin lit loc
+------------------------------------------------------
+\begin{code}
+tcConstructor pat con_name pat_ty
+  =    -- Check that it's a constructor
+    tcLookupValue con_name             `thenNF_Tc` \ con_id ->
+    case isDataConWrapId_maybe con_id of {
+       Nothing -> failWithTc (badCon con_id);
+       Just data_con ->
+
+       -- Instantiate it
+    let 
+       (tvs, _, ex_tvs, ex_theta, arg_tys, tycon) = dataConSig data_con
+            -- Ignore the theta; overloaded constructors only
+            -- behave differently when called, not when used for
+            -- matching.
     in
-    newPolyTyVarTy                     `thenNF_Tc` \ tyvar_ty ->
+    tcInstTyVars (ex_tvs ++ tvs)       `thenNF_Tc` \ (all_tvs', ty_args', tenv) ->
     let
-       eq_id         = lookupE_ClassOpByKey e eqClassKey         SLIT("==")
-       from_rational = lookupE_ClassOpByKey e fractionalClassKey SLIT("fromRational")
-    in
-    newOverloadedLit origin
-                    (OverloadedFractional f from_rational)
-                    tyvar_ty           `thenNF_Tc` \ over_lit ->
+       ex_theta' = substClasses tenv ex_theta
+       arg_tys'  = map (substTy tenv) arg_tys
 
-    newMethod origin eq_id [tyvar_ty]  `thenNF_Tc` \ eq ->
+       n_ex_tvs  = length ex_tvs
+       ex_tvs'   = take n_ex_tvs all_tvs'
+       result_ty = mkTyConApp tycon (drop n_ex_tvs ty_args')
+    in
+    newClassDicts (PatOrigin pat) ex_theta'    `thenNF_Tc` \ (lie_avail, dicts) ->
 
-    returnTc (NPat lit tyvar_ty (App (Var (mkInstId eq))
-                                    (Var (mkInstId over_lit))),
-             mkLIE [over_lit, eq],
-             tyvar_ty)
+       -- Check overall type matches
+    unifyTauTy pat_ty result_ty                `thenTc_`
 
-tcPat e (LitPatIn lit@(LitLitLitIn s))
-  = error "tcPat: can't handle ``literal-literal'' patterns"
-{-
-  = getSrcLocTc                                `thenNF_Tc` \ loc ->
-    let
-       origin = LiteralOrigin lit loc
-    in
-    newPolyTyVarTy                     `thenNF_Tc` \ tyvar_ty ->
-    let
-       eq_id = lookupE_ClassOpByKey e eqClassKey "=="
-    in
-    newOverloadedLit origin
-                    (OverloadedLitLit s)
-                    tyvar_ty           `thenNF_Tc` \ over_lit ->
+    returnTc (data_con, ex_tvs', dicts, lie_avail, arg_tys')
+    }
+\end{code}           
 
-    newMethod origin eq_id [tyvar_ty]  `thenNF_Tc` \ eq ->
+------------------------------------------------------
+\begin{code}
+tcConPat tc_bndr pat con_name arg_pats pat_ty
+  = tcAddErrCtxt (patCtxt pat) $
 
-    returnTc (NPat lit tyvar_ty (App (Var (mkInstId eq))
-                                    (Var (mkInstId over_lit))),
-             mkLIE [over_lit, eq],
-             tyvar_ty)
--}
+       -- Check the constructor itself
+    tcConstructor pat con_name pat_ty  `thenTc` \ (data_con, ex_tvs', dicts, lie_avail1, arg_tys') ->
 
-tcPat e (NPlusKPatIn name lit@(IntLit k))
-  = getSrcLocTc                                `thenNF_Tc` \ loc ->
+       -- Check correct arity
     let
-       origin   = LiteralOrigin lit loc
+       con_arity  = dataConSourceArity data_con
+       no_of_args = length arg_pats
+    in
+    checkTc (con_arity == no_of_args)
+           (arityErr "Constructor" data_con con_arity no_of_args)      `thenTc_`
 
-       local    = lookupE_Binder e name
-       local_ty = getIdUniType local
+       -- Check arguments
+    tcPats tc_bndr arg_pats arg_tys'   `thenTc` \ (arg_pats', lie_req, tvs, ids, lie_avail2) ->
 
-       ge_id        = lookupE_ClassOpByKey e ordClassKey SLIT(">=")
-       minus_id     = lookupE_ClassOpByKey e numClassKey SLIT("-")
-       from_int     = lookupE_ClassOpByKey e numClassKey SLIT("fromInt")
-       from_integer = lookupE_ClassOpByKey e numClassKey SLIT("fromInteger")
-    in
-    newOverloadedLit origin
-                    (OverloadedIntegral k from_int from_integer)
-                    local_ty              `thenNF_Tc` \ over_lit ->
-
-    newMethod origin ge_id     [local_ty] `thenNF_Tc` \ ge ->
-    newMethod origin minus_id  [local_ty] `thenNF_Tc` \ minus ->
-
-    returnTc (NPlusKPat local lit local_ty
-                       (Var (mkInstId over_lit))
-                       (Var (mkInstId ge))
-                       (Var (mkInstId minus)),
-             mkLIE [over_lit, ge, minus],
-             local_ty)
-
-tcPat e (NPlusKPatIn pat other) = panic "TcPat:NPlusKPat: not an IntLit"
-
-#ifdef DPH
-tcPat e (ProcessorPatIn pats pat)
-  = tcPidPats e pats           `thenTc` \ (pats',convs, lie, tys)->
-    tcPat e pat                `thenTc` \ (pat', ty, lie') ->
-    returnTc (ProcessorPat pats' convs pat',
-             plusLIE lie lie',
-             mkProcessorTy tys ty)
-#endif {- Data Parallel Haskell -}
+    returnTc (ConPat data_con pat_ty ex_tvs' dicts arg_pats',
+             lie_req,
+             listToBag ex_tvs' `unionBags` tvs,
+             ids,
+             lie_avail1 `plusLIE` lie_avail2)
 \end{code}
 
+
 %************************************************************************
 %*                                                                     *
-\subsection{Lists of patterns}
+\subsection{Errors and contexts}
 %*                                                                     *
 %************************************************************************
 
 \begin{code}
-tcPats :: E -> [RenamedPat] -> TcM ([TypecheckedPat], LIE, [UniType])
-
-tcPats e [] = returnTc ([], nullLIE, [])
-
-tcPats e (pat:pats)
-  = tcPat e pat                        `thenTc` \ (pat',  lie,  ty)  ->
-    tcPats e pats              `thenTc` \ (pats', lie', tys) ->
-
-    returnTc (pat':pats', plusLIE lie lie', ty:tys)
+patCtxt pat = hang (ptext SLIT("In the pattern:")) 
+                4 (ppr pat)
+
+badFieldCon :: Name -> Name -> SDoc
+badFieldCon con field
+  = hsep [ptext SLIT("Constructor") <+> quotes (ppr con),
+         ptext SLIT("does not have field"), quotes (ppr field)]
+
+polyPatSig :: TcType -> SDoc
+polyPatSig sig_ty
+  = hang (ptext SLIT("Illegal polymorphic type signature in pattern:"))
+        4 (ppr sig_ty)
 \end{code}
 
-@matchConArgTys@ grabs the signature of the data constructor, and
-unifies the actual args against the expected ones.
-
-\begin{code}
-matchConArgTys :: Id -> [UniType] -> (UniType -> UnifyErrContext) -> TcM UniType
-
-matchConArgTys con_id arg_tys err_ctxt
-  = let
-       no_of_args = length arg_tys
-       (sig_tyvars, sig_theta, sig_tys, _) = getDataConSig con_id
-            -- Ignore the sig_theta; overloaded constructors only
-            -- behave differently when called, not when used for
-            -- matching.
-       con_arity  = length sig_tys
-    in
-    getSrcLocTc                                `thenNF_Tc` \ loc ->
-    checkTc (con_arity /= no_of_args) 
-           (dataConArityErr con_id con_arity no_of_args loc) `thenTc_`
-
-    copyTyVars sig_tyvars              `thenNF_Tc` \ (inst_env, _, new_tyvar_tys) ->
-    let 
-       (_,inst_arg_tys,inst_result_ty) = getInstantiatedDataConSig con_id new_tyvar_tys
-    in
-    unifyTauTyLists arg_tys inst_arg_tys (err_ctxt inst_result_ty)  `thenTc_`
-    returnTc inst_result_ty
-\end{code}