[project @ 2002-04-02 13:21:36 by simonpj]
[ghc-hetmet.git] / ghc / compiler / typecheck / Inst.lhs
index f00e8a1..6b25d8a 100644 (file)
@@ -8,86 +8,73 @@ module Inst (
        LIE, emptyLIE, unitLIE, plusLIE, consLIE, zonkLIE,
        plusLIEs, mkLIE, isEmptyLIE, lieToList, listToLIE,
 
-       Inst, OverloadedLit(..),
-       pprInst, pprInsts, pprInstsInFull, tidyInst, tidyInsts,
+       Inst, 
+       pprInst, pprInsts, pprInstsInFull, tidyInsts, tidyMoreInsts,
 
-        InstanceMapper,
+       newDictsFromOld, newDicts, cloneDict,
+       newMethod, newMethodWithGivenTy, newMethodAtLoc,
+       newOverloadedLit, newIPDict, tcInstCall, tcInstDataCon,
 
-       newDictFromOld, newDicts, newClassDicts, newDictsAtLoc,
-       newMethod, newMethodWithGivenTy, newOverloadedLit,
-       newIPDict, instOverloadedFun,
-
-       tyVarsOfInst, tyVarsOfInsts, tyVarsOfLIE, instLoc, getDictClassTys,
-       getDictPred_maybe, getMethodTheta_maybe,
-       getFunDeps, getFunDepsOfLIE,
-       getIPs, getIPsOfLIE,
-       getAllFunDeps, getAllFunDepsOfLIE,
+       tyVarsOfInst, tyVarsOfInsts, tyVarsOfLIE, 
+       ipNamesOfInst, ipNamesOfInsts, predsOfInst, predsOfInsts,
+       instLoc, getDictClassTys, dictPred,
 
        lookupInst, lookupSimpleInst, LookupInstResult(..),
 
-       isDict, isClassDict, isMethod,
-       isTyVarDict, isStdClassTyVarDict, isMethodFor, notFunDep,
+       isDict, isClassDict, isMethod, 
+       isLinearInst, linearInstType,
+       isTyVarDict, isStdClassTyVarDict, isMethodFor, 
        instBindingRequired, instCanBeGeneralised,
 
-       zonkInst, zonkInsts, zonkFunDeps, zonkTvFunDeps,
-       instToId, instToIdBndr, ipToId,
+       zonkInst, zonkInsts,
+       instToId, instName,
 
        InstOrigin(..), InstLoc, pprInstLoc
     ) where
 
 #include "HsVersions.h"
 
