[project @ 2000-11-10 15:12:50 by simonpj]
[ghc-hetmet.git] / ghc / compiler / rename / Rename.lhs
index 690b377..ad60177 100644 (file)
@@ -4,70 +4,70 @@
 \section[Rename]{Renaming and dependency analysis passes}
 
 \begin{code}
-module Rename ( renameModule ) where
+module Rename ( renameModule, closeIfaceDecls, checkOldIface ) where
 
 #include "HsVersions.h"
 
 import HsSyn
 import RdrHsSyn                ( RdrNameHsModule, RdrNameHsDecl, RdrNameDeprecation, 
-                         RdrNameTyClDecl, RdrNameRuleDecl, RdrNameInstDecl
+                         RdrNameTyClDecl, RdrNameRuleDecl, RdrNameInstDecl, RdrNameImportDecl
                        )
 import RnHsSyn         ( RenamedHsDecl, RenamedTyClDecl, RenamedRuleDecl, RenamedInstDecl,
-                         extractHsTyNames, extractHsCtxtTyNames
+                         extractHsTyNames, 
+                         instDeclFVs, tyClDeclFVs, ruleDeclFVs
                        )
 
 import CmdLineOpts     ( DynFlags, DynFlag(..) )
 import RnMonad
 import RnNames         ( getGlobalNames )
