[project @ 2004-11-26 16:19:45 by simonmar]
[ghc-hetmet.git] / ghc / compiler / iface / LoadIface.lhs
index bf5f694..69896be 100644 (file)
@@ -6,54 +6,59 @@
 \begin{code}
 module LoadIface (
        loadHomeInterface, loadInterface,
-       loadSrcInterface, loadOrphanModules,
+       loadSrcInterface, loadOrphanModules, loadHiBootInterface,
        readIface,      -- Used when reading the module's old interface
-       predInstGates, ifaceInstGates, ifaceStats,
-       initExternalPackageState
+       predInstGates, ifaceInstGates, ifaceStats, discardDeclPrags,
+       initExternalPackageState,
+       noIfaceErr,   -- used by CompManager too
    ) where
 
 #include "HsVersions.h"
 
+import {-# SOURCE #-}  TcIface( tcIfaceDecl )
+
+import Packages                ( PackageState(..), isHomeModule  )
 import DriverState     ( v_GhcMode, isCompManagerMode )
 import DriverUtil      ( replaceFilenameSuffix )
-import CmdLineOpts     ( DynFlags( verbosity ), DynFlag( Opt_IgnoreInterfacePragmas ), 
-                         opt_InPackage )
+import CmdLineOpts     ( DynFlags(..), DynFlag( Opt_IgnoreInterfacePragmas ) )
 import Parser          ( parseIface )
 
-import IfaceSyn                ( IfaceDecl(..), IfaceConDecls(..), IfaceConDecl(..), IfaceClassOp(..), 
-                         IfaceInst(..), IfaceRule(..), IfaceExpr(..), IfaceTyCon(..), IfaceIdInfo(..), 
-                         IfaceType(..), IfacePredType(..), IfaceExtName, visibleIfConDecls, mkIfaceExtName )
-import IfaceEnv                ( newGlobalBinder, lookupIfaceExt, lookupIfaceTc )
-import HscTypes                ( HscEnv(..), ModIface(..), emptyModIface,
-                         ExternalPackageState(..), emptyTypeEnv, emptyPool, 
-                         lookupIfaceByModName, emptyPackageIfaceTable,
-                         IsBootInterface, mkIfaceFixCache, 
-                         Pool(..), DeclPool, InstPool, 
-                         RulePool, addRuleToPool, RulePoolContents
+import IfaceSyn                ( IfaceDecl(..), IfaceConDecl(..), IfaceClassOp(..),
+                         IfaceConDecls(..), IfaceInst(..), IfaceRule(..),
+                         IfaceExpr(..), IfaceTyCon(..), IfaceIdInfo(..), 
+                         IfaceType(..), IfacePredType(..), IfaceExtName,
+                         mkIfaceExtName )
+import IfaceEnv                ( newGlobalBinder, lookupIfaceExt, lookupIfaceTc,
+                         lookupOrig )
+import HscTypes                ( ModIface(..), TyThing, emptyModIface, EpsStats(..),
+                         addEpsInStats, ExternalPackageState(..),
+                         PackageTypeEnv, emptyTypeEnv,  IfacePackage(..),
+                         lookupIfaceByModule, emptyPackageIfaceTable,
+                         IsBootInterface, mkIfaceFixCache, Gated,
+                         implicitTyThings, addRulesToPool, addInstsToPool,
+                         availNames
                         )
 
-import BasicTypes      ( Version, Fixity(..), FixityDirection(..) )
+import BasicTypes      ( Version, Fixity(..), FixityDirection(..),
+                         isMarkedStrict )
 import TcType          ( Type, tcSplitTyConApp_maybe )
 import Type            ( funTyCon )
 import TcRnMonad
 
-import PrelNames       ( gHC_PRIM_Name )
+import PrelNames       ( gHC_PRIM )
 import PrelInfo                ( ghcPrimExports )
 import PrelRules       ( builtinRules )
 import Rules           ( emptyRuleBase )
 import InstEnv         ( emptyInstEnv )
 import Name            ( Name {-instance NamedThing-}, getOccName,
-                         nameModuleName, isInternalName )
+                         nameModule, isInternalName )
 import NameEnv
 import MkId            ( seqId )
-import Packages                ( basePackage )
-import Module          ( Module, ModuleName, ModLocation(ml_hi_file),
-                         moduleName, isHomeModule, moduleEnvElts,
-                         extendModuleEnv, lookupModuleEnvByName, moduleUserString
+import Module          ( Module, ModLocation(ml_hi_file), emptyModuleEnv, 
+                         extendModuleEnv, lookupModuleEnv, moduleUserString
                        )
-import OccName         ( OccName, mkClassTyConOcc, mkClassDataConOcc,
-                         mkSuperDictSelOcc, 
-                         mkDataConWrapperOcc, mkDataConWorkerOcc )
+import OccName         ( OccName, mkOccEnv, lookupOccEnv, mkClassTyConOcc, mkClassDataConOcc,
+                         mkSuperDictSelOcc, mkDataConWrapperOcc, mkDataConWorkerOcc )
 import Class           ( Class, className )
 import TyCon           ( tyConName )
 import SrcLoc          ( mkSrcLoc, importedSrcLoc )
@@ -61,12 +66,13 @@ import Maybes               ( isJust, mapCatMaybes )
 import StringBuffer     ( hGetStringBuffer )
 import FastString      ( mkFastString )
 import ErrUtils         ( Message, mkLocMessage )
-import Finder          ( findModule, findPackageModule, 
+import Finder          ( findModule, findPackageModule,  FindResult(..),
                          hiBootExt, hiBootVerExt )
 import Lexer
 import Outputable
 import BinIface                ( readBinIface )
 import Panic
+import List            ( nub )
 
 import DATA_IOREF      ( readIORef )
 
@@ -83,7 +89,7 @@ import Directory
 %************************************************************************
 
 \begin{code}
-loadSrcInterface :: SDoc -> ModuleName -> IsBootInterface -> RnM ModIface
+loadSrcInterface :: SDoc -> Module -> IsBootInterface -> RnM ModIface
 -- This is called for each 'import' declaration in the source code
 -- On a failure, fail in the monad with an error message
 
@@ -98,7 +104,42 @@ loadSrcInterface doc mod_name want_boot
     elaborate err = hang (ptext SLIT("Failed to load interface for") <+> 
                         quotes (ppr mod_name) <> colon) 4 err
 
-loadOrphanModules :: [ModuleName] -> TcM ()
+loadHiBootInterface :: TcRn [Name]
+-- Load the hi-boot iface for the module being compiled,
+-- if it indeed exists in the transitive closure of imports
+-- Return the list of names exported by the hi-boot file
+loadHiBootInterface
+  = do         { eps <- getEps
+       ; mod <- getModule
+
+       ; traceIf (text "loadHiBootInterface" <+> ppr mod)
+
+       -- We're read all the direct imports by now, so eps_is_boot will
+       -- record if any of our imports mention us by way of hi-boot file
+       ; case lookupModuleEnv (eps_is_boot eps) mod of {
+           Nothing             -> return [] ;  -- The typical case
+
+           Just (_, False) ->          -- Someone below us imported us!
+               -- This is a loop with no hi-boot in the way
+               failWithTc (moduleLoop mod) ;
+
+           Just (mod_nm, True) ->      -- There's a hi-boot interface below us
+               
+
+    do {       -- Load it (into the PTE, and return the exported names
+         iface <- loadSrcInterface (mk_doc mod_nm) mod_nm True
+       ; sequenceM [ lookupOrig mod_nm occ
+                   | (mod,avails) <- mi_exports iface, 
+                     avail <- avails, occ <- availNames avail]
+    }}}
+  where
+    mk_doc mod = ptext SLIT("Need the hi-boot interface for") <+> ppr mod
+                <+> ptext SLIT("to compare against the Real Thing")
+
+    moduleLoop mod = ptext SLIT("Circular imports: module") <+> quotes (ppr mod) 
+                    <+> ptext SLIT("depends on itself")
+
+loadOrphanModules :: [Module] -> TcM ()
 loadOrphanModules mods
   | null mods = returnM ()
   | otherwise = initIfaceTcRn $
@@ -122,9 +163,9 @@ loadOrphanModules mods
 loadHomeInterface :: SDoc -> Name -> IfM lcl ModIface
 loadHomeInterface doc name
   = ASSERT2( not (isInternalName name), ppr name <+> parens doc )
-    loadSysInterface doc (nameModuleName name)
+    loadSysInterface doc (nameModule name)
 
-loadSysInterface :: SDoc -> ModuleName -> IfM lcl ModIface
+loadSysInterface :: SDoc -> Module -> IfM lcl ModIface
 -- A wrapper for loadInterface that Throws an exception if it fails
 loadSysInterface doc mod_name
   = do { mb_iface <- loadInterface doc mod_name ImportBySystem
@@ -145,7 +186,7 @@ loadSysInterface doc mod_name
 %*********************************************************
 
 \begin{code}
-loadInterface :: SDoc -> ModuleName -> WhereFrom 
+loadInterface :: SDoc -> Module -> WhereFrom 
              -> IfM lcl (Either Message ModIface)
 -- If it can't find a suitable interface file, we
 --     a) modify the PackageIfaceTable to have an empty entry
@@ -159,14 +200,12 @@ loadInterface :: SDoc -> ModuleName -> WhereFrom
 
 loadInterface doc_str mod_name from
   = do {       -- Read the state
-         env <- getTopEnv 
-       ; let { hpt     = hsc_HPT env
-             ; eps_var = hsc_EPS env }
-       ; eps <- readMutVar eps_var
-       ; let { pit = eps_PIT eps }
+         (eps,hpt) <- getEpsAndHpt
+
+       ; traceIf (text "Considering whether to load" <+> ppr mod_name <+> ppr from)
 
                -- Check whether we have the interface already
-       ; case lookupIfaceByModName hpt pit mod_name of {
+       ; case lookupIfaceByModule hpt (eps_PIT eps) mod_name of {
            Just iface 
                -> returnM (Right iface) ;      -- Already loaded
                        -- The (src_imp == mi_boot iface) test checks that the already-loaded
@@ -174,12 +213,11 @@ loadInterface doc_str mod_name from
                        -- if an earlier import had a before we got to real imports.   I think.
            other -> do
 
-       { if_gbl_env <- getGblEnv
-       ; let { hi_boot_file = case from of
+       { let { hi_boot_file = case from of
                                ImportByUser usr_boot -> usr_boot
                                ImportBySystem        -> sys_boot
 
-             ; mb_dep   = lookupModuleEnvByName (if_is_boot if_gbl_env) mod_name
+             ; mb_dep   = lookupModuleEnv (eps_is_boot eps) mod_name
              ; sys_boot = case mb_dep of
                                Just (_, is_boot) -> is_boot
                                Nothing           -> False
@@ -187,15 +225,19 @@ loadInterface doc_str mod_name from
              }         -- based on the dependencies in directly-imported modules
 
        -- READ THE MODULE IN
-       ; read_result <- findAndReadIface doc_str mod_name hi_boot_file
+       ; let explicit | ImportByUser _ <- from = True
+                      | otherwise              = False
+       ; read_result <- findAndReadIface explicit doc_str mod_name hi_boot_file
+       ; dflags <- getDOpts
        ; case read_result of {
            Left err -> do
-               { let { -- Not found, so add an empty iface to 
+               { let fake_iface = emptyModIface ThisPackage mod_name
+
+               ; updateEps_ $ \eps ->
+                       eps { eps_PIT = extendModuleEnv (eps_PIT eps) (mi_module fake_iface) fake_iface }
+                       -- Not found, so add an empty iface to 
                        -- the EPS map so that we don't look again
-                       fake_iface = emptyModIface opt_InPackage mod_name
-                     ; new_pit    = extendModuleEnv pit (mi_module fake_iface) fake_iface
-                     ; new_eps    = eps { eps_PIT = new_pit } }
-               ; writeMutVar eps_var new_eps
+                               
                ; returnM (Left err) } ;
 
        -- Found and parsed!
@@ -207,10 +249,10 @@ loadInterface doc_str mod_name from
        -- about, it should be from a different package to this one
        WARN(   case from of { ImportBySystem -> True; other -> False } &&
                not (isJust mb_dep) && 
-               isHomeModule mod,
-               ppr mod $$ ppr mb_dep)
+               isHomeModule dflags mod,
+               ppr mod $$ ppr mb_dep $$ ppr (eps_is_boot eps) )
 
-       initIfaceLcl (moduleName mod) $ do
+       initIfaceLcl mod_name $ do
        --      Load the new ModIface into the External Package State
        -- Even home-package interfaces loaded by loadInterface 
        --      (which only happens in OneShot mode; in Batch/Interactive 
@@ -227,19 +269,23 @@ loadInterface doc_str mod_name from
        --     If we do loadExport first the wrong info gets into the cache (unless we
        --      explicitly tag each export which seems a bit of a bore)
 
-       { new_eps_decls <- loadDecls mod (eps_decls eps) (mi_decls iface)
-       ; new_eps_insts <- loadInsts mod (eps_insts eps) (mi_insts iface)
-       ; new_eps_rules <- loadRules mod (eps_rules eps) (mi_rules iface)
+       { ignore_prags <- doptM Opt_IgnoreInterfacePragmas
+       ; new_eps_decls <- loadDecls ignore_prags mod      (mi_decls iface)
+       ; new_eps_rules <- loadRules ignore_prags mod_name (mi_rules iface)
+       ; new_eps_insts <- loadInsts              mod_name (mi_insts iface)
 
        ; let { final_iface = iface {   mi_decls = panic "No mi_decls in PIT",
                                        mi_insts = panic "No mi_insts in PIT",
-                                       mi_rules = panic "No mi_rules in PIT" }
+                                       mi_rules = panic "No mi_rules in PIT" } }
+
+       ; updateEps_  $ \ eps -> 
+               eps {   eps_PIT   = extendModuleEnv (eps_PIT eps) mod final_iface,
+                       eps_PTE   = addDeclsToPTE   (eps_PTE eps) new_eps_decls,
+                       eps_rules = addRulesToPool  (eps_rules eps) new_eps_rules,
+                       eps_insts = addInstsToPool  (eps_insts eps) new_eps_insts,
+                       eps_stats = addEpsInStats   (eps_stats eps) (length new_eps_decls)
+                                                   (length new_eps_insts) (length new_eps_rules) }
 
-             ; new_eps = eps { eps_PIT   = extendModuleEnv pit mod final_iface,
-                               eps_decls = new_eps_decls,
-                               eps_rules = new_eps_rules,
-                               eps_insts = new_eps_insts } }
-       ; writeMutVar eps_var new_eps
        ; return (Right final_iface)
     }}}}}
 
@@ -252,25 +298,37 @@ loadInterface doc_str mod_name from
 -- the declaration itself, will find the fully-glorious Name
 -----------------------------------------------------
 
-loadDecls :: Module -> DeclPool
+addDeclsToPTE :: PackageTypeEnv -> [[(Name,TyThing)]] -> PackageTypeEnv
+addDeclsToPTE pte things = foldl extendNameEnvList pte things
+
+loadDecls :: Bool      -- Don't load pragmas into the decl pool
+         -> Module
          -> [(Version, IfaceDecl)]
-         -> IfM lcl DeclPool
-loadDecls mod (Pool decls_map n_in n_out) decls
-  = do { ignore_prags <- doptM Opt_IgnoreInterfacePragmas
-       ; decls_map' <- foldlM (loadDecl ignore_prags mod) decls_map decls
-       ; returnM (Pool decls_map' (n_in + length decls) n_out) }
-
-loadDecl ignore_prags mod decls_map (_version, decl)
-  = do         { main_name <- mk_new_bndr Nothing (ifName decl)
-       ; let decl' | ignore_prags = zapIdInfo decl
-                   | otherwise    = decl
-
-       -- Populate the name cache with final versions of all the subordinate names
-       ; mapM_ (mk_new_bndr (Just main_name)) (ifaceDeclSubBndrs decl')
-
-       -- Extend the decls pool with a mapping for the main name (only)
-       ; returnM (extendNameEnv decls_map main_name decl') }
+         -> IfL [[(Name,TyThing)]]     -- The list can be poked eagerly, but the
+                                       -- TyThings are forkM'd thunks
+loadDecls ignore_prags mod decls = mapM (loadDecl ignore_prags mod) decls
+
+loadDecl ignore_prags mod (_version, decl)
+  = do         {       -- Populate the name cache with final versions of all 
+               -- the names associated with the decl
+         main_name      <- mk_new_bndr Nothing (ifName decl)
+       ; implicit_names <- mapM (mk_new_bndr (Just main_name)) (ifaceDeclSubBndrs decl)
+
+       -- Typecheck the thing, lazily
+       ; thing <- forkM doc (bumpDeclStats main_name >> tcIfaceDecl stripped_decl)
+       ; let mini_env = mkOccEnv [(getOccName t, t) | t <- implicitTyThings thing]
+             lookup n = case lookupOccEnv mini_env (getOccName n) of
+                          Just thing -> thing
+                          Nothing    -> pprPanic "loadDecl" (ppr main_name <+> ppr n)
+
+       ; returnM ((main_name, thing) : [(n, lookup n) | n <- implicit_names]) }
+               -- We build a list from the *known* names, with (lookup n) thunks
+               -- as the TyThings.  That way we can extend the PTE without poking the
+               -- thunks
   where
+    stripped_decl | ignore_prags = discardDeclPrags decl
+                 | otherwise    = decl
+
        -- mk_new_bndr allocates in the name cache the final canonical
        -- name for the thing, with the correct 
        --      * package info
@@ -279,48 +337,73 @@ loadDecl ignore_prags mod decls_map (_version, decl)
        -- imported name, to fix the module correctly in the cache
     mk_new_bndr mb_parent occ = newGlobalBinder mod occ mb_parent loc
     loc = importedSrcLoc (moduleUserString mod)
+    doc = ptext SLIT("Declaration for") <+> ppr (ifName decl)
 
-zapIdInfo decl@(IfaceId {ifIdInfo = HasInfo _}) = decl { ifIdInfo = DiscardedInfo }
-zapIdInfo decl                                         = decl
-       -- Don't alter "NoInfo", just "HasInfo"
+discardDeclPrags :: IfaceDecl -> IfaceDecl
+discardDeclPrags decl@(IfaceId {ifIdInfo = HasInfo _}) = decl { ifIdInfo = NoInfo }
+discardDeclPrags decl                                 = decl
+
+bumpDeclStats :: Name -> IfL ()                -- Record that one more declaration has actually been used
+bumpDeclStats name
+  = do { traceIf (text "Loading decl for" <+> ppr name)
+       ; updateEps_ (\eps -> let stats = eps_stats eps
+                             in eps { eps_stats = stats { n_decls_out = n_decls_out stats + 1 } })
+       }
 
 -----------------
 ifaceDeclSubBndrs :: IfaceDecl -> [OccName]
 -- *Excludes* the 'main' name, but *includes* the implicitly-bound names
--- Rather revolting, because it has to predict what gets bound
+-- Deeply revolting, because it has to predict what gets bound,
+-- especially the question of whether there's a wrapper for a datacon
 
 ifaceDeclSubBndrs (IfaceClass {ifCtxt = sc_ctxt, ifName = cls_occ, ifSigs = sigs })
-  = [tc_occ, dc_occ] ++ 
+  = [tc_occ, dc_occ, dcww_occ] ++
     [op | IfaceClassOp op _ _ <- sigs] ++
-    [mkSuperDictSelOcc n cls_occ | n <- [1..length sc_ctxt]] ++
-       -- The worker and wrapper for the DataCon of the class TyCon
-       -- are based off the data-con name
-    [mkDataConWrapperOcc dc_occ, mkDataConWorkerOcc dc_occ]
+    [mkSuperDictSelOcc n cls_occ | n <- [1..n_ctxt]] 
   where
+    n_ctxt = length sc_ctxt
+    n_sigs = length sigs
     tc_occ  = mkClassTyConOcc cls_occ
     dc_occ  = mkClassDataConOcc cls_occ        
-
-ifaceDeclSubBndrs (IfaceData {ifCons = cons}) = foldr ((++) . conDeclBndrs) [] 
-                                                     (visibleIfConDecls cons)
-ifaceDeclSubBndrs other                      = []
-
-conDeclBndrs (IfaceConDecl con_occ _ _ _ _ fields)
-  = fields ++ 
-    [con_occ, mkDataConWrapperOcc con_occ, mkDataConWorkerOcc con_occ]
-
+    dcww_occ | is_newtype = mkDataConWrapperOcc dc_occ -- Newtypes have wrapper but no worker
+            | otherwise  = mkDataConWorkerOcc dc_occ   -- Otherwise worker but no wrapper
+    is_newtype = n_sigs + n_ctxt == 1                  -- Sigh 
+
+ifaceDeclSubBndrs (IfaceData {ifCons = IfAbstractTyCon}) 
+  = []
+-- Newtype
+ifaceDeclSubBndrs (IfaceData {ifCons = IfNewTyCon (IfVanillaCon { ifConOcc = con_occ, 
+                                                                 ifConFields = fields})}) 
+  = fields ++ [con_occ, mkDataConWrapperOcc con_occ]   
+       -- Wrapper, no worker; see MkId.mkDataConIds
+
+ifaceDeclSubBndrs (IfaceData {ifCons = IfDataTyCon _ cons})
+  = nub (concatMap fld_occs cons)      -- Eliminate duplicate fields
+    ++ concatMap dc_occs cons
+  where
+    fld_occs (IfVanillaCon { ifConFields = fields }) = fields
+    fld_occs (IfGadtCon {})                         = []
+    dc_occs con_decl
+       | has_wrapper = [con_occ, work_occ, wrap_occ]
+       | otherwise   = [con_occ, work_occ]
+       where
+         con_occ = ifConOcc con_decl
+         strs    = ifConStricts con_decl
+         wrap_occ = mkDataConWrapperOcc con_occ
+         work_occ = mkDataConWorkerOcc con_occ
+         has_wrapper = any isMarkedStrict strs -- See MkId.mkDataConIds (sigh)
+               -- ToDo: may miss strictness in existential dicts
+
+ifaceDeclSubBndrs _other                     = []
 
 -----------------------------------------------------
 --     Loading instance decls
 -----------------------------------------------------
 
-loadInsts :: Module -> InstPool -> [IfaceInst] -> IfL InstPool
-loadInsts mod (Pool pool n_in n_out) decls
-  = do { new_pool <- foldlM (loadInstDecl (moduleName mod)) pool decls
-       ; returnM (Pool new_pool
-                       (n_in + length decls) 
-                       n_out) }
+loadInsts :: Module -> [IfaceInst] -> IfL [(Name, Gated IfaceInst)]
+loadInsts mod decls = mapM (loadInstDecl mod) decls
 
-loadInstDecl mod pool decl@(IfaceInst {ifInstHead = inst_ty})
+loadInstDecl mod decl@(IfaceInst {ifInstHead = inst_ty})
   = do         {
        -- Find out what type constructors and classes are "gates" for the
        -- instance declaration.  If all these "gates" are slurped in then
@@ -350,33 +433,30 @@ loadInstDecl mod pool decl@(IfaceInst {ifInstHead = inst_ty})
          let { (cls_ext, tc_exts) = ifaceInstGates inst_ty }
        ; cls <- lookupIfaceExt cls_ext
        ; tcs <- mapM lookupIfaceTc tc_exts
-       ; let { new_pool = extendNameEnv_C combine pool cls [(tcs, (mod,decl))]
-             ; combine old _ = (tcs,(mod,decl)) : old }
-       ; returnM new_pool
+       ; returnM (cls, (tcs, (mod,decl)))
        }
 
 -----------------------------------------------------
 --     Loading Rules
 -----------------------------------------------------
 
-loadRules :: Module -> RulePool -> [IfaceRule] -> IfL RulePool
-loadRules mod pool@(Pool rule_pool n_in n_out) rules
-  = do { ignore_prags <- doptM Opt_IgnoreInterfacePragmas
-       ; if ignore_prags then 
-                returnM pool
-         else do
-       { new_pool <- foldlM (loadRule (moduleName mod)) rule_pool rules
-       ; returnM (Pool new_pool (n_in + length rules) n_out) } }
+loadRules :: Bool      -- Don't load pragmas into the decl pool
+         -> Module
+         -> [IfaceRule] -> IfL [Gated IfaceRule]
+loadRules ignore_prags mod rules
+  | ignore_prags = returnM []
+  | otherwise    = mapM (loadRule mod) rules
 
-loadRule :: ModuleName -> RulePoolContents -> IfaceRule -> IfL RulePoolContents
+loadRule :: Module -> IfaceRule -> IfL (Gated IfaceRule)
 -- "Gate" the rule simply by a crude notion of the free vars of
 -- the LHS.  It can be crude, because having too few free vars is safe.
-loadRule mod_name pool decl@(IfaceRule {ifRuleHead = fn, ifRuleArgs = args})
+loadRule mod decl@(IfaceRule {ifRuleHead = fn, ifRuleArgs = args})
   = do { names <- mapM lookupIfaceExt (fn : arg_fvs)
-       ; returnM (addRuleToPool pool (mod_name, decl) names) }
+       ; returnM (names, (mod, decl)) }
   where
     arg_fvs = [n | arg <- args, n <- crudeIfExprGblFvs arg]
 
+
 ---------------------------
 crudeIfExprGblFvs :: IfaceExpr -> [IfaceExtName]
 -- A crude approximation to the free external names of an IfExpr
@@ -457,7 +537,8 @@ predInstGates cls tys
 %*********************************************************
 
 \begin{code}
-findAndReadIface :: SDoc -> ModuleName 
+findAndReadIface :: Bool               -- True <=> explicit user import
+                -> SDoc -> Module 
                 -> IsBootInterface     -- True  <=> Look for a .hi-boot file
                                        -- False <=> Look for .hi file
                 -> IfM lcl (Either Message ModIface)
@@ -467,7 +548,7 @@ findAndReadIface :: SDoc -> ModuleName
        -- It *doesn't* add an error to the monad, because 
        -- sometimes it's ok to fail... see notes with loadInterface
 
-findAndReadIface doc_str mod_name hi_boot_file
+findAndReadIface explicit doc_str mod_name hi_boot_file
   = do { traceIf (sep [hsep [ptext SLIT("Reading"), 
                              if hi_boot_file 
                                then ptext SLIT("[boot]") 
@@ -477,19 +558,26 @@ findAndReadIface doc_str mod_name hi_boot_file
                        nest 4 (ptext SLIT("reason:") <+> doc_str)])
 
        -- Check for GHC.Prim, and return its static interface
-       ; if mod_name == gHC_PRIM_Name
-         then returnM (Right ghcPrimIface)
+       ; dflags <- getDOpts
+       ; let base_id = basePackageId (pkgState dflags)
+             base_pkg 
+               | Just id <- base_id = ExternalPackage id
+               | otherwise          = ThisPackage
+               -- if basePackageId is Nothing, it means we must be
+               -- compiling the base package.
+       ; if mod_name == gHC_PRIM
+         then returnM (Right (ghcPrimIface{ mi_package = base_pkg }))
          else do
 
        -- Look for the file
-       ; mb_found <- ioToIOEnv (findHiFile mod_name hi_boot_file)
+       ; mb_found <- ioToIOEnv (findHiFile dflags explicit mod_name hi_boot_file)
        ; case mb_found of {
-             Left files -> do
+             Left err -> do
                { traceIf (ptext SLIT("...not found"))
                ; dflags <- getDOpts
-               ; returnM (Left (noIfaceErr dflags mod_name hi_boot_file files)) } ;
+               ; returnM (Left (noIfaceErr dflags mod_name err)) } ;
 
-             Right file_path -> do
+             Right (file_path,pkg) -> do 
 
        -- Found file, so read it
        { traceIf (ptext SLIT("readIFace") <+> text file_path)
@@ -497,15 +585,16 @@ findAndReadIface doc_str mod_name hi_boot_file
        ; case read_result of
            Left err    -> returnM (Left (badIfaceFile file_path err))
            Right iface 
-               | moduleName (mi_module iface) /= mod_name ->
+               | mi_module iface /= mod_name ->
                  return (Left (wrongIfaceModErr iface mod_name file_path))
                | otherwise ->
-                 returnM (Right iface)
+                 returnM (Right iface{mi_package=pkg})
+                       -- don't forget to fill in the package name...
        }}}
 
-findHiFile :: ModuleName -> IsBootInterface
-          -> IO (Either [FilePath] FilePath)
-findHiFile mod_name hi_boot_file
+findHiFile :: DynFlags -> Bool -> Module -> IsBootInterface
+          -> IO (Either FindResult (FilePath, IfacePackage))
+findHiFile dflags explicit mod_name hi_boot_file
  = do { 
        -- In interactive or --make mode, we are *not allowed* to demand-load
        -- a home package .hi file.  So don't even look for them.
@@ -516,13 +605,15 @@ findHiFile mod_name hi_boot_file
        let { home_allowed = hi_boot_file || 
                             not (isCompManagerMode ghci_mode) } ;
        maybe_found <-  if home_allowed 
-                       then findModule mod_name
-                       else findPackageModule mod_name ;
+                       then findModule dflags mod_name explicit
+                       else findPackageModule dflags mod_name explicit;
 
        case maybe_found of {
-         Left files -> return (Left files) ;
-
-         Right (_, loc) -> do {        -- Don't need module returned by finder
+         Found loc pkg -> foundOk loc hi_boot_file pkg;
+         err           -> return (Left err) ;
+       }}
+   where
+    foundOk loc hi_boot_file pkg = do {        -- Don't need module returned by finder
 
        -- Return the path to M.hi, M.hi-boot, or M.hi-boot-n as appropriate
        let { hi_path            = ml_hi_file loc ;
@@ -531,18 +622,18 @@ findHiFile mod_name hi_boot_file
            };
 
        if not hi_boot_file then
-          return (Right hi_path)
+          return (Right (hi_path,pkg))
        else do {
                hi_ver_exists <- doesFileExist hi_boot_ver_path ;
-               if hi_ver_exists then return (Right hi_boot_ver_path)
-                                else return (Right hi_boot_path)
-       }}}}
+               if hi_ver_exists then return (Right (hi_boot_ver_path,pkg))
+                                else return (Right (hi_boot_path,pkg))
+       }}
 \end{code}
 
 @readIface@ tries just the one file.
 
 \begin{code}
-readIface :: ModuleName -> String -> IsBootInterface 
+readIface :: Module -> String -> IsBootInterface 
          -> IfM lcl (Either Message ModIface)
        -- Left err    <=> file not found, or unreadable, or illegible
        -- Right iface <=> successfully found and parsed 
@@ -563,7 +654,7 @@ read_iface dflags wanted_mod file_path is_hi_boot_file
             | wanted_mod == actual_mod -> return (Right iface)
             | otherwise                -> return (Left err) 
             where
-               actual_mod = moduleName (mi_module iface)
+               actual_mod = mi_module iface
                err = hiModuleNameMismatchWarn wanted_mod actual_mod
      }}
 
@@ -587,21 +678,21 @@ read_iface dflags wanted_mod file_path is_hi_boot_file
 initExternalPackageState :: ExternalPackageState
 initExternalPackageState
   = EPS { 
+      eps_is_boot    = emptyModuleEnv,
       eps_PIT        = emptyPackageIfaceTable,
       eps_PTE        = emptyTypeEnv,
       eps_inst_env   = emptyInstEnv,
       eps_rule_base  = emptyRuleBase,
-      eps_decls      = emptyPool emptyNameEnv,
-      eps_insts      = emptyPool emptyNameEnv,
-      eps_rules      = foldr add (emptyPool []) builtinRules
+      eps_insts      = emptyNameEnv,
+      eps_rules      = addRulesToPool [] (map mk_gated_rule builtinRules),
+       -- Initialise the EPS rule pool with the built-in rules
+      eps_stats = EpsStats { n_ifaces_in = 0, n_decls_in = 0, n_decls_out = 0
+                          , n_insts_in = 0, n_insts_out = 0
+                          , n_rules_in = length builtinRules, n_rules_out = 0 }
     }
   where
-       -- Initialise the EPS rule pool with the built-in rules
-    add (fn_name, core_rule) (Pool rules n_in n_out) 
-      = Pool rules' (n_in+1) n_out
-      where
-       rules' = addRuleToPool rules iface_rule [fn_name]
-       iface_rule = (nameModuleName fn_name, IfaceBuiltinRule (mkIfaceExtName fn_name) core_rule)
+    mk_gated_rule (fn_name, core_rule)
+       = ([fn_name], (nameModule fn_name, IfaceBuiltinRule (mkIfaceExtName fn_name) core_rule))
 \end{code}
 
 
@@ -614,8 +705,8 @@ initExternalPackageState
 \begin{code}
 ghcPrimIface :: ModIface
 ghcPrimIface
-  = (emptyModIface basePackage gHC_PRIM_Name) {
-       mi_exports  = [(gHC_PRIM_Name, ghcPrimExports)],
+  = (emptyModIface ThisPackage gHC_PRIM) {
+       mi_exports  = [(gHC_PRIM, ghcPrimExports)],
        mi_decls    = [],
        mi_fixities = fixities,
        mi_fix_fn  = mkIfaceFixCache fixities
@@ -634,23 +725,17 @@ ghcPrimIface
 \begin{code}
 ifaceStats :: ExternalPackageState -> SDoc
 ifaceStats eps 
-  = hcat [text "Renamer stats: ", stats]
+  = hcat [text "Renamer stats: ", msg]
   where
-    n_mods = length [() | _ <- moduleEnvElts (eps_PIT eps)]
-       -- This is really only right for a one-shot compile
-
-    Pool _ n_decls_in n_decls_out = eps_decls eps
-    Pool _ n_insts_in n_insts_out = eps_insts eps
-    Pool _ n_rules_in n_rules_out = eps_rules eps
-    
-    stats = vcat 
-       [int n_mods <+> text "interfaces read",
-        hsep [ int n_decls_out, text "type/class/variable imported, out of", 
-               int n_decls_in, text "read"],
-        hsep [ int n_insts_out, text "instance decls imported, out of",  
-               int n_insts_in, text "read"],
-        hsep [ int n_rules_out, text "rule decls imported, out of",  
-               int n_rules_in, text "read"]
+    stats = eps_stats eps
+    msg = vcat 
+       [int (n_ifaces_in stats) <+> text "interfaces read",
+        hsep [ int (n_decls_out stats), text "type/class/variable imported, out of", 
+               int (n_decls_in stats), text "read"],
+        hsep [ int (n_insts_out stats), text "instance decls imported, out of",  
+               int (n_insts_in stats), text "read"],
+        hsep [ int (n_rules_out stats), text "rule decls imported, out of",  
+               int (n_rules_in stats), text "read"]
        ]
 \end{code}    
 
@@ -666,7 +751,7 @@ badIfaceFile file err
   = vcat [ptext SLIT("Bad interface file:") <+> text file, 
          nest 4 err]
 
-hiModuleNameMismatchWarn :: ModuleName -> ModuleName -> Message
+hiModuleNameMismatchWarn :: Module -> Module -> Message
 hiModuleNameMismatchWarn requested_mod read_mod = 
     hsep [ ptext SLIT("Something is amiss; requested module name")
         , ppr requested_mod
@@ -674,11 +759,21 @@ hiModuleNameMismatchWarn requested_mod read_mod =
         , ppr read_mod
         ]
 
-noIfaceErr dflags mod_name boot_file files
+noIfaceErr dflags mod_name (PackageHidden pkg)
+  = ptext SLIT("Could not import") <+> quotes (ppr mod_name) <> colon
+    $$ ptext SLIT("it is a member of package") <+> quotes (ppr pkg) <> comma
+        <+> ptext SLIT("which is hidden")
+
+noIfaceErr dflags mod_name (ModuleHidden pkg)
+  = ptext SLIT("Could not import") <+> quotes (ppr mod_name) <> colon
+    $$ ptext SLIT("it is hidden") 
+       <+> parens (ptext SLIT("in package") <+> quotes (ppr pkg))
+
+noIfaceErr dflags mod_name (NotFound files)
   = ptext SLIT("Could not find interface file for") <+> quotes (ppr mod_name)
-    $$ extra
+    $$ extra files
   where 
-   extra
+  extra files
     | verbosity dflags < 3 = 
         text "(use -v to see a list of the files searched for)"
     | otherwise =