-import HsSyn   ( HsLit(..), HsExpr(..) )
-import RnHsSyn ( RenamedArithSeqInfo, RenamedHsExpr, RenamedPat )
-import TcHsSyn ( TcExpr, TcId, 
+import HsSyn   ( HsLit(..), HsOverLit(..), HsExpr(..) )
+import TcHsSyn ( TcExpr, TcId, TypecheckedHsExpr,
                  mkHsTyApp, mkHsDictApp, mkHsConApp, zonkId
                )
 import TcMonad
-import TcEnv   ( TcIdSet, tcLookupValueByKey, tcLookupTyConByKey )
-import TcType  ( TcThetaType,
-                 TcType, TcTauType, TcTyVarSet,
-                 zonkTcTyVars, zonkTcType, zonkTcTypes, 
-                 zonkTcThetaType
+import TcEnv   ( TcIdSet, tcGetInstEnv, tcLookupId )
+import InstEnv ( InstLookupResult(..), lookupInstEnv )
+import TcMType ( zonkTcType, zonkTcTypes, zonkTcPredType, zapToType,
+                 zonkTcThetaType, tcInstTyVar, tcInstType, tcInstTyVars
                )
-import Bag
-import Class   ( classInstEnv, Class )
-import FunDeps ( instantiateFdClassTys )
-import Id      ( Id, idFreeTyVars, idType, mkUserLocal, mkSysLocal )
-import PrelInfo        ( isStandardClass, isCcallishClass, isNoDictClass )
-import Name    ( OccName, Name, mkDictOcc, mkMethodOcc, mkIPOcc,
-                 getOccName, nameUnique )
-import PprType ( pprPred )     
-import InstEnv ( InstEnv, lookupInstEnv )
-import SrcLoc  ( SrcLoc )
-import Type    ( Type, PredType(..), ThetaType,
-                 mkTyVarTy, isTyVarTy, mkDictTy, mkPredTy,
-                 splitForAllTys, splitSigmaTy,
-                 splitRhoTy, tyVarsOfType, tyVarsOfTypes, tyVarsOfPred,
-                 mkSynTy, tidyOpenType, tidyOpenTypes
+import TcType  ( Type, TcType, TcThetaType, TcPredType, TcTauType, TcTyVarSet,
+                 SourceType(..), PredType, ThetaType, TyVarDetails(VanillaTv),
+                 tcSplitForAllTys, tcSplitForAllTys, mkTyConApp,
+                 tcSplitMethodTy, tcSplitPhiTy, tcFunArgTy,
+                 isIntTy,isFloatTy, isIntegerTy, isDoubleTy,
+                 tcIsTyVarTy, mkPredTy, mkTyVarTy, mkTyVarTys,
+                 tyVarsOfType, tyVarsOfTypes, tyVarsOfPred, tidyPred,
+                 isClassPred, isTyVarClassPred, isLinearPred,
+                 getClassPredTys, getClassPredTys_maybe, mkPredName,
+                 tidyType, tidyTypes, tidyFreeTyVars,
+                 tcCmpType, tcCmpTypes, tcCmpPred
                )
-import InstEnv ( InstEnv )
-import Subst   ( emptyInScopeSet, mkSubst,
-                 substTy, substClasses, mkTyVarSubst, mkTopTyVarSubst
+import CoreFVs ( idFreeTyVars )
+import Class   ( Class )
+import DataCon ( dataConSig )
+import Id      ( Id, idName, idType, mkUserLocal, mkSysLocal, mkLocalId, setIdUnique )
+import PrelInfo        ( isStandardClass, isCcallishClass, isNoDictClass )
+import Name    ( Name, mkMethodOcc, getOccName )
+import PprType ( pprPred, pprParendType )      
+import Subst   ( emptyInScopeSet, mkSubst, 
+                 substTy, substTyWith, substTheta, mkTyVarSubst, mkTopTyVarSubst
                )
-import TyCon   ( TyCon )
 import Literal ( inIntRange )
-import Var     ( TyVar )
-import VarEnv  ( lookupVarEnv, TidyEnv,
-                 lookupSubstEnv, SubstResult(..)
-               )
+import VarEnv  ( TidyEnv, lookupSubstEnv, SubstResult(..) )
 import VarSet  ( elemVarSet, emptyVarSet, unionVarSet )
-import TysPrim   ( intPrimTy, floatPrimTy, doublePrimTy )
-import TysWiredIn ( intDataCon, isIntTy,
-                   floatDataCon, isFloatTy,
-                   doubleDataCon, isDoubleTy,
-                   integerTy, isIntegerTy
-                 ) 
-import Unique  ( fromRationalClassOpKey, rationalTyConKey,
-                 fromIntClassOpKey, fromIntegerClassOpKey, Unique
-               )
-import Maybes  ( expectJust )
-import Maybe   ( catMaybes )
-import Util    ( thenCmp, zipWithEqual, mapAccumL )
+import TysWiredIn ( floatDataCon, doubleDataCon )
+import PrelNames( fromIntegerName, fromRationalName )
+import Util    ( thenCmp, equalLength )
+import BasicTypes( IPName(..), mapIPName, ipNameName )
+
+import Bag
 import Outputable
 \end{code}
 
@@ -110,11 +97,11 @@ plusLIEs lies        = unionManyBags lies
 lieToList        = bagToList
 listToLIE        = listToBag
 
-zonkLIE :: LIE -> NF_TcM s LIE
+zonkLIE :: LIE -> NF_TcM LIE
 zonkLIE lie = mapBagNF_Tc zonkInst lie
 
 pprInsts :: [Inst] -> SDoc
-pprInsts insts = parens (sep (punctuate comma (map pprInst insts)))
+pprInsts insts  = parens (sep (punctuate comma (map pprInst insts)))
 
 
 pprInstsInFull insts
@@ -140,12 +127,12 @@ type Int, represented by
 \begin{code}
 data Inst
   = Dict
-       Unique
+       Id
        TcPredType
        InstLoc
 
   | Method
-       Unique
+       Id
 
        TcId    -- The overloaded function
                        -- This function will be a global, local, or ClassOpId;
@@ -169,19 +156,10 @@ data Inst
        --      type of (f tys dicts(from theta)) = tau
 
   | LitInst
-       Unique
-       OverloadedLit
+       Id
+       HsOverLit       -- The literal from the occurrence site
        TcType          -- The type at which the literal is used
        InstLoc
-
-  | FunDep
-       Class           -- the class from which this arises
-       [([TcType], [TcType])]
-       InstLoc
-
-data OverloadedLit
-  = OverloadedIntegral  Integer        -- The number
-  | OverloadedFractional Rational      -- The number
 \end{code}
 
 Ordering
@@ -193,109 +171,77 @@ maps to do their stuff.
 \begin{code}
 instance Ord Inst where
   compare = cmpInst
-instance Ord PredType where
-  compare = cmpPred
 
 instance Eq Inst where
   (==) i1 i2 = case i1 `cmpInst` i2 of
                 EQ    -> True
                 other -> False
-instance Eq PredType where
-  (==) p1 p2 = case p1 `cmpPred` p2 of
-                EQ    -> True
-                other -> False
 
-cmpInst  (Dict _ pred1 _) (Dict _ pred2 _)
-  = (pred1 `cmpPred` pred2)
-cmpInst (Dict _ _ _) other
-  = LT
-
-cmpInst (Method _ _ _ _ _ _) (Dict _ _ _)
-  = GT
-cmpInst (Method _ id1 tys1 _ _ _) (Method _ id2 tys2 _ _ _)
-  = (id1 `compare` id2) `thenCmp` (tys1 `compare` tys2)
-cmpInst (Method _ _ _ _ _ _) other
-  = LT
-
-cmpInst (LitInst _ lit1 ty1 _) (LitInst _ lit2 ty2 _)
-  = (lit1 `cmpOverLit` lit2) `thenCmp` (ty1 `compare` ty2)
-cmpInst (LitInst _ _ _ _) (FunDep _ _ _)
-  = LT
-cmpInst (LitInst _ _ _ _) other
-  = GT
-
-cmpInst (FunDep clas1 fds1 _) (FunDep clas2 fds2 _)
-  = (clas1 `compare` clas2) `thenCmp` (fds1 `compare` fds2)
-cmpInst (FunDep _ _ _) other
-  = GT
-
-cmpPred (Class c1 tys1) (Class c2 tys2)
-  = (c1 `compare` c2) `thenCmp` (tys1 `compare` tys2)
-cmpPred (IParam n1 ty1) (IParam n2 ty2)
-  = (n1 `compare` n2) `thenCmp` (ty1 `compare` ty2)
-cmpPred (Class _ _) (IParam _ _) = LT
-cmpPred _           _            = GT
-
-cmpOverLit (OverloadedIntegral   i1) (OverloadedIntegral   i2) = i1 `compare` i2
-cmpOverLit (OverloadedFractional f1) (OverloadedFractional f2) = f1 `compare` f2
-cmpOverLit (OverloadedIntegral _)    (OverloadedFractional _)  = LT
-cmpOverLit (OverloadedFractional _)  (OverloadedIntegral _)    = GT
+cmpInst (Dict _ pred1 _)         (Dict _ pred2 _)          = pred1 `tcCmpPred` pred2
+cmpInst (Dict _ _ _)             other                     = LT
+
+cmpInst (Method _ _ _ _ _ _)     (Dict _ _ _)              = GT
+cmpInst (Method _ id1 tys1 _ _ _) (Method _ id2 tys2 _ _ _) = (id1 `compare` id2) `thenCmp` (tys1 `tcCmpTypes` tys2)
+cmpInst (Method _ _ _ _ _ _)      other                            = LT
+
+cmpInst (LitInst _ lit1 ty1 _)   (LitInst _ lit2 ty2 _)    = (lit1 `compare` lit2) `thenCmp` (ty1 `tcCmpType` ty2)
+cmpInst (LitInst _ _ _ _)        other                     = GT
+
+-- and they can only have HsInt or HsFracs in them.
 \end{code}
 
 
 Selection
 ~~~~~~~~~
 \begin{code}
-instLoc (Dict   u pred      loc) = loc
-instLoc (Method u _ _ _ _   loc) = loc
-instLoc (LitInst u lit ty   loc) = loc
-instLoc (FunDep _ _        loc) = loc
-
-getDictPred_maybe (Dict _ p _) = Just p
-getDictPred_maybe _           = Nothing
+instName :: Inst -> Name
+instName inst = idName (instToId inst)
 
-getMethodTheta_maybe (Method _ _ _ theta _ _) = Just theta
-getMethodTheta_maybe _                       = Nothing
+instToId :: Inst -> TcId
+instToId (Dict id _ _)        = id
+instToId (Method id _ _ _ _ _) = id
+instToId (LitInst id _ _ _)    = id
 
-getDictClassTys (Dict u (Class clas tys) _) = (clas, tys)
+instLoc (Dict _ _         loc) = loc
+instLoc (Method _ _ _ _ _ loc) = loc
+instLoc (LitInst _ _ _    loc) = loc
 
-getFunDeps (FunDep clas fds _) = Just (clas, fds)
-getFunDeps _ = Nothing
+dictPred (Dict _ pred _ ) = pred
+dictPred inst            = pprPanic "dictPred" (ppr inst)
 
-getFunDepsOfLIE lie = catMaybes (map getFunDeps (lieToList lie))
+getDictClassTys (Dict _ pred _) = getClassPredTys pred
 
-getIPsOfPred (IParam n ty) = [(n, ty)]
-getIPsOfPred _             = []
-getIPsOfTheta theta = concatMap getIPsOfPred theta
+predsOfInsts :: [Inst] -> [PredType]
+predsOfInsts insts = concatMap predsOfInst insts
 
-getIPs (Dict u (IParam n ty) loc) = [(n, ty)]
-getIPs (Method u id _ theta t loc) = getIPsOfTheta theta
-getIPs _ = []
+predsOfInst (Dict _ pred _)          = [pred]
+predsOfInst (Method _ _ _ theta _ _) = theta
+predsOfInst (LitInst _ _ _ _)       = []
+       -- The last case is is really a big cheat
+       -- LitInsts to give rise to a (Num a) or (Fractional a) predicate
+       -- But Num and Fractional have only one parameter and no functional
+       -- dependencies, so I think no caller of predsOfInst will care.
 
-getIPsOfLIE lie = concatMap getIPs (lieToList lie)
+ipNamesOfInsts :: [Inst] -> [Name]
+ipNamesOfInst  :: Inst   -> [Name]
+-- Get the implicit parameters mentioned by these Insts
+-- NB: ?x and %x get different Names
 
-getAllFunDeps (FunDep clas fds _) = fds
-getAllFunDeps inst = map (\(n,ty) -> ([], [ty])) (getIPs inst)
+ipNamesOfInsts insts = [n | inst <- insts, n <- ipNamesOfInst inst]
 
-getAllFunDepsOfLIE lie = concat (map getAllFunDeps (lieToList lie))
+ipNamesOfInst (Dict _ (IParam n _) _)  = [ipNameName n]
+ipNamesOfInst (Method _ _ _ theta _ _) = [ipNameName n | IParam n _ <- theta]
+ipNamesOfInst other                   = []
 
 tyVarsOfInst :: Inst -> TcTyVarSet
+tyVarsOfInst (LitInst _ _ ty _)      = tyVarsOfType  ty
 tyVarsOfInst (Dict _ pred _)         = tyVarsOfPred pred
 tyVarsOfInst (Method _ id tys _ _ _) = tyVarsOfTypes tys `unionVarSet` idFreeTyVars id
                                         -- The id might have free type variables; in the case of
                                         -- locally-overloaded class methods, for example
-tyVarsOfInst (LitInst _ _ ty _)      = tyVarsOfType  ty
-tyVarsOfInst (FunDep _ fds _)
-  = foldr unionVarSet emptyVarSet (map tyVarsOfFd fds)
-  where tyVarsOfFd (ts1, ts2) =
-           tyVarsOfTypes ts1 `unionVarSet` tyVarsOfTypes ts2
-
-tyVarsOfInsts insts
-  = foldr unionVarSet emptyVarSet (map tyVarsOfInst insts)
 
-tyVarsOfLIE lie
-  = foldr unionVarSet emptyVarSet (map tyVarsOfInst insts)
-  where insts = lieToList lie
+tyVarsOfInsts insts = foldr (unionVarSet . tyVarsOfInst) emptyVarSet insts
+tyVarsOfLIE   lie   = tyVarsOfInsts (lieToList lie)
 \end{code}
 
 Predicates
@@ -304,32 +250,38 @@ Predicates
 isDict :: Inst -> Bool
 isDict (Dict _ _ _) = True
 isDict other       = False
+
 isClassDict :: Inst -> Bool
-isClassDict (Dict _ (Class _ _) _) = True
-isClassDict other                 = False
+isClassDict (Dict _ pred _) = isClassPred pred
+isClassDict other          = False
+
+isTyVarDict :: Inst -> Bool
+isTyVarDict (Dict _ pred _) = isTyVarClassPred pred
+isTyVarDict other          = False
 
 isMethod :: Inst -> Bool
 isMethod (Method _ _ _ _ _ _) = True
 isMethod other               = False
 
 isMethodFor :: TcIdSet -> Inst -> Bool
-isMethodFor ids (Method uniq id tys _ _ loc) 
-  = id `elemVarSet` ids
-isMethodFor ids inst 
-  = False
+isMethodFor ids (Method uniq id tys _ _ loc) = id `elemVarSet` ids
+isMethodFor ids inst                        = False
 
-isTyVarDict :: Inst -> Bool
-isTyVarDict (Dict _ (Class _ tys) _) = all isTyVarTy tys
-isTyVarDict other                   = False
+isLinearInst :: Inst -> Bool
+isLinearInst (Dict _ pred _) = isLinearPred pred
+isLinearInst other          = False
+       -- We never build Method Insts that have
+       -- linear implicit paramters in them.
+       -- Hence no need to look for Methods
+       -- See TcExpr.tcId 
 
-isStdClassTyVarDict (Dict _ (Class clas [ty]) _)
-  = isStandardClass clas && isTyVarTy ty
-isStdClassTyVarDict other
-  = False
+linearInstType :: Inst -> TcType       -- %x::t  -->  t
+linearInstType (Dict _ (IParam _ ty) _) = ty
 
-notFunDep :: Inst -> Bool
-notFunDep (FunDep _ _ _) = False
-notFunDep other                 = True
+
+isStdClassTyVarDict (Dict _ pred _) = case getClassPredTys_maybe pred of
+                                       Just (clas, [ty]) -> isStandardClass clas && tcIsTyVarTy ty
+                                       other             -> False
 \end{code}
 
 Two predicates which deal with the case where class constraints don't
@@ -339,102 +291,143 @@ must be witnessed by an actual binding; the second tells whether an
 
 \begin{code}
 instBindingRequired :: Inst -> Bool
-instBindingRequired (Dict _ (Class clas _) _) = not (isNoDictClass clas)
-instBindingRequired (Dict _ (IParam _ _) _)   = False
-instBindingRequired other                    = True
+instBindingRequired (Dict _ (ClassP clas _) _) = not (isNoDictClass clas)
+instBindingRequired other                     = True
 
 instCanBeGeneralised :: Inst -> Bool
-instCanBeGeneralised (Dict _ (Class clas _) _) = not (isCcallishClass clas)
-instCanBeGeneralised other                    = True
+instCanBeGeneralised (Dict _ (ClassP clas _) _) = not (isCcallishClass clas)
+instCanBeGeneralised other                     = True
 \end{code}
 
 
-Construction
-~~~~~~~~~~~~
+%************************************************************************
+%*                                                                     *
+\subsection{Building dictionaries}
+%*                                                                     *
+%************************************************************************
 
 \begin{code}
 newDicts :: InstOrigin
         -> TcThetaType
-        -> NF_TcM s (LIE, [TcId])
+        -> NF_TcM [Inst]
 newDicts orig theta
   = tcGetInstLoc orig          `thenNF_Tc` \ loc ->
-    newDictsAtLoc loc theta    `thenNF_Tc` \ (dicts, ids) ->
-    returnNF_Tc (listToBag dicts, ids)
+    newDictsAtLoc loc theta
 
-newClassDicts :: InstOrigin
-             -> [(Class,[TcType])]
-             -> NF_TcM s (LIE, [TcId])
-newClassDicts orig theta
-  = newDicts orig (map (uncurry Class) theta)
+cloneDict :: Inst -> NF_TcM Inst
+cloneDict (Dict id ty loc) = tcGetUnique       `thenNF_Tc` \ uniq ->
+                            returnNF_Tc (Dict (setIdUnique id uniq) ty loc)
+
+newDictsFromOld :: Inst -> TcThetaType -> NF_TcM [Inst]
+newDictsFromOld (Dict _ _ loc) theta = newDictsAtLoc loc theta
 
 -- Local function, similar to newDicts, 
 -- but with slightly different interface
 newDictsAtLoc :: InstLoc
              -> TcThetaType
-             -> NF_TcM s ([Inst], [TcId])
-newDictsAtLoc loc theta =
- tcGetUniques (length theta)           `thenNF_Tc` \ new_uniqs ->
- let
-  mk_dict u pred = Dict u pred loc
-  dicts = zipWithEqual "newDictsAtLoc" mk_dict new_uniqs theta
- in
- returnNF_Tc (dicts, map instToId dicts)
+             -> NF_TcM [Inst]
+newDictsAtLoc inst_loc@(_,loc,_) theta
+  = tcGetUniques                       `thenNF_Tc` \ new_uniqs ->
+    returnNF_Tc (zipWith mk_dict new_uniqs theta)
+  where
+    mk_dict uniq pred = Dict (mkLocalId (mkPredName uniq loc pred) (mkPredTy pred)) pred inst_loc
+
+-- For vanilla implicit parameters, there is only one in scope
+-- at any time, so we used to use the name of the implicit parameter itself
+-- But with splittable implicit parameters there may be many in 
+-- scope, so we make up a new name.
+newIPDict :: InstOrigin -> IPName Name -> Type 
+         -> NF_TcM (IPName Id, Inst)
+newIPDict orig ip_name ty
+  = tcGetInstLoc orig                  `thenNF_Tc` \ inst_loc@(_,loc,_) ->
+    tcGetUnique                                `thenNF_Tc` \ uniq ->
+    let
+       pred = IParam ip_name ty
+       id   = mkLocalId (mkPredName uniq loc pred) (mkPredTy pred)
+    in
+    returnNF_Tc (mapIPName (\n -> id) ip_name, Dict id pred inst_loc)
+\end{code}
 
-newDictFromOld :: Inst -> Class -> [TcType] -> NF_TcM s Inst
-newDictFromOld (Dict _ _ loc) clas tys
-  = tcGetUnique              `thenNF_Tc` \ uniq ->
-    returnNF_Tc (Dict uniq (Class clas tys) loc)
+
+%************************************************************************
+%*                                                                     *
+\subsection{Building methods (calls of overloaded functions)}
+%*                                                                     *
+%************************************************************************
+
+
+\begin{code}
+tcInstCall :: InstOrigin  -> TcType -> NF_TcM (TypecheckedHsExpr -> TypecheckedHsExpr, LIE, TcType)
+tcInstCall orig fun_ty -- fun_ty is usually a sigma-type
+  = tcInstType VanillaTv fun_ty        `thenNF_Tc` \ (tyvars, theta, tau) ->
+    newDicts orig theta                `thenNF_Tc` \ dicts ->
+    let
+       inst_fn e = mkHsDictApp (mkHsTyApp e (mkTyVarTys tyvars)) (map instToId dicts)
+    in
+    returnNF_Tc (inst_fn, mkLIE dicts, tau)
+
+tcInstDataCon orig data_con
+  = let 
+       (tvs, stupid_theta, ex_tvs, ex_theta, arg_tys, tycon) = dataConSig data_con
+            -- We generate constraints for the stupid theta even when 
+            -- pattern matching (as the Report requires)
+    in
+    tcInstTyVars VanillaTv (ex_tvs ++ tvs)     `thenNF_Tc` \ (all_tvs', ty_args', tenv) ->
+    let
+       stupid_theta' = substTheta tenv stupid_theta
+       ex_theta'     = substTheta tenv ex_theta
+       arg_tys'      = map (substTy tenv) arg_tys
+
+       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
+    newDicts orig stupid_theta'        `thenNF_Tc` \ stupid_dicts ->
+    newDicts orig ex_theta'    `thenNF_Tc` \ ex_dicts ->
+
+       -- Note that we return the stupid theta *only* in the LIE;
+       -- we don't otherwise use it at all
+    returnNF_Tc (ty_args', map instToId ex_dicts, arg_tys', result_ty,
+                mkLIE stupid_dicts, mkLIE ex_dicts, ex_tvs')
 
 
 newMethod :: InstOrigin
          -> TcId
          -> [TcType]
-         -> NF_TcM s (LIE, TcId)
+         -> NF_TcM Inst
 newMethod orig id tys
   =    -- Get the Id type and instantiate it at the specified types
     let
-       (tyvars, rho) = splitForAllTys (idType id)
-       rho_ty        = substTy (mkTyVarSubst tyvars tys) rho
-       (theta, tau)  = splitRhoTy rho_ty
+       (tyvars, rho) = tcSplitForAllTys (idType id)
+       rho_ty        = substTyWith tyvars tys rho
+       (pred, tau)   = tcSplitMethodTy rho_ty
     in
-    newMethodWithGivenTy orig id tys theta tau `thenNF_Tc` \ meth_inst ->
-    returnNF_Tc (unitLIE meth_inst, instToId meth_inst)
-
-instOverloadedFun orig (HsVar v) arg_tys theta tau
-  = newMethodWithGivenTy orig v arg_tys theta tau      `thenNF_Tc` \ inst ->
-    instFunDeps orig theta                             `thenNF_Tc` \ fds ->
-    returnNF_Tc (HsVar (instToId inst), mkLIE (inst : fds))
-
-instFunDeps orig theta
-  = tcGetInstLoc orig  `thenNF_Tc` \ loc ->
-    let ifd (Class clas tys) =
-           let fds = instantiateFdClassTys clas tys in
-           if null fds then Nothing else Just (FunDep clas fds loc)
-       ifd _ = Nothing
-    in returnNF_Tc (catMaybes (map ifd theta))
+    newMethodWithGivenTy orig id tys [pred] tau
 
 newMethodWithGivenTy orig id tys theta tau
   = tcGetInstLoc orig  `thenNF_Tc` \ loc ->
-    newMethodWith id tys theta tau loc
+    newMethodWith loc id tys theta tau
 
-newMethodWith id tys theta tau loc
+newMethodWith inst_loc@(_,loc,_) id tys theta tau
   = tcGetUnique                `thenNF_Tc` \ new_uniq ->
-    returnNF_Tc (Method new_uniq id tys theta tau loc)
+    let
+       meth_id = mkUserLocal (mkMethodOcc (getOccName id)) new_uniq tau loc
+    in
+    returnNF_Tc (Method meth_id id tys theta tau inst_loc)
 
 newMethodAtLoc :: InstLoc
               -> Id -> [TcType]
-              -> NF_TcM s (Inst, TcId)
-newMethodAtLoc loc real_id tys         -- Local function, similar to newMethod but with 
-                                       -- slightly different interface
+              -> NF_TcM (Inst, TcId)
+newMethodAtLoc inst_loc real_id tys
+       -- This actually builds the Inst
   =    -- Get the Id type and instantiate it at the specified types
-    tcGetUnique                                        `thenNF_Tc` \ new_uniq ->
     let
-       (tyvars,rho) = splitForAllTys (idType real_id)
-       rho_ty        = ASSERT( length tyvars == length tys )
+       (tyvars,rho)  = tcSplitForAllTys (idType real_id)
+       rho_ty        = ASSERT( equalLength tyvars tys )
                        substTy (mkTopTyVarSubst tyvars tys) rho
-       (theta, tau)  = splitRhoTy rho_ty
-       meth_inst     = Method new_uniq real_id tys theta tau loc
+       (theta, tau)  = tcSplitPhiTy rho_ty
     in
+    newMethodWith inst_loc real_id tys theta tau       `thenNF_Tc` \ meth_inst ->
     returnNF_Tc (meth_inst, instToId meth_inst)
 \end{code}
 
@@ -445,82 +438,60 @@ cases (the rest are caught in lookupInst).
 
 \begin{code}
 newOverloadedLit :: InstOrigin
-                -> OverloadedLit
+                -> HsOverLit
                 -> TcType
-                -> NF_TcM s (TcExpr, LIE)
-newOverloadedLit orig (OverloadedIntegral i) ty
-  | isIntTy ty && inIntRange i         -- Short cut for Int
-  = returnNF_Tc (int_lit, emptyLIE)
-
-  | isIntegerTy ty                     -- Short cut for Integer
-  = returnNF_Tc (integer_lit, emptyLIE)
-
-  where
-    intprim_lit    = HsLitOut (HsIntPrim i) intPrimTy
-    integer_lit    = HsLitOut (HsInt i) integerTy
-    int_lit        = mkHsConApp intDataCon [] [intprim_lit]
+                -> NF_TcM (TcExpr, LIE)
+newOverloadedLit orig lit expected_ty
+  | Just expr <- shortCutLit lit expected_ty
+  = returnNF_Tc (expr, emptyLIE)
 
-newOverloadedLit orig lit ty           -- The general case
+  | otherwise
   = tcGetInstLoc orig          `thenNF_Tc` \ loc ->
     tcGetUnique                        `thenNF_Tc` \ new_uniq ->
+    zapToType expected_ty      `thenNF_Tc_` 
+       -- The expected type might be a 'hole' type variable, 
+       -- in which case we must zap it to an ordinary type variable
     let
-       lit_inst = LitInst new_uniq lit ty loc
+       lit_inst = LitInst lit_id lit expected_ty loc
+       lit_id   = mkSysLocal FSLIT("lit") new_uniq expected_ty
     in
     returnNF_Tc (HsVar (instToId lit_inst), unitLIE lit_inst)
-\end{code}
 
-\begin{code}
-newIPDict name ty loc
-  = tcGetUnique                `thenNF_Tc` \ new_uniq ->
-    let d = Dict new_uniq (IParam name ty) loc in
-    returnNF_Tc d
+shortCutLit :: HsOverLit -> TcType -> Maybe TcExpr
+shortCutLit (HsIntegral i fi) ty
+  | isIntTy ty && inIntRange i && fi == fromIntegerName                -- Short cut for Int
+  = Just (HsLit (HsInt i))
+  | isIntegerTy ty && fi == fromIntegerName                    -- Short cut for Integer
+  = Just (HsLit (HsInteger i))
+
+shortCutLit (HsFractional f fr) ty
+  | isFloatTy ty  && fr == fromRationalName 
+  = Just (mkHsConApp floatDataCon [] [HsLit (HsFloatPrim f)])
+  | isDoubleTy ty && fr == fromRationalName 
+  = Just (mkHsConApp doubleDataCon [] [HsLit (HsDoublePrim f)])
+
+shortCutLit lit ty
+  = Nothing
 \end{code}
 
-\begin{code}
-instToId :: Inst -> TcId
-instToId inst = instToIdBndr inst
-
-instToIdBndr :: Inst -> TcId
-instToIdBndr (Dict u (Class clas tys) (_,loc,_))
-  = mkUserLocal (mkDictOcc (getOccName clas)) u (mkDictTy clas tys) loc
-instToIdBndr (Dict u (IParam n ty) (_,loc,_))
-  = ipToId n ty loc
-
-instToIdBndr (Method u id tys theta tau (_,loc,_))
-  = mkUserLocal (mkMethodOcc (getOccName id)) u tau loc
-
-instToIdBndr (LitInst u list ty loc)
-  = mkSysLocal SLIT("lit") u ty
-
-instToIdBndr (FunDep clas fds _)
-  = panic "FunDep escaped!!!"
-
-ipToId n ty loc
-  = mkUserLocal (mkIPOcc (getOccName n)) (nameUnique n) (mkPredTy (IParam n ty)) loc
-\end{code}
 
+%************************************************************************
+%*                                                                     *
+\subsection{Zonking}
+%*                                                                     *
+%************************************************************************
 
-Zonking
-~~~~~~~
 Zonking makes sure that the instance types are fully zonked,
-but doesn't do the same for the Id in a Method.  There's no
+but doesn't do the same for any of the Ids in an Inst.  There's no
 need, and it's a lot of extra work.
 
 \begin{code}
-zonkPred :: TcPredType -> NF_TcM s TcPredType
-zonkPred (Class clas tys)
-  = zonkTcTypes tys                    `thenNF_Tc` \ new_tys ->
-    returnNF_Tc (Class clas new_tys)
-zonkPred (IParam n ty)
-  = zonkTcType ty                      `thenNF_Tc` \ new_ty ->
-    returnNF_Tc (IParam n new_ty)
-
-zonkInst :: Inst -> NF_TcM s Inst
-zonkInst (Dict u pred loc)
-  = zonkPred pred                      `thenNF_Tc` \ new_pred ->
-    returnNF_Tc (Dict u new_pred loc)
+zonkInst :: Inst -> NF_TcM Inst
+zonkInst (Dict id pred loc)
+  = zonkTcPredType pred                        `thenNF_Tc` \ new_pred ->
+    returnNF_Tc (Dict id new_pred loc)
 
-zonkInst (Method u id tys theta tau loc) 
+zonkInst (Method m id tys theta tau loc) 
   = zonkId id                  `thenNF_Tc` \ new_id ->
        -- Essential to zonk the id in case it's a local variable
        -- Can't use zonkIdOcc because the id might itself be
@@ -529,37 +500,22 @@ zonkInst (Method u id tys theta tau loc)
     zonkTcTypes tys            `thenNF_Tc` \ new_tys ->
     zonkTcThetaType theta      `thenNF_Tc` \ new_theta ->
     zonkTcType tau             `thenNF_Tc` \ new_tau ->
-    returnNF_Tc (Method u new_id new_tys new_theta new_tau loc)
+    returnNF_Tc (Method m new_id new_tys new_theta new_tau loc)
 
-zonkInst (LitInst u lit ty loc)
+zonkInst (LitInst id lit ty loc)
   = zonkTcType ty                      `thenNF_Tc` \ new_ty ->
-    returnNF_Tc (LitInst u lit new_ty loc)
+    returnNF_Tc (LitInst id lit new_ty loc)
 
-zonkInst (FunDep clas fds loc)
-  = zonkFunDeps fds                    `thenNF_Tc` \ fds' ->
-    returnNF_Tc (FunDep clas fds' loc)
-
-zonkPreds preds = mapNF_Tc zonkPred preds
 zonkInsts insts = mapNF_Tc zonkInst insts
-
-zonkFunDeps fds = mapNF_Tc zonkFd fds
-  where
-  zonkFd (ts1, ts2)
-    = zonkTcTypes ts1                  `thenNF_Tc` \ ts1' ->
-      zonkTcTypes ts2                  `thenNF_Tc` \ ts2' ->
-      returnNF_Tc (ts1', ts2')
-
-zonkTvFunDeps fds = mapNF_Tc zonkFd fds
-  where
-  zonkFd (tvs1, tvs2)
-    = zonkTcTyVars tvs1                        `thenNF_Tc` \ tvs1' ->
-      zonkTcTyVars tvs2                        `thenNF_Tc` \ tvs2' ->
-      returnNF_Tc (tvs1', tvs2')
 \end{code}
 
 
-Printing
-~~~~~~~~
+%************************************************************************
+%*                                                                     *
+\subsection{Printing}
+%*                                                                     *
+%************************************************************************
+
 ToDo: improve these pretty-printing things.  The ``origin'' is really only
 relevant in error messages.
 
@@ -568,186 +524,126 @@ instance Outputable Inst where
     ppr inst = pprInst inst
 
 pprInst (LitInst u lit ty loc)
-  = hsep [case lit of
-             OverloadedIntegral   i -> integer i
-             OverloadedFractional f -> rational f,
-          ptext SLIT("at"),
-          ppr ty,
-          show_uniq u]
+  = hsep [ppr lit, ptext SLIT("at"), ppr ty, show_uniq u]
 
 pprInst (Dict u pred loc) = pprPred pred <+> show_uniq u
 
 pprInst m@(Method u id tys theta tau loc)
   = hsep [ppr id, ptext SLIT("at"), 
-         brackets (interppSP tys) {- ,
-         ppr theta, ppr tau,
+         brackets (sep (map pprParendType tys)) {- ,
+         ptext SLIT("theta"), ppr theta,
+         ptext SLIT("tau"), ppr tau
          show_uniq u,
          ppr (instToId m) -}]
 
-pprInst (FunDep clas fds loc)
-  = hsep [ppr clas, ppr fds]
-
-tidyPred :: TidyEnv -> TcPredType -> (TidyEnv, TcPredType)
-tidyPred env (Class clas tys)
-  = (env', Class clas tys')
-  where
-    (env', tys') = tidyOpenTypes env tys
-tidyPred env (IParam n ty)
-  = (env', IParam n ty')
-  where
-    (env', ty') = tidyOpenType env ty
-
-tidyInst :: TidyEnv -> Inst -> (TidyEnv, Inst)
-tidyInst env (LitInst u lit ty loc)
-  = (env', LitInst u lit ty' loc)
-  where
-    (env', ty') = tidyOpenType env ty
+show_uniq u = ifPprDebug (text "{-" <> ppr u <> text "-}")
 
-tidyInst env (Dict u pred loc)
-  = (env', Dict u pred' loc)
-  where
-    (env', pred') = tidyPred env pred
+tidyInst :: TidyEnv -> Inst -> Inst
+tidyInst env (LitInst u lit ty loc)         = LitInst u lit (tidyType env ty) loc
+tidyInst env (Dict u pred loc)              = Dict u (tidyPred env pred) loc
+tidyInst env (Method u id tys theta tau loc) = Method u id (tidyTypes env tys) theta tau loc
 
-tidyInst env (Method u id tys theta tau loc)
-  = (env', Method u id tys' theta tau loc)
-               -- Leave theta, tau alone cos we don't print them
+tidyMoreInsts :: TidyEnv -> [Inst] -> (TidyEnv, [Inst])
+-- This function doesn't assume that the tyvars are in scope
+-- so it works like tidyOpenType, returning a TidyEnv
+tidyMoreInsts env insts
+  = (env', map (tidyInst env') insts)
   where
-    (env', tys') = tidyOpenTypes env tys
-
--- this case shouldn't arise... (we never print fundeps)
-tidyInst env fd@(FunDep clas fds loc)
-  = (env, fd)
-
-tidyInsts env insts = mapAccumL tidyInst env insts
+    env' = tidyFreeTyVars env (tyVarsOfInsts insts)
 
-show_uniq u = ifPprDebug (text "{-" <> ppr u <> text "-}")
+tidyInsts :: [Inst] -> (TidyEnv, [Inst])
+tidyInsts insts = tidyMoreInsts emptyTidyEnv insts
 \end{code}
 
 
 %************************************************************************
 %*                                                                     *
-\subsection[InstEnv-types]{Type declarations}
+\subsection{Looking up Insts}
 %*                                                                     *
 %************************************************************************
 
 \begin{code}
-type InstanceMapper = Class -> InstEnv
-\end{code}
-
-A @ClassInstEnv@ lives inside a class, and identifies all the instances
-of that class.  The @Id@ inside a ClassInstEnv mapping is the dfun for
-that instance.  
-
-There is an important consistency constraint between the @MatchEnv@s
-in and the dfun @Id@s inside them: the free type variables of the
-@Type@ key in the @MatchEnv@ must be a subset of the universally-quantified
-type variables of the dfun.  Thus, the @ClassInstEnv@ for @Eq@ might
-contain the following entry:
-@
-       [a] ===> dfun_Eq_List :: forall a. Eq a => Eq [a]
-@
-The "a" in the pattern must be one of the forall'd variables in
-the dfun type.
-
-\begin{code}
 data LookupInstResult s
   = NoInstance
   | SimpleInst TcExpr          -- Just a variable, type application, or literal
   | GenInst    [Inst] TcExpr   -- The expression and its needed insts
 
 lookupInst :: Inst 
-          -> NF_TcM s (LookupInstResult s)
+          -> NF_TcM (LookupInstResult s)
 
 -- Dictionaries
 
-lookupInst dict@(Dict _ (Class clas tys) loc)
-  = case lookupInstEnv (ppr clas) (classInstEnv clas) tys of
-
-      Just (tenv, dfun_id)
-       -> let
-               subst         = mkSubst (tyVarsOfTypes tys) tenv
-               (tyvars, rho) = splitForAllTys (idType dfun_id)
-               ty_args       = map subst_tv tyvars
-               dfun_rho      = substTy subst rho
-               (theta, tau)  = splitRhoTy dfun_rho
-               ty_app        = mkHsTyApp (HsVar dfun_id) ty_args
-               subst_tv tv   = case lookupSubstEnv tenv tv of
-                                  Just (DoneTy ty)  -> ty
-                                       -- tenv should bind all the tyvars
+lookupInst dict@(Dict _ (ClassP clas tys) loc)
+  = getDOptsTc                 `thenNF_Tc` \ dflags ->
+    tcGetInstEnv               `thenNF_Tc` \ inst_env ->
+    case lookupInstEnv dflags inst_env clas tys of
+
+      FoundInst tenv dfun_id
+       ->      -- It's possible that not all the tyvars are in
+               -- the substitution, tenv. For example:
+               --      instance C X a => D X where ...
+               -- (presumably there's a functional dependency in class C)
+               -- Hence the mk_ty_arg to instantiate any un-substituted tyvars.        
+          let
+               (tyvars, rho) = tcSplitForAllTys (idType dfun_id)
+               mk_ty_arg tv  = case lookupSubstEnv tenv tv of
+                                  Just (DoneTy ty) -> returnNF_Tc ty
+                                  Nothing          -> tcInstTyVar VanillaTv tv `thenNF_Tc` \ tc_tv ->
+                                                      returnTc (mkTyVarTy tc_tv)
+          in
+          mapNF_Tc mk_ty_arg tyvars    `thenNF_Tc` \ ty_args ->
+          let
+               dfun_rho   = substTy (mkTyVarSubst tyvars ty_args) rho
+               (theta, _) = tcSplitPhiTy dfun_rho
+               ty_app     = mkHsTyApp (HsVar dfun_id) ty_args
           in
           if null theta then
                returnNF_Tc (SimpleInst ty_app)
           else
-          newDictsAtLoc loc theta      `thenNF_Tc` \ (dicts, dict_ids) ->
+          newDictsAtLoc loc theta      `thenNF_Tc` \ dicts ->
           let 
-               rhs = mkHsDictApp ty_app dict_ids
+               rhs = mkHsDictApp ty_app (map instToId dicts)
           in
           returnNF_Tc (GenInst dicts rhs)
 
-      Nothing  -> returnNF_Tc NoInstance
+      other    -> returnNF_Tc NoInstance
+
 lookupInst dict@(Dict _ _ loc) = returnNF_Tc NoInstance
 
 -- Methods
 
 lookupInst inst@(Method _ id tys theta _ loc)
-  = newDictsAtLoc loc theta            `thenNF_Tc` \ (dicts, dict_ids) ->
-    returnNF_Tc (GenInst dicts (mkHsDictApp (mkHsTyApp (HsVar id) tys) dict_ids))
+  = newDictsAtLoc loc theta            `thenNF_Tc` \ dicts ->
+    returnNF_Tc (GenInst dicts (mkHsDictApp (mkHsTyApp (HsVar id) tys) (map instToId dicts)))
 
 -- Literals
 
-lookupInst inst@(LitInst u (OverloadedIntegral i) ty loc)
-  | isIntTy ty && in_int_range                 -- Short cut for Int
-  = returnNF_Tc (GenInst [] int_lit)
-       -- GenInst, not SimpleInst, because int_lit is actually a constructor application
-
-  | isIntegerTy ty                             -- Short cut for Integer
-  = returnNF_Tc (GenInst [] integer_lit)
+-- Look for short cuts first: if the literal is *definitely* a 
+-- int, integer, float or a double, generate the real thing here.
+-- This is essential  (see nofib/spectral/nucleic).
+-- [Same shortcut as in newOverloadedLit, but we
+--  may have done some unification by now]             
 
-  | in_int_range                               -- It's overloaded but small enough to fit into an Int
-  = tcLookupValueByKey fromIntClassOpKey       `thenNF_Tc` \ from_int ->
-    newMethodAtLoc loc from_int [ty]           `thenNF_Tc` \ (method_inst, method_id) ->
-    returnNF_Tc (GenInst [method_inst] (HsApp (HsVar method_id) int_lit))
+lookupInst inst@(LitInst u lit ty loc)
+  | Just expr <- shortCutLit lit ty
+  = returnNF_Tc (GenInst [] expr)      -- GenInst, not SimpleInst, because 
+                                       -- expr may be a constructor application
 
-  | otherwise                                  -- Alas, it is overloaded and a big literal!
-  = tcLookupValueByKey fromIntegerClassOpKey   `thenNF_Tc` \ from_integer ->
+lookupInst inst@(LitInst u (HsIntegral i from_integer_name) ty loc)
+  = tcLookupId from_integer_name               `thenNF_Tc` \ from_integer ->
     newMethodAtLoc loc from_integer [ty]       `thenNF_Tc` \ (method_inst, method_id) ->
-    returnNF_Tc (GenInst [method_inst] (HsApp (HsVar method_id) integer_lit))
-  where
-    in_int_range   = inIntRange i
-    intprim_lit    = HsLitOut (HsIntPrim i) intPrimTy
-    integer_lit    = HsLitOut (HsInt i) integerTy
-    int_lit        = mkHsConApp intDataCon [] [intprim_lit]
+    returnNF_Tc (GenInst [method_inst] 
+                        (HsApp (HsVar method_id) (HsLit (HsInteger i))))
 
--- similar idea for overloaded floating point literals: if the literal is
--- *definitely* a float or a double, generate the real thing here.
--- This is essential  (see nofib/spectral/nucleic).
 
-lookupInst inst@(LitInst u (OverloadedFractional f) ty loc)
-  | isFloatTy ty    = returnNF_Tc (GenInst [] float_lit)
-  | isDoubleTy ty   = returnNF_Tc (GenInst [] double_lit)
-
-  | otherwise 
-  = tcLookupValueByKey fromRationalClassOpKey  `thenNF_Tc` \ from_rational ->
-
-       -- The type Rational isn't wired in so we have to conjure it up
-    tcLookupTyConByKey rationalTyConKey        `thenNF_Tc` \ rational_tycon ->
+lookupInst inst@(LitInst u (HsFractional f from_rat_name) ty loc)
+  = tcLookupId from_rat_name                   `thenNF_Tc` \ from_rational ->
+    newMethodAtLoc loc from_rational [ty]      `thenNF_Tc` \ (method_inst, method_id) ->
     let
-       rational_ty  = mkSynTy rational_tycon []
-       rational_lit = HsLitOut (HsFrac f) rational_ty
+       rational_ty  = tcFunArgTy (idType method_id)
+       rational_lit = HsLit (HsRat f rational_ty)
     in
-    newMethodAtLoc loc from_rational [ty]              `thenNF_Tc` \ (method_inst, method_id) ->
     returnNF_Tc (GenInst [method_inst] (HsApp (HsVar method_id) rational_lit))
-
-  where
-    floatprim_lit  = HsLitOut (HsFloatPrim f) floatPrimTy
-    float_lit      = mkHsConApp floatDataCon [] [floatprim_lit]
-    doubleprim_lit = HsLitOut (HsDoublePrim f) doublePrimTy
-    double_lit     = mkHsConApp doubleDataCon [] [doubleprim_lit]
-
--- there are no `instances' of functional dependencies or implicit params
-
-lookupInst _  = returnNF_Tc NoInstance
-
 \end{code}
 
 There is a second, simpler interface, when you want an instance of a
@@ -756,18 +652,19 @@ appropriate dictionary if it exists.  It is used only when resolving
 ambiguous dictionaries.
 
 \begin{code}
-lookupSimpleInst :: InstEnv
-                -> Class
-                -> [Type]                              -- Look up (c,t)
-                -> NF_TcM s (Maybe [(Class,[Type])])   -- Here are the needed (c,t)s
-
-lookupSimpleInst class_inst_env clas tys
-  = case lookupInstEnv (ppr clas) class_inst_env tys of
-      Nothing   -> returnNF_Tc Nothing
-
-      Just (tenv, dfun)
-       -> returnNF_Tc (Just (substClasses (mkSubst emptyInScopeSet tenv) theta'))
+lookupSimpleInst :: Class
+                -> [Type]                      -- Look up (c,t)
+                -> NF_TcM (Maybe ThetaType)    -- Here are the needed (c,t)s
+
+lookupSimpleInst clas tys
+  = getDOptsTc                 `thenNF_Tc` \ dflags ->
+    tcGetInstEnv               `thenNF_Tc` \ inst_env -> 
+    case lookupInstEnv dflags inst_env clas tys of
+      FoundInst tenv dfun
+       -> returnNF_Tc (Just (substTheta (mkSubst emptyInScopeSet tenv) theta))
         where
-          (_, theta, _) = splitSigmaTy (idType dfun)
-          theta' = map (\(Class clas tys) -> (clas,tys)) theta
+          (_, rho)  = tcSplitForAllTys (idType dfun)
+          (theta,_) = tcSplitPhiTy rho
+
+      other  -> returnNF_Tc Nothing
 \end{code}