-import RnSource                ( rnSourceDecls, rnDecl, rnTyClDecl, rnRuleDecl, rnInstDecl )
-import RnIfaces                ( getImportedInstDecls, importDecl, mkImportInfo, 
-                         getInterfaceExports,
-                         getImportedRules, getSlurped,
-                         ImportDeclResult(..),
-                         RecompileRequired, recompileRequired
+import RnSource                ( rnSourceDecls, rnTyClDecl, rnIfaceRuleDecl, rnInstDecl )
+import RnIfaces                ( slurpImpDecls, mkImportInfo, 
+                         getInterfaceExports, closeDecls,
+                         RecompileRequired, outOfDate, recompileRequired
                        )
-import RnHiFiles       ( findAndReadIface, removeContext, loadExports, loadFixDecls, loadDeprecs )
-import RnEnv           ( availName, availsToNameSet, 
-                         emptyAvailEnv, unitAvailEnv, availEnvElts, plusAvailEnv, sortAvails,
+import RnHiFiles       ( readIface, removeContext, loadInterface,
+                         loadExports, loadFixDecls, loadDeprecs )
+import RnEnv           ( availsToNameSet, availName, 
+                         emptyAvailEnv, unitAvailEnv, availEnvElts, plusAvailEnv, groupAvails,
                          warnUnusedImports, warnUnusedLocalBinds, warnUnusedModules,
-                         lookupOrigNames, lookupGlobalRn, newGlobalName,
-                         FreeVars, plusFVs, plusFV, unitFV, emptyFVs, isEmptyFVs, addOneFV
+                         lookupOrigNames, lookupSrcName, newGlobalName, unQualInScope
                        )
 import Module           ( Module, ModuleName, WhereFrom(..),
-                         moduleNameUserString, moduleName, 
-                         lookupModuleEnv
+                         moduleNameUserString, moduleName,
+                         moduleEnvElts
                        )
-import Name            ( Name, isLocallyDefined, NamedThing(..), getSrcLoc,
-                         nameOccName, nameUnique, nameModule,
-                         mkNameEnv, nameEnvElts, extendNameEnv
+import Name            ( Name, NamedThing(..), getSrcLoc,
+                         nameIsLocalOrFrom, nameOccName, nameModule,
                        )
+import Name            ( mkNameEnv, nameEnvElts, extendNameEnv )
+import RdrName         ( elemRdrEnv, foldRdrEnv, isQual )
 import OccName         ( occNameFlavour )
-import Id              ( idType )
-import TyCon           ( isSynTyCon, getSynTyConDefn )
 import NameSet
-import TysWiredIn      ( unitTyCon, intTyCon, doubleTyCon, boolTyCon )
+import TysWiredIn      ( unitTyCon, intTyCon, boolTyCon )
 import PrelNames       ( mAIN_Name, pREL_MAIN_Name, pRELUDE_Name,
-                         ioTyCon_RDR,
+                         ioTyCon_RDR, main_RDR_Unqual,
                          unpackCString_RDR, unpackCStringFoldr_RDR, unpackCStringUtf8_RDR,
                          eqString_RDR
                        )
-import PrelInfo                ( fractionalClassKeys, derivingOccurrences, wiredInThingEnv )
-import Type            ( namesOfType, funTyCon )
-import ErrUtils                ( dumpIfSet )
+import PrelInfo                ( derivingOccurrences )
+import Type            ( funTyCon )
+import ErrUtils                ( dumpIfSet, showPass, printErrorsAndWarnings, errorsFound )
 import Bag             ( bagToList )
-import FiniteMap       ( FiniteMap, eltsFM, fmToList, emptyFM, lookupFM, 
+import FiniteMap       ( FiniteMap, fmToList, emptyFM, lookupFM, 
                          addToFM_C, elemFM, addToFM
                        )
 import UniqFM          ( lookupUFM )
 import Maybes          ( maybeToBool, catMaybes )
 import Outputable
 import IO              ( openFile, IOMode(..) )
-import HscTypes                ( Finder, PersistentCompilerState, HomeIfaceTable, HomeSymbolTable, 
-                         ModIface(..), TyThing(..), WhatsImported(..), 
-                         VersionInfo(..), ImportVersion, IfaceDecls(..),
-                         GlobalRdrEnv, AvailEnv, Avails, GenAvailInfo(..), AvailInfo, 
+import HscTypes                ( PersistentCompilerState, HomeIfaceTable, HomeSymbolTable, 
+                         ModIface(..), WhatsImported(..), 
+                         VersionInfo(..), ImportVersion, 
+                         IfaceDecls, mkIfaceDecls, dcl_tycl, dcl_rules, dcl_insts,
+                         GlobalRdrEnv, pprGlobalRdrEnv,
+                         AvailEnv, GenAvailInfo(..), AvailInfo, Avails,
                          Provenance(..), ImportReason(..), initialVersionInfo,
-                         Deprecations(..), lookupDeprec
+                         Deprecations(..), lookupDeprec, lookupIface
                         )
 import List            ( partition, nub )
 \end{code}
@@ -81,43 +81,56 @@ import List         ( partition, nub )
 %*********************************************************
 
 \begin{code}
-renameModule :: DynFlags -> Finder 
+renameModule :: DynFlags
             -> HomeIfaceTable -> HomeSymbolTable
             -> PersistentCompilerState 
             -> Module -> RdrNameHsModule 
-            -> IO (PersistentCompilerState, Maybe (ModIface, [RenamedHsDecl]))
+            -> IO (PersistentCompilerState, Maybe (PrintUnqualified, ModIface, [RenamedHsDecl]))
        -- Nothing => some error occurred in the renamer
 
-renameModule dflags finder hit hst old_pcs this_module rdr_module
-  =    -- Initialise the renamer monad
-    do {
-       (new_pcs, errors_found, (maybe_rn_stuff, dump_action)) 
-          <- initRn dflags finder hit hst old_pcs this_module (rename this_module rdr_module) ;
+renameModule dflags hit hst old_pcs this_module rdr_module
+  = do { showPass dflags "Renamer"
 
-       -- Dump any debugging output
-       dump_action ;
+               -- Initialise the renamer monad
+       ; (new_pcs, msgs, maybe_rn_stuff) <- initRn dflags hit hst old_pcs this_module 
+                                                   (rename this_module rdr_module)
 
-       -- Return results.  No harm in updating the PCS
-       if errors_found then
+       ; let print_unqualified :: Name -> Bool -- Is this chap in scope unqualified?
+             print_unqualified = case maybe_rn_stuff of
+                                   Just (unqual, _, _) -> unqual
+                                   Nothing             -> alwaysQualify
+
+
+               -- Print errors from renaming
+       ;  printErrorsAndWarnings print_unqualified msgs ;
+
+               -- Return results.  No harm in updating the PCS
+       ; if errorsFound msgs then
            return (new_pcs, Nothing)
-        else
+          else     
            return (new_pcs, maybe_rn_stuff)
     }
 \end{code}
 
 \begin{code}
-rename :: Module -> RdrNameHsModule -> RnMG (Maybe (ModIface, [RenamedHsDecl]), IO ())
-rename this_module this_mod@(HsModule mod_name vers exports imports local_decls mod_deprec loc)
-  =    -- FIND THE GLOBAL NAME ENVIRONMENT
-    getGlobalNames this_mod                    `thenRn` \ maybe_stuff ->
+rename :: Module -> RdrNameHsModule -> RnMG (Maybe (PrintUnqualified, ModIface, [RenamedHsDecl]))
+rename this_module contents@(HsModule _ _ _ imports local_decls mod_deprec loc)
+  = pushSrcLocRn loc           $
 
-       -- CHECK FOR EARLY EXIT
-    case maybe_stuff of {
-       Nothing ->      -- Everything is up to date; no need to recompile further
-               rnDump [] []            `thenRn` \ dump_action ->
-               returnRn (Nothing, dump_action) ;
+       -- FIND THE GLOBAL NAME ENVIRONMENT
+    getGlobalNames this_module contents        `thenRn` \ (gbl_env, local_gbl_env, 
+                                                           export_avails, global_avail_env) ->
 
-       Just (gbl_env, local_gbl_env, export_avails, global_avail_env) ->
+       -- Exit if we've found any errors
+    checkErrsRn                                `thenRn` \ no_errs_so_far ->
+    if not no_errs_so_far then
+       -- Found errors already, so exit now
+       rnDump [] []            `thenRn_`
+       returnRn Nothing 
+    else
+       
+    traceRn (text "Local top-level environment" $$ 
+            nest 4 (pprGlobalRdrEnv local_gbl_env))    `thenRn_`
 
        -- DEAL WITH DEPRECATIONS
     rnDeprecs local_gbl_env mod_deprec 
@@ -127,77 +140,81 @@ rename this_module this_mod@(HsModule mod_name vers exports imports local_decls
     fixitiesFromLocalDecls local_gbl_env local_decls   `thenRn` \ local_fixity_env ->
 
        -- RENAME THE SOURCE
-    initRnMS gbl_env local_fixity_env SourceMode (
-       rnSourceDecls local_decls
-    )                                  `thenRn` \ (rn_local_decls, source_fvs) ->
+    rnSourceDecls gbl_env local_fixity_env local_decls `thenRn` \ (rn_local_decls, source_fvs) ->
+
+       -- CHECK THAT main IS DEFINED, IF REQUIRED
+    checkMain this_module local_gbl_env                `thenRn_`
 
        -- SLURP IN ALL THE NEEDED DECLARATIONS
     implicitFVs mod_name rn_local_decls        `thenRn` \ implicit_fvs -> 
     let
-               -- The export_fvs make the exported names look just as if they
-               -- occurred in the source program.  For the reasoning, see the
-               -- comments with RnIfaces.getImportVersions.
-               -- We only need the 'parent name' of the avail;
-               -- that's enough to suck in the declaration.
-       export_fvs      = mkNameSet (map availName export_avails)
-       real_source_fvs = source_fvs `plusFV` export_fvs
-
-       slurp_fvs       = implicit_fvs `plusFV` real_source_fvs
+       slurp_fvs = implicit_fvs `plusFV` source_fvs
                -- It's important to do the "plus" this way round, so that
                -- when compiling the prelude, locally-defined (), Bool, etc
                -- override the implicit ones. 
     in
+    traceRn (text "Source FVs:" <+> fsep (map ppr (nameSetToList slurp_fvs)))  `thenRn_`
     slurpImpDecls slurp_fvs            `thenRn` \ rn_imp_decls ->
 
        -- EXIT IF ERRORS FOUND
-    rnDump rn_imp_decls rn_local_decls         `thenRn` \ dump_action ->
+    rnDump rn_imp_decls rn_local_decls         `thenRn_` 
     checkErrsRn                                        `thenRn` \ no_errs_so_far ->
     if not no_errs_so_far then
        -- Found errors already, so exit now
-       returnRn (Nothing, dump_action)
+       returnRn Nothing
     else
 
        -- GENERATE THE VERSION/USAGE INFO
-    mkImportInfo mod_name imports      `thenRn` \ my_usages ->
+    mkImportInfo mod_name imports                      `thenRn` \ my_usages ->
 
-       -- RETURN THE RENAMED MODULE
-    getNameSupplyRn                    `thenRn` \ name_supply ->
-    getIfacesRn                        `thenRn` \ ifaces ->
+       -- BUILD THE MODULE INTERFACE
     let
-       direct_import_mods :: [ModuleName]
-       direct_import_mods = nub [m | ImportDecl m _ _ _ _ _ <- imports]
-
        -- We record fixities even for things that aren't exported,
        -- so that we can change into the context of this moodule easily
        fixities = mkNameEnv [ (name, fixity)
                             | FixitySig name fixity loc <- nameEnvElts local_fixity_env
                             ]
 
-
        -- Sort the exports to make them easier to compare for versions
-       my_exports = sortAvails export_avails
+       my_exports = groupAvails this_module export_avails
        
+       final_decls = rn_local_decls ++ rn_imp_decls
+       is_orphan   = any (isOrphanDecl this_module) rn_local_decls
+
        mod_iface = ModIface {  mi_module   = this_module,
                                mi_version  = initialVersionInfo,
-                               mi_orphan   = any isOrphanDecl rn_local_decls,
+                               mi_usages = my_usages,
+                               mi_boot     = False,
+                               mi_orphan   = is_orphan,
                                mi_exports  = my_exports,
                                mi_globals  = gbl_env,
-                               mi_usages   = my_usages,
                                mi_fixities = fixities,
                                mi_deprecs  = my_deprecs,
                                mi_decls    = panic "mi_decls"
                    }
 
-       final_decls = rn_local_decls ++ rn_imp_decls
+       print_unqualified = unQualInScope gbl_env
     in
 
        -- REPORT UNUSED NAMES, AND DEBUG DUMP 
-    reportUnusedNames mod_name direct_import_mods
-                     gbl_env global_avail_env
-                     export_avails source_fvs
-                     rn_imp_decls                      `thenRn_`
+    reportUnusedNames mod_iface print_unqualified 
+                     imports global_avail_env
+                     source_fvs export_avails rn_imp_decls     `thenRn_`
 
-    returnRn (Just (mod_iface, final_decls), dump_action) }
+    returnRn (Just (print_unqualified, mod_iface, final_decls))
+  where
+    mod_name = moduleName this_module
+\end{code}
+
+Checking that main is defined
+
+\begin{code}
+checkMain :: Module -> GlobalRdrEnv -> RnMG ()
+checkMain this_mod local_env
+  | moduleName this_mod == mAIN_Name 
+  = checkRn (main_RDR_Unqual `elemRdrEnv` local_env) noMainErr
+  | otherwise
+  = returnRn ()
 \end{code}
 
 @implicitFVs@ forces the renamer to slurp in some things which aren't
@@ -230,8 +247,8 @@ implicitFVs mod_name decls
     implicit_occs = string_occs ++ foldr ((++) . get) implicit_main decls
 
        -- Virtually every program has error messages in it somewhere
-    string_occs = [unpackCString_RDR, unpackCStringFoldr_RDR, unpackCStringUtf8_RDR,
-                  eqString_RDR]
+    string_occs = [unpackCString_RDR, unpackCStringFoldr_RDR, 
+                  unpackCStringUtf8_RDR, eqString_RDR]
 
     get (TyClD (TyData _ _ _ _ _ _ (Just deriv_classes) _ _ _))
        = concat (map get_deriv deriv_classes)
@@ -243,20 +260,21 @@ implicitFVs mod_name decls
 \end{code}
 
 \begin{code}
-isOrphanDecl (InstD (InstDecl inst_ty _ _ _ _))
-  = not (foldNameSet ((||) . isLocallyDefined) False (extractHsTyNames (removeContext inst_ty)))
+isOrphanDecl this_mod (InstD (InstDecl inst_ty _ _ _ _))
+  = not (foldNameSet ((||) . nameIsLocalOrFrom this_mod) False 
+                    (extractHsTyNames (removeContext inst_ty)))
        -- The 'removeContext' is because of
        --      instance Foo a => Baz T where ...
        -- The decl is an orphan if Baz and T are both not locally defined,
        --      even if Foo *is* locally defined
 
-isOrphanDecl (RuleD (HsRule _ _ _ lhs _ _))
+isOrphanDecl this_mod (RuleD (HsRule _ _ _ lhs _ _))
   = check lhs
   where
        -- At the moment we just check for common LHS forms
        -- Expand as necessary.  Getting it wrong just means
        -- more orphans than necessary
-    check (HsVar v)      = not (isLocallyDefined v)
+    check (HsVar v)      = not (nameIsLocalOrFrom this_mod v)
     check (HsApp f a)    = check f && check a
     check (HsLit _)      = False
     check (HsOverLit _)          = False
@@ -268,297 +286,7 @@ isOrphanDecl (RuleD (HsRule _ _ _ lhs _ _))
 
     check other                  = True        -- Safe fall through
 
-isOrphanDecl other = False
-\end{code}
-
-
-%*********************************************************
-%*                                                      *
-\subsection{Slurping declarations}
-%*                                                      *
-%*********************************************************
-
-\begin{code}
--------------------------------------------------------
-slurpImpDecls source_fvs
-  = traceRn (text "slurpImp" <+> fsep (map ppr (nameSetToList source_fvs))) `thenRn_`
-
-       -- The current slurped-set records all local things
-    getSlurped                                 `thenRn` \ source_binders ->
-    slurpSourceRefs source_binders source_fvs  `thenRn` \ (decls, needed) ->
-
-       -- Then get everything else
-    closeDecls decls needed                    `thenRn` \ decls1 ->
-
-       -- Finally, get any deferred data type decls
-    slurpDeferredDecls decls1                  `thenRn` \ final_decls -> 
-
-    returnRn final_decls
-
--------------------------------------------------------
-slurpSourceRefs :: NameSet                     -- Variables defined in source
-               -> FreeVars                     -- Variables referenced in source
-               -> RnMG ([RenamedHsDecl],
-                        FreeVars)              -- Un-satisfied needs
--- The declaration (and hence home module) of each gate has
--- already been loaded
-
-slurpSourceRefs source_binders source_fvs
-  = go_outer []                        -- Accumulating decls
-            emptyFVs                   -- Unsatisfied needs
-            emptyFVs                   -- Accumulating gates
-            (nameSetToList source_fvs) -- Things whose defn hasn't been loaded yet
-  where
-       -- The outer loop repeatedly slurps the decls for the current gates
-       -- and the instance decls 
-
-       -- The outer loop is needed because consider
-       --      instance Foo a => Baz (Maybe a) where ...
-       -- It may be that @Baz@ and @Maybe@ are used in the source module,
-       -- but not @Foo@; so we need to chase @Foo@ too.
-       --
-       -- We also need to follow superclass refs.  In particular, 'chasing @Foo@' must
-       -- include actually getting in Foo's class decl
-       --      class Wib a => Foo a where ..
-       -- so that its superclasses are discovered.  The point is that Wib is a gate too.
-       -- We do this for tycons too, so that we look through type synonyms.
-
-    go_outer decls fvs all_gates []    
-       = returnRn (decls, fvs)
-
-    go_outer decls fvs all_gates refs  -- refs are not necessarily slurped yet
-       = traceRn (text "go_outer" <+> ppr refs)                `thenRn_`
-         foldlRn go_inner (decls, fvs, emptyFVs) refs          `thenRn` \ (decls1, fvs1, gates1) ->
-         getImportedInstDecls (all_gates `plusFV` gates1)      `thenRn` \ inst_decls ->
-         rnInstDecls decls1 fvs1 gates1 inst_decls             `thenRn` \ (decls2, fvs2, gates2) ->
-         go_outer decls2 fvs2 (all_gates `plusFV` gates2)
-                              (nameSetToList (gates2 `minusNameSet` all_gates))
-               -- Knock out the all_gates because even if we don't slurp any new
-               -- decls we can get some apparently-new gates from wired-in names
-
-    go_inner (decls, fvs, gates) wanted_name
-       = importDecl wanted_name                `thenRn` \ import_result ->
-         case import_result of
-           AlreadySlurped -> returnRn (decls, fvs, gates)
-           WiredIn        -> returnRn (decls, fvs, gates `plusFV` getWiredInGates wanted_name)
-           Deferred       -> returnRn (decls, fvs, gates `addOneFV` wanted_name)       -- It's a type constructor
-                       
-           HereItIs decl -> rnIfaceTyClDecl decl               `thenRn` \ (new_decl, fvs1) ->
-                            returnRn (TyClD new_decl : decls, 
-                                      fvs1 `plusFV` fvs,
-                                      gates `plusFV` getGates source_fvs new_decl)
-
-rnInstDecls decls fvs gates []
-  = returnRn (decls, fvs, gates)
-rnInstDecls decls fvs gates (d:ds) 
-  = rnIfaceDecl d              `thenRn` \ (new_decl, fvs1) ->
-    rnInstDecls (new_decl:decls) 
-               (fvs1 `plusFV` fvs)
-               (gates `plusFV` getInstDeclGates new_decl)
-               ds
-\end{code}
-
-
-\begin{code}
--------------------------------------------------------
--- closeDecls keeps going until the free-var set is empty
-closeDecls decls needed
-  | not (isEmptyFVs needed)
-  = slurpDecls decls needed    `thenRn` \ (decls1, needed1) ->
-    closeDecls decls1 needed1
-
-  | otherwise
-  = getImportedRules                   `thenRn` \ rule_decls ->
-    case rule_decls of
-       []    -> returnRn decls -- No new rules, so we are done
-       other -> rnIfaceDecls decls emptyFVs rule_decls         `thenRn` \ (decls1, needed1) ->
-                closeDecls decls1 needed1
-                
-
--------------------------------------------------------
--- Augment decls with any decls needed by needed.
--- Return also free vars of the new decls (only)
-slurpDecls decls needed
-  = go decls emptyFVs (nameSetToList needed) 
-  where
-    go decls fvs []         = returnRn (decls, fvs)
-    go decls fvs (ref:refs) = slurpDecl decls fvs ref  `thenRn` \ (decls1, fvs1) ->
-                             go decls1 fvs1 refs
-
--------------------------------------------------------
-slurpDecl decls fvs wanted_name
-  = importDecl wanted_name             `thenRn` \ import_result ->
-    case import_result of
-       -- Found a declaration... rename it
-       HereItIs decl -> rnIfaceTyClDecl decl           `thenRn` \ (new_decl, fvs1) ->
-                        returnRn (TyClD new_decl:decls, fvs1 `plusFV` fvs)
-
-       -- No declaration... (wired in thing, or deferred, or already slurped)
-       other -> returnRn (decls, fvs)
-
-
--------------------------------------------------------
-rnIfaceDecls :: [RenamedHsDecl] -> FreeVars
-            -> [(Module, RdrNameHsDecl)]
-            -> RnM d ([RenamedHsDecl], FreeVars)
-rnIfaceDecls decls fvs []     = returnRn (decls, fvs)
-rnIfaceDecls decls fvs (d:ds) = rnIfaceDecl d          `thenRn` \ (new_decl, fvs1) ->
-                               rnIfaceDecls (new_decl:decls) (fvs1 `plusFV` fvs) ds
-
-rnIfaceDecl    (mod, decl) = initIfaceRnMS mod (rnDecl decl)   
-rnIfaceTyClDecl (mod, decl) = initIfaceRnMS mod (rnTyClDecl decl)      
-\end{code}
-
-
-%*********************************************************
-%*                                                      *
-\subsection{Deferred declarations}
-%*                                                      *
-%*********************************************************
-
-The idea of deferred declarations is this.  Suppose we have a function
-       f :: T -> Int
-       data T = T1 A | T2 B
-       data A = A1 X | A2 Y
-       data B = B1 P | B2 Q
-Then we don't want to load T and all its constructors, and all
-the types those constructors refer to, and all the types *those*
-constructors refer to, and so on.  That might mean loading many more
-interface files than is really necessary.  So we 'defer' loading T.
-
-But f might be strict, and the calling convention for evaluating
-values of type T depends on how many constructors T has, so 
-we do need to load T, but not the full details of the type T.
-So we load the full decl for T, but only skeleton decls for A and B:
-       f :: T -> Int
-       data T = {- 2 constructors -}
-
-Whether all this is worth it is moot.
-
-\begin{code}
-slurpDeferredDecls :: [RenamedHsDecl] -> RnMG [RenamedHsDecl]
-slurpDeferredDecls decls = returnRn decls
-
-{-     OMIT FOR NOW
-slurpDeferredDecls :: [RenamedHsDecl] -> RnMG [RenamedHsDecl]
-slurpDeferredDecls decls
-  = getDeferredDecls                                           `thenRn` \ def_decls ->
-    rnIfaceDecls decls emptyFVs (map stripDecl def_decls)      `thenRn` \ (decls1, fvs) ->
-    ASSERT( isEmptyFVs fvs )
-    returnRn decls1
-
-stripDecl (mod, TyClD (TyData dt _ tc tvs _ nconstrs _ loc name1 name2))
-  = (mod, TyClD (TyData dt [] tc tvs [] nconstrs Nothing loc
-               name1 name2))
-       -- Nuke the context and constructors
-       -- But retain the *number* of constructors!
-       -- Also the tvs will have kinds on them.
--}
-\end{code}
-
-
-%*********************************************************
-%*                                                      *
-\subsection{Extracting the `gates'}
-%*                                                      *
-%*********************************************************
-
-When we import a declaration like
-\begin{verbatim}
-       data T = T1 Wibble | T2 Wobble
-\end{verbatim}
-we don't want to treat @Wibble@ and @Wobble@ as gates
-{\em unless} @T1@, @T2@ respectively are mentioned by the user program.
-If only @T@ is mentioned
-we want only @T@ to be a gate;
-that way we don't suck in useless instance
-decls for (say) @Eq Wibble@, when they can't possibly be useful.
-
-@getGates@ takes a newly imported (and renamed) decl, and the free
-vars of the source program, and extracts from the decl the gate names.
-
-\begin{code}
-getGates source_fvs (IfaceSig _ ty _ _)
-  = extractHsTyNames ty
-
-getGates source_fvs (ClassDecl ctxt cls tvs _ sigs _ _ _ )
-  = (delListFromNameSet (foldr (plusFV . get) (extractHsCtxtTyNames ctxt) sigs)
-                       (hsTyVarNames tvs)
-     `addOneToNameSet` cls)
-    `plusFV` maybe_double
-  where
-    get (ClassOpSig n _ ty _) 
-       | n `elemNameSet` source_fvs = extractHsTyNames ty
-       | otherwise                  = emptyFVs
-
-       -- If we load any numeric class that doesn't have
-       -- Int as an instance, add Double to the gates. 
-       -- This takes account of the fact that Double might be needed for
-       -- defaulting, but we don't want to load Double (and all its baggage)
-       -- if the more exotic classes aren't used at all.
-    maybe_double | nameUnique cls `elem` fractionalClassKeys 
-                = unitFV (getName doubleTyCon)
-                | otherwise
-                = emptyFVs
-
-getGates source_fvs (TySynonym tycon tvs ty _)
-  = delListFromNameSet (extractHsTyNames ty)
-                      (hsTyVarNames tvs)
-       -- A type synonym type constructor isn't a "gate" for instance decls
-
-getGates source_fvs (TyData _ ctxt tycon tvs cons _ _ _ _ _)
-  = delListFromNameSet (foldr (plusFV . get) (extractHsCtxtTyNames ctxt) cons)
-                      (hsTyVarNames tvs)
-    `addOneToNameSet` tycon
-  where
-    get (ConDecl n _ tvs ctxt details _)
-       | n `elemNameSet` source_fvs
-               -- If the constructor is method, get fvs from all its fields
-       = delListFromNameSet (get_details details `plusFV` 
-                             extractHsCtxtTyNames ctxt)
-                            (hsTyVarNames tvs)
-    get (ConDecl n _ tvs ctxt (RecCon fields) _)
-               -- Even if the constructor isn't mentioned, the fields
-               -- might be, as selectors.  They can't mention existentially
-               -- bound tyvars (typechecker checks for that) so no need for 
-               -- the deleteListFromNameSet part
-       = foldr (plusFV . get_field) emptyFVs fields
-       
-    get other_con = emptyFVs
-
-    get_details (VanillaCon tys) = plusFVs (map get_bang tys)
-    get_details (InfixCon t1 t2) = get_bang t1 `plusFV` get_bang t2
-    get_details (RecCon fields)  = plusFVs [get_bang t | (_, t) <- fields]
-
-    get_field (fs,t) | any (`elemNameSet` source_fvs) fs = get_bang t
-                    | otherwise                         = emptyFVs
-
-    get_bang bty = extractHsTyNames (getBangType bty)
-\end{code}
-
-@getWiredInGates@ is just like @getGates@, but it sees a wired-in @Name@
-rather than a declaration.
-
-\begin{code}
-getWiredInGates :: Name -> FreeVars
-getWiredInGates name   -- No classes are wired in
-  = case lookupNameEnv wiredInThingEnv name of
-       Just (AnId the_id) -> getWiredInGates_s (namesOfType (idType the_id))
-
-       Just (ATyCon tc)
-         |  isSynTyCon tc
-         -> getWiredInGates_s (delListFromNameSet (namesOfType ty) (map getName tyvars))
-         where
-            (tyvars,ty)  = getSynTyConDefn tc
-
-       other -> unitFV name
-
-getWiredInGates_s names = foldr (plusFV . getWiredInGates) emptyFVs (nameSetToList names)
-\end{code}
-
-\begin{code}
-getInstDeclGates (InstD (InstDecl inst_ty _ _ _ _)) = extractHsTyNames inst_ty
-getInstDeclGates other                             = emptyFVs
+isOrphanDecl _ _  = False
 \end{code}
 
 
@@ -571,39 +299,31 @@ getInstDeclGates other                                = emptyFVs
 \begin{code}
 fixitiesFromLocalDecls :: GlobalRdrEnv -> [RdrNameHsDecl] -> RnMG LocalFixityEnv
 fixitiesFromLocalDecls gbl_env decls
-  = doptRn Opt_WarnUnusedBinds                           `thenRn` \ warn_unused ->
-    foldlRn (getFixities warn_unused) emptyNameEnv decls  `thenRn` \ env -> 
-    traceRn (text "fixity env" <+> vcat (map ppr (nameEnvElts env)))
-                                                         `thenRn_`
+  = foldlRn getFixities emptyNameEnv decls                             `thenRn` \ env -> 
+    traceRn (text "fixity env" <+> vcat (map ppr (nameEnvElts env)))   `thenRn_`
     returnRn env
   where
-    getFixities :: Bool -> LocalFixityEnv -> RdrNameHsDecl -> RnMG LocalFixityEnv
-    getFixities warn_uu acc (FixD fix)
-      = fix_decl warn_uu acc fix
+    getFixities :: LocalFixityEnv -> RdrNameHsDecl -> RnMG LocalFixityEnv
+    getFixities acc (FixD fix)
+      = fix_decl acc fix
 
-    getFixities warn_uu acc (TyClD (ClassDecl _ _ _ _ sigs _ _ _ ))
-      = foldlRn (fix_decl warn_uu) acc [sig | FixSig sig <- sigs]
+    getFixities acc (TyClD (ClassDecl _ _ _ _ sigs _ _ _ ))
+      = foldlRn fix_decl acc [sig | FixSig sig <- sigs]
                -- Get fixities from class decl sigs too.
-    getFixities warn_uu acc other_decl
+    getFixities acc other_decl
       = returnRn acc
 
-    fix_decl warn_uu acc sig@(FixitySig rdr_name fixity loc)
+    fix_decl acc sig@(FixitySig rdr_name fixity loc)
        =       -- Check for fixity decl for something not declared
          pushSrcLocRn loc                      $
-         lookupGlobalRn gbl_env rdr_name       `thenRn` \  maybe_name ->
-         case maybe_name of {
-           Nothing ->  checkRn (not warn_uu) (unusedFixityDecl rdr_name fixity)        `thenRn_` 
-                       returnRn acc ;
-
-           Just name ->
+         lookupSrcName gbl_env rdr_name        `thenRn` \ name ->
 
                -- Check for duplicate fixity decl
-         case lookupNameEnv acc name of {
-           Just (FixitySig _ _ loc') -> addErrRn (dupFixityDecl rdr_name loc loc')
-                                        `thenRn_` returnRn acc ;
+         case lookupNameEnv acc name of
+           Just (FixitySig _ _ loc') -> addErrRn (dupFixityDecl rdr_name loc loc')     `thenRn_`
+                                        returnRn acc ;
 
-           Nothing -> returnRn (extendNameEnv acc name (FixitySig name fixity loc))
-         }}
+           Nothing                   -> returnRn (extendNameEnv acc name (FixitySig name fixity loc))
 \end{code}
 
 
@@ -632,11 +352,9 @@ rnDeprecs gbl_env Nothing decls
     returnRn (DeprecSome (mkNameEnv (catMaybes pairs)))
  where
    rn_deprec (Deprecation rdr_name txt loc)
-     = pushSrcLocRn loc                        $
-       lookupGlobalRn gbl_env rdr_name `thenRn` \ maybe_name ->
-       case maybe_name of
-        Just n  -> returnRn (Just (n,txt))
-        Nothing -> returnRn Nothing
+     = pushSrcLocRn loc                                $
+       lookupSrcName gbl_env rdr_name          `thenRn` \ name ->
+       returnRn (Just (name, (name,txt)))
 \end{code}
 
 
@@ -647,53 +365,61 @@ rnDeprecs gbl_env Nothing decls
 %************************************************************************
 
 \begin{code}
-checkOldIface :: DynFlags -> Finder
+checkOldIface :: DynFlags
              -> HomeIfaceTable -> HomeSymbolTable
              -> PersistentCompilerState
-             -> Module 
+             -> FilePath
              -> Bool                   -- Source unchanged
              -> Maybe ModIface         -- Old interface from compilation manager, if any
              -> IO (PersistentCompilerState, Bool, (RecompileRequired, Maybe ModIface))
                                -- True <=> errors happened
 
-checkOldIface dflags finder hit hst pcs mod source_unchanged maybe_iface
-  = initRn dflags finder hit hst pcs mod $
-       
-       -- Load the old interface file, if we havn't already got it
-    loadOldIface mod maybe_iface                       `thenRn` \ maybe_iface ->
-
-       -- Check versions
-    recompileRequired mod source_unchanged maybe_iface `thenRn` \ recompile ->
-
-    returnRn (recompile, maybe_iface)
+checkOldIface dflags hit hst pcs iface_path source_unchanged maybe_iface
+  = runRn dflags hit hst pcs (panic "Bogus module") $
+    case maybe_iface of
+       Just old_iface -> -- Use the one we already have
+                         setModuleRn (mi_module old_iface) (check_versions old_iface)
+
+       Nothing -- try and read it from a file
+          -> readIface iface_path      `thenRn` \ read_result ->
+             case read_result of
+               Left err -> -- Old interface file not found, or garbled; give up
+                          traceRn (text "Bad old interface file" $$ nest 4 err)        `thenRn_`
+                          returnRn (outOfDate, Nothing)
+
+               Right parsed_iface
+                      -> setModuleRn (pi_mod parsed_iface) $
+                         loadOldIface parsed_iface `thenRn` \ m_iface ->
+                         check_versions m_iface
+    where
+       check_versions :: ModIface -> RnMG (RecompileRequired, Maybe ModIface)
+       check_versions iface
+          = -- Check versions
+            recompileRequired iface_path source_unchanged iface
+                                                       `thenRn` \ recompile ->
+            returnRn (recompile, Just iface)
 \end{code}
 
+I think the following function should now have a more representative name,
+but what?
 
 \begin{code}
-loadOldIface :: Module -> Maybe ModIface -> RnMG (Maybe ModIface)
-loadOldIface mod (Just iface) 
-  = returnRn (Just iface)
+loadOldIface :: ParsedIface -> RnMG ModIface
 
-loadOldIface mod Nothing
-  =    -- LOAD THE OLD INTERFACE FILE
-    findAndReadIface doc_str (moduleName mod) False {- Not hi-boot -}  `thenRn` \ read_result ->
-    case read_result of {
-       Left err ->     -- Old interface file not found, or garbled, so we'd better bail out
-                   traceRn (vcat [ptext SLIT("No old interface file:"), err])  `thenRn_`
-                   returnRn Nothing ;
-
-       Right (_, iface) ->
-
-       -- RENAME IT
+loadOldIface parsed_iface
+  = let iface = parsed_iface 
+        mod = pi_mod iface
+    in
     initIfaceRnMS mod (
        loadHomeDecls (pi_decls iface)  `thenRn` \ decls ->
        loadHomeRules (pi_rules iface)  `thenRn` \ rules -> 
        loadHomeInsts (pi_insts iface)  `thenRn` \ insts ->
        returnRn (decls, rules, insts)
-    )                          `thenRn` \ ((decls_vers, new_decls), (rule_vers, new_rules), new_insts) ->
+    )  
+       `thenRn` \ ((decls_vers, new_decls), (rule_vers, new_rules), new_insts) ->
 
     mapRn loadHomeUsage        (pi_usages iface)       `thenRn` \ usages ->
-    loadExports        (pi_exports iface)      `thenRn` \ (export_vers, avails) ->
+    loadExports         (pi_exports iface)     `thenRn` \ (export_vers, avails) ->
     loadFixDecls mod   (pi_fixity iface)       `thenRn` \ fix_env ->
     loadDeprecs mod    (pi_deprecs iface)      `thenRn` \ deprec_env ->
     let
@@ -702,24 +428,17 @@ loadOldIface mod Nothing
                                vers_rules   = rule_vers,
                                vers_decls   = decls_vers }
 
-       decls = IfaceDecls { dcl_tycl = new_decls,
-                            dcl_rules = new_rules,
-                            dcl_insts = new_insts }
+       decls = mkIfaceDecls new_decls new_rules new_insts
 
        mod_iface = ModIface { mi_module = mod, mi_version = version,
-                              mi_exports = avails, mi_orphan = pi_orphan iface,
+                              mi_exports = avails, mi_usages  = usages,
+                              mi_boot = False, mi_orphan = pi_orphan iface, 
                               mi_fixities = fix_env, mi_deprecs = deprec_env,
-                              mi_usages  = usages,
                               mi_decls   = decls,
                               mi_globals = panic "No mi_globals in old interface"
                    }
     in
-    returnRn (Just mod_iface)
-    }
-
-    
-  where
-    doc_str = ptext SLIT("need usage info from") <+> ppr mod
+    returnRn mod_iface
 \end{code}
 
 \begin{code}
@@ -731,21 +450,20 @@ loadHomeDecl :: (NameEnv Version, [RenamedTyClDecl])
             -> (Version, RdrNameTyClDecl)
             -> RnMS (NameEnv Version, [RenamedTyClDecl])
 loadHomeDecl (version_map, decls) (version, decl)
-  = rnTyClDecl decl    `thenRn` \ (decl', _) ->
+  = rnTyClDecl decl    `thenRn` \ decl' ->
     returnRn (extendNameEnv version_map (tyClDeclName decl') version, decl':decls)
 
 ------------------
 loadHomeRules :: (Version, [RdrNameRuleDecl])
              -> RnMS (Version, [RenamedRuleDecl])
 loadHomeRules (version, rules)
-  = mapAndUnzipRn rnRuleDecl rules     `thenRn` \ (rules', _) ->
+  = mapRn rnIfaceRuleDecl rules        `thenRn` \ rules' ->
     returnRn (version, rules')
 
 ------------------
 loadHomeInsts :: [RdrNameInstDecl]
              -> RnMS [RenamedInstDecl]
-loadHomeInsts insts = mapAndUnzipRn rnInstDecl insts   `thenRn` \ (insts', _) ->
-                     returnRn insts'
+loadHomeInsts insts = mapRn rnInstDecl insts
 
 ------------------
 loadHomeUsage :: ImportVersion OccName
@@ -763,6 +481,41 @@ loadHomeUsage (mod_name, orphans, is_boot, whats_imported)
 \end{code}
 
 
+
+%*********************************************************
+%*                                                      *
+\subsection{Closing up the interface decls}
+%*                                                      *
+%*********************************************************
+
+Suppose we discover we don't need to recompile.   Then we start from the
+IfaceDecls in the ModIface, and fluff them up by sucking in all the decls they need.
+
+\begin{code}
+closeIfaceDecls :: DynFlags
+               -> HomeIfaceTable -> HomeSymbolTable
+               -> PersistentCompilerState
+               -> ModIface     -- Get the decls from here
+               -> IO (PersistentCompilerState, Bool, [RenamedHsDecl])
+                               -- True <=> errors happened
+closeIfaceDecls dflags hit hst pcs
+               mod_iface@(ModIface { mi_module = mod, mi_decls = iface_decls })
+  = runRn dflags hit hst pcs mod $
+
+    let
+       rule_decls = dcl_rules iface_decls
+       inst_decls = dcl_insts iface_decls
+       tycl_decls = dcl_tycl  iface_decls
+       decls = map RuleD rule_decls ++
+               map InstD inst_decls ++
+               map TyClD tycl_decls
+       needed = unionManyNameSets (map ruleDeclFVs rule_decls) `unionNameSets`
+                unionManyNameSets (map instDeclFVs inst_decls) `unionNameSets`
+                unionManyNameSets (map tyClDeclFVs tycl_decls)
+    in
+    closeDecls decls needed
+\end{code}
+
 %*********************************************************
 %*                                                      *
 \subsection{Unused names}
@@ -770,24 +523,32 @@ loadHomeUsage (mod_name, orphans, is_boot, whats_imported)
 %*********************************************************
 
 \begin{code}
-reportUnusedNames :: ModuleName -> [ModuleName] 
-                 -> GlobalRdrEnv -> AvailEnv
-                 -> Avails -> NameSet -> [RenamedHsDecl] 
+reportUnusedNames :: ModIface -> PrintUnqualified
+                 -> [RdrNameImportDecl] 
+                 -> AvailEnv
+                 -> NameSet            -- Used in this module
+                 -> Avails             -- Exported by this module
+                 -> [RenamedHsDecl] 
                  -> RnMG ()
-reportUnusedNames mod_name direct_import_mods 
-                 gbl_env avail_env 
-                 export_avails mentioned_names
-                 imported_decls
+reportUnusedNames my_mod_iface unqual imports avail_env 
+                 source_fvs export_avails imported_decls
   = warnUnusedModules unused_imp_mods                          `thenRn_`
     warnUnusedLocalBinds bad_locals                            `thenRn_`
     warnUnusedImports bad_imp_names                            `thenRn_`
-    printMinimalImports mod_name minimal_imports               `thenRn_`
-    warnDeprecations really_used_names                         `thenRn_`
-    returnRn ()
+    printMinimalImports this_mod unqual minimal_imports                `thenRn_`
+    warnDeprecations this_mod export_avails my_deprecs 
+                    really_used_names
 
   where
-    used_names = mentioned_names `unionNameSets` availsToNameSet export_avails
+    this_mod   = mi_module my_mod_iface
+    gbl_env    = mi_globals my_mod_iface
+    my_deprecs = mi_deprecs my_mod_iface
     
+       -- The export_fvs make the exported names look just as if they
+       -- occurred in the source program.  
+    export_fvs = availsToNameSet export_avails
+    used_names = source_fvs `plusFV` export_fvs
+
     -- Now, a use of C implies a use of T,
     -- if C was brought into scope by T(..) or T(C)
     really_used_names = used_names `unionNameSets`
@@ -807,17 +568,23 @@ reportUnusedNames mod_name direct_import_mods
                                        other              -> Nothing]
            ]
     
-    defined_names, defined_and_used, defined_but_not_used :: [(Name,Provenance)]
-    defined_names                           = concat (rdrEnvElts gbl_env)
+       -- Collect the defined names from the in-scope environment
+       -- Look for the qualified ones only, else get duplicates
+    defined_names :: [(Name,Provenance)]
+    defined_names = foldRdrEnv add [] gbl_env
+    add rdr_name ns acc | isQual rdr_name = ns ++ acc
+                       | otherwise       = acc
+
+    defined_and_used, defined_but_not_used :: [(Name,Provenance)]
     (defined_and_used, defined_but_not_used) = partition used defined_names
-    used (name,_)                           = not (name `elemNameSet` really_used_names)
+    used (name,_)                           = name `elemNameSet` really_used_names
     
     -- Filter out the ones only defined implicitly
     bad_locals :: [Name]
     bad_locals     = [n     | (n,LocalDef) <- defined_but_not_used]
     
     bad_imp_names :: [(Name,Provenance)]
-    bad_imp_names  = [(n,p) | (n,p@(NonLocalDef (UserImport mod _ True) _)) <- defined_but_not_used,
+    bad_imp_names  = [(n,p) | (n,p@(NonLocalDef (UserImport mod _ True))) <- defined_but_not_used,
                              not (module_unused mod)]
     
     -- inst_mods are directly-imported modules that 
@@ -850,9 +617,9 @@ reportUnusedNames mod_name direct_import_mods
     minimal_imports1 = foldr add_name     minimal_imports0 defined_and_used
     minimal_imports  = foldr add_inst_mod minimal_imports1 inst_mods
     
-    add_name (n,NonLocalDef (UserImport m _ _) _) acc = addToFM_C plusAvailEnv acc (moduleName (nameModule n))
-                                                                 (unitAvailEnv (mk_avail n))
-    add_name (n,other_prov)                      acc = acc
+    add_name (n,NonLocalDef (UserImport m _ _)) acc = addToFM_C plusAvailEnv acc (moduleName (nameModule n))
+                                                               (unitAvailEnv (mk_avail n))
+    add_name (n,other_prov)                    acc = acc
 
     mk_avail n = case lookupNameEnv avail_env n of
                Just (AvailTC m _) | n==m      -> AvailTC n [n]
@@ -865,7 +632,10 @@ reportUnusedNames mod_name direct_import_mods
       | otherwise      = addToFM acc m emptyAvailEnv
        -- Add an empty collection of imports for a module
        -- from which we have sucked only instance decls
-    
+   
+    direct_import_mods :: [ModuleName]
+    direct_import_mods = nub [m | ImportDecl m _ _ _ _ _ <- imports]
+
     -- unused_imp_mods are the directly-imported modules 
     -- that are not mentioned in minimal_imports
     unused_imp_mods = [m | m <- direct_import_mods,
@@ -875,13 +645,17 @@ reportUnusedNames mod_name direct_import_mods
     module_unused :: Module -> Bool
     module_unused mod = moduleName mod `elem` unused_imp_mods
 
-
-warnDeprecations used_names
+warnDeprecations this_mod export_avails my_deprecs used_names
   = doptRn Opt_WarnDeprecations                                `thenRn` \ warn_drs ->
     if not warn_drs then returnRn () else
 
-    getIfacesRn                                                `thenRn` \ ifaces ->
-    getHomeIfaceTableRn                                        `thenRn` \ hit ->
+       -- The home modules for things in the export list
+       -- may not have been loaded yet; do it now, so 
+       -- that we can see their deprecations, if any
+    mapRn_ load_home export_mods               `thenRn_`
+
+    getIfacesRn                                        `thenRn` \ ifaces ->
+    getHomeIfaceTableRn                                `thenRn` \ hit ->
     let
        pit     = iPIT ifaces
        deprecs = [ (n,txt)
@@ -891,27 +665,33 @@ warnDeprecations used_names
     mapRn_ warnDeprec deprecs
 
   where
+    export_mods = nub [ moduleName (nameModule name) 
+                     | avail <- export_avails,
+                       let name = availName avail,
+                       not (nameIsLocalOrFrom this_mod name) ]
+  
+    load_home m = loadInterface (text "Check deprecations for" <+> ppr m) m ImportBySystem
+
     lookup_deprec hit pit n
-       = case lookupModuleEnv hit mod of
-               Just iface -> lookupDeprec iface n
-               Nothing    -> case lookupModuleEnv pit mod of
-                               Just iface -> lookupDeprec iface n
-                               Nothing    -> pprPanic "warnDeprecations:" (ppr n)
-       where
-         mod = nameModule n
+       | nameIsLocalOrFrom this_mod n
+       = lookupDeprec my_deprecs n 
+       | otherwise
+       = case lookupIface hit pit this_mod n of
+               Just iface -> lookupDeprec (mi_deprecs iface) n
+               Nothing    -> pprPanic "warnDeprecations:" (ppr n)
 
 -- ToDo: deal with original imports with 'qualified' and 'as M' clauses
-printMinimalImports mod_name imps
+printMinimalImports this_mod unqual imps
   = doptRn Opt_D_dump_minimal_imports          `thenRn` \ dump_minimal ->
     if not dump_minimal then returnRn () else
 
     mapRn to_ies (fmToList imps)               `thenRn` \ mod_ies ->
     ioToRnM (do { h <- openFile filename WriteMode ;
-                 printForUser h (vcat (map ppr_mod_ie mod_ies))
+                 printForUser h unqual (vcat (map ppr_mod_ie mod_ies))
        })                                      `thenRn_`
     returnRn ()
   where
-    filename = moduleNameUserString mod_name ++ ".imports"
+    filename = moduleNameUserString (moduleName this_mod) ++ ".imports"
     ppr_mod_ie (mod_name, ies) 
        | mod_name == pRELUDE_Name 
        = empty
@@ -926,28 +706,37 @@ printMinimalImports mod_name imps
     to_ie (Avail n)       = returnRn (IEVar n)
     to_ie (AvailTC n [m]) = ASSERT( n==m ) 
                            returnRn (IEThingAbs n)
-    to_ie (AvailTC n ns)  = getInterfaceExports (moduleName (nameModule n)) 
-                                               ImportBySystem          `thenRn` \ (_, avails) ->
-                           case [ms | AvailTC m ms <- avails, m == n] of
-                             [ms] | all (`elem` ns) ms -> returnRn (IEThingAll n)
-                                  | otherwise          -> returnRn (IEThingWith n (filter (/= n) ns))
-                             other -> pprTrace "to_ie" (ppr n <+> ppr (nameModule n) <+> ppr other) $
-                                      returnRn (IEVar n)
+    to_ie (AvailTC n ns)  
+       = getInterfaceExports n_mod ImportBySystem              `thenRn` \ (_, avails_by_module) ->
+         case [xs | (m,as) <- avails_by_module,
+                    m == n_mod,
+                    AvailTC x xs <- as, 
+                    x == n] of
+             [xs] | all (`elem` ns) xs -> returnRn (IEThingAll n)
+                  | otherwise          -> returnRn (IEThingWith n (filter (/= n) ns))
+             other                     -> pprTrace "to_ie" (ppr n <+> ppr (nameModule n) <+> ppr other) $
+                                          returnRn (IEVar n)
+       where
+         n_mod = moduleName (nameModule n)
 
 rnDump  :: [RenamedHsDecl]     -- Renamed imported decls
        -> [RenamedHsDecl]      -- Renamed local decls
-       -> RnMG (IO ())
+       -> RnMG ()
 rnDump imp_decls local_decls
-   = doptRn Opt_D_dump_rn_trace        `thenRn` \ dump_rn_trace ->
-     doptRn Opt_D_dump_rn_stats        `thenRn` \ dump_rn_stats ->
-     doptRn Opt_D_dump_rn              `thenRn` \ dump_rn ->
-     if dump_rn_trace || dump_rn_stats || dump_rn then
-       getRnStats imp_decls            `thenRn` \ stats_msg ->
-       returnRn (printErrs stats_msg >> 
-                 dumpIfSet dump_rn "Renamer:" 
-                           (vcat (map ppr (local_decls ++ imp_decls))))
-     else
-       returnRn (return ())
+  = doptRn Opt_D_dump_rn_trace         `thenRn` \ dump_rn_trace ->
+    doptRn Opt_D_dump_rn_stats         `thenRn` \ dump_rn_stats ->
+    doptRn Opt_D_dump_rn       `thenRn` \ dump_rn ->
+    getIfacesRn                        `thenRn` \ ifaces ->
+
+    ioToRnM (do { dumpIfSet (dump_rn_trace || dump_rn_stats || dump_rn)
+                           "Renamer statistics"
+                           (getRnStats imp_decls ifaces) ;
+
+                 dumpIfSet dump_rn "Renamer:" 
+                           (vcat (map ppr (local_decls ++ imp_decls)))
+    })                         `thenRn_`
+
+    returnRn ()
 \end{code}
 
 
@@ -958,60 +747,37 @@ rnDump imp_decls local_decls
 %*********************************************************
 
 \begin{code}
-getRnStats :: [RenamedHsDecl] -> RnMG SDoc
-getRnStats imported_decls
-  = getIfacesRn                `thenRn` \ ifaces ->
-    let
-       n_mods = length [() | (_, _, True) <- eltsFM (iImpModInfo ifaces)]
-
-       decls_read     = [decl | (avail, True, (_,decl)) <- nameEnvElts (iDecls ifaces),
-                               -- Data, newtype, and class decls are in the decls_fm
-                               -- under multiple names; the tycon/class, and each
-                               -- constructor/class op too.
-                               -- The 'True' selects just the 'main' decl
-                                not (isLocallyDefined (availName avail))
-                            ]
-
-       (cd_rd, dd_rd, nd_rd, sd_rd, vd_rd)        = countTyClDecls decls_read
-       (cd_sp, dd_sp, nd_sp, sd_sp, vd_sp, id_sp) = count_decls imported_decls
-
-       unslurped_insts       = iInsts ifaces
-       inst_decls_unslurped  = length (bagToList unslurped_insts)
-       inst_decls_read       = id_sp + inst_decls_unslurped
-
-       stats = vcat 
-               [int n_mods <+> text "interfaces read",
-                hsep [ int cd_sp, text "class decls imported, out of", 
-                       int cd_rd, text "read"],
-                hsep [ int dd_sp, text "data decls imported, out of",  
-                       int dd_rd, text "read"],
-                hsep [ int nd_sp, text "newtype decls imported, out of",  
-                       int nd_rd, text "read"],
-                hsep [int sd_sp, text "type synonym decls imported, out of",  
-                       int sd_rd, text "read"],
-                hsep [int vd_sp, text "value signatures imported, out of",  
-                       int vd_rd, text "read"],
-                hsep [int id_sp, text "instance decls imported, out of",  
-                       int inst_decls_read, text "read"],
-                text "cls dcls slurp" <+> fsep (map (ppr . tyClDeclName) 
-                                          [d | TyClD d <- imported_decls, isClassDecl d]),
-                text "cls dcls read"  <+> fsep (map (ppr . tyClDeclName) 
-                                          [d | d <- decls_read, isClassDecl d])]
-    in
-    returnRn (hcat [text "Renamer stats: ", stats])
-
-count_decls decls
-  = (class_decls, 
-     data_decls, 
-     newtype_decls,
-     syn_decls, 
-     val_decls, 
-     inst_decls)
+getRnStats :: [RenamedHsDecl] -> Ifaces -> SDoc
+getRnStats imported_decls ifaces
+  = hcat [text "Renamer stats: ", stats]
   where
-    tycl_decls = [d | TyClD d <- decls]
-    (class_decls, data_decls, newtype_decls, syn_decls, val_decls) = countTyClDecls tycl_decls
+    n_mods = length [() | _ <- moduleEnvElts (iPIT ifaces)]
+       -- This is really only right for a one-shot compile
 
-    inst_decls    = length [() | InstD _  <- decls]
+    (decls_map, n_decls_slurped) = iDecls ifaces
+    
+    n_decls_left   = length [decl | (avail, True, (_,decl)) <- nameEnvElts decls_map
+                       -- Data, newtype, and class decls are in the decls_fm
+                       -- under multiple names; the tycon/class, and each
+                       -- constructor/class op too.
+                       -- The 'True' selects just the 'main' decl
+                    ]
+    
+    (insts_left, n_insts_slurped) = iInsts ifaces
+    n_insts_left  = length (bagToList insts_left)
+    
+    (rules_left, n_rules_slurped) = iRules ifaces
+    n_rules_left  = length (bagToList rules_left)
+    
+    stats = vcat 
+       [int n_mods <+> text "interfaces read",
+        hsep [ int n_decls_slurped, text "type/class/variable imported, out of", 
+               int (n_decls_slurped + n_decls_left), text "read"],
+        hsep [ int n_insts_slurped, text "instance decls imported, out of",  
+               int (n_insts_slurped + n_insts_left), text "read"],
+        hsep [ int n_rules_slurped, text "rule decls imported, out of",  
+               int (n_rules_slurped + n_rules_left), text "read"]
+       ]
 \end{code}    
 
 
@@ -1030,9 +796,6 @@ warnDeprec (name, txt)
           text "is deprecated:", nest 4 (ppr txt) ]
 
 
-unusedFixityDecl rdr_name fixity
-  = hsep [ptext SLIT("Unused fixity declaration for"), quotes (ppr rdr_name)]
-
 dupFixityDecl rdr_name loc1 loc2
   = vcat [ptext SLIT("Multiple fixity declarations for") <+> quotes (ppr rdr_name),
          ptext SLIT("at ") <+> ppr loc1,
@@ -1041,6 +804,10 @@ dupFixityDecl rdr_name loc1 loc2
 badDeprec d
   = sep [ptext SLIT("Illegal deprecation when whole module is deprecated"),
         nest 4 (ppr d)]
+
+noMainErr
+  = hsep [ptext SLIT("Module"), quotes (ppr mAIN_Name), 
+         ptext SLIT("must include a definition for"), quotes (ptext SLIT("main"))]
 \end{code}