[project @ 2000-10-23 16:43:42 by simonpj]
[ghc-hetmet.git] / ghc / compiler / rename / Rename.lhs
index 3f27194..8790ef0 100644 (file)
@@ -9,171 +9,202 @@ module Rename ( renameModule ) where
 #include "HsVersions.h"
 
 import HsSyn
-import RdrHsSyn                ( RdrNameHsModule )
+import HsPragmas       ( DataPragmas(..) )
+import RdrHsSyn                ( RdrNameHsModule, RdrNameHsDecl, RdrNameDeprecation )
 import RnHsSyn         ( RenamedHsModule, RenamedHsDecl, 
                          extractHsTyNames, extractHsCtxtTyNames
                        )
 
-import CmdLineOpts     ( opt_HiMap, opt_D_dump_rn_trace, opt_D_dump_minimal_imports,
-                         opt_D_dump_rn, opt_D_dump_rn_stats, opt_WarnDeprecations
-                       )
+import CmdLineOpts     ( DynFlags, DynFlag(..) )
 import RnMonad
 import RnNames         ( getGlobalNames )
 import RnSource                ( rnSourceDecls, rnDecl )
-import RnIfaces                ( getImportedInstDecls, importDecl, getImportVersions, getInterfaceExports,
-                         getImportedRules, loadHomeInterface, getSlurped, removeContext
+import RnIfaces                ( getImportedInstDecls, importDecl, mkImportInfo, 
+                         getInterfaceExports,
+                         getImportedRules, getSlurped, removeContext,
+                         ImportDeclResult(..), findAndReadIface
                        )
-import RnEnv           ( availName, availsToNameSet, unitAvailEnv, availEnvElts, plusAvailEnv, 
-                         warnUnusedImports, warnUnusedLocalBinds, lookupImplicitOccRn, pprAvail,
-                         FreeVars, plusFVs, plusFV, unitFV, emptyFVs, isEmptyFVs
+import RnEnv           ( availName, availsToNameSet, 
+                         emptyAvailEnv, unitAvailEnv, availEnvElts, plusAvailEnv, sortAvails,
+                         warnUnusedImports, warnUnusedLocalBinds, warnUnusedModules,
+                         lookupOrigNames, unknownNameErr,
+                         FreeVars, plusFVs, plusFV, unitFV, emptyFVs, isEmptyFVs, addOneFV
                        )
 import Module           ( Module, ModuleName, WhereFrom(..),
-                         moduleNameUserString, mkSearchPath, moduleName, mkThisModule
+                         moduleNameUserString, moduleName, mkModuleInThisPackage,
+                         lookupModuleEnv
                        )
 import Name            ( Name, isLocallyDefined, NamedThing(..), getSrcLoc,
-                         nameOccName, nameUnique, nameModule, maybeUserImportedFrom,
-                         isUserImportedExplicitlyName, isUserImportedName,
-                         maybeWiredInTyConName, maybeWiredInIdName, isWiredInName
+                         nameOccName, nameUnique, nameModule,
+                         isUserExportedName, toRdrName,
+                         mkNameEnv, nameEnvElts, extendNameEnv
                        )
 import OccName         ( occNameFlavour, isValOcc )
 import Id              ( idType )
 import TyCon           ( isSynTyCon, getSynTyConDefn )
 import NameSet
-import PrelMods                ( mAIN_Name, pREL_MAIN_Name, pRELUDE_Name )
 import TysWiredIn      ( unitTyCon, intTyCon, doubleTyCon, boolTyCon )
-import PrelInfo                ( ioTyCon_NAME, thinAirIdNames, fractionalClassKeys, derivingOccurrences )
+import PrelNames       ( mAIN_Name, pREL_MAIN_Name, pRELUDE_Name,
+                         ioTyCon_RDR,
+                         unpackCString_RDR, unpackCStringFoldr_RDR, unpackCStringUtf8_RDR,
+                         eqString_RDR
+                       )
+import PrelInfo                ( fractionalClassKeys, derivingOccurrences, wiredInThingEnv )
 import Type            ( namesOfType, funTyCon )
 import ErrUtils                ( printErrorsAndWarnings, dumpIfSet, ghcExit )
-import BasicTypes      ( NewOrData(..) )
+import BasicTypes      ( Version, initialVersion )
 import Bag             ( isEmptyBag, bagToList )
-import FiniteMap       ( FiniteMap, eltsFM, fmToList, emptyFM, addToFM_C )
+import FiniteMap       ( FiniteMap, eltsFM, fmToList, emptyFM, lookupFM, 
+                         addToFM_C, elemFM, addToFM
+                       )
 import UniqSupply      ( UniqSupply )
 import UniqFM          ( lookupUFM )
-import Maybes          ( maybeToBool )
+import SrcLoc          ( noSrcLoc )
+import Maybes          ( maybeToBool, expectJust )
 import Outputable
 import IO              ( openFile, IOMode(..) )
+import HscTypes                ( Finder, PersistentCompilerState, HomeIfaceTable, HomeSymbolTable, 
+                         ModIface(..), TyThing(..),
+                         GlobalRdrEnv, AvailEnv, Avails, GenAvailInfo(..), AvailInfo, 
+                         Provenance(..), pprNameProvenance, ImportReason(..),
+                         lookupDeprec
+                        )
+import List            ( partition, nub )
 \end{code}
 
 
 
 \begin{code}
-renameModule :: UniqSupply
-            -> RdrNameHsModule
-            -> IO (Maybe 
-                     ( Module
-                     , RenamedHsModule   -- Output, after renaming
-                     , InterfaceDetails  -- Interface; for interface file generation
-                     , RnNameSupply      -- Final env; for renaming derivings
-                     , [ModuleName]      -- Imported modules; for profiling
-                     ))
-
-renameModule us this_mod@(HsModule mod_name vers exports imports local_decls _ loc)
+renameModule :: DynFlags -> Finder 
+            -> HomeIfaceTable -> HomeSymbolTable
+            -> PersistentCompilerState 
+            -> Module -> RdrNameHsModule 
+            -> IO (PersistentCompilerState, Maybe ModIface)
+                       -- The mi_decls in the ModIface include
+                       -- ones imported from packages too
+
+renameModule dflags finder hit hst old_pcs this_module 
+            this_mod@(HsModule _ _ _ _ _ _ loc)
   =    -- Initialise the renamer monad
-    initRn mod_name us (mkSearchPath opt_HiMap) loc
-          (rename this_mod)                            >>=
-       \ ((maybe_rn_stuff, dump_action), rn_errs_bag, rn_warns_bag) ->
+    do {
+       ((maybe_rn_stuff, dump_action), (rn_warns_bag, rn_errs_bag), new_pcs) 
+          <- initRn dflags finder hit hst old_pcs this_module loc (rename this_module this_mod) ;
 
        -- Check for warnings
-    printErrorsAndWarnings rn_errs_bag rn_warns_bag    >>
+       printErrorsAndWarnings (rn_warns_bag, rn_errs_bag) ;
 
        -- Dump any debugging output
-    dump_action                                        >>
+       dump_action ;
 
        -- Return results
-    if not (isEmptyBag rn_errs_bag) then
-           ghcExit 1 >> return Nothing
-    else
-           return maybe_rn_stuff
+       if not (isEmptyBag rn_errs_bag) then
+           return (old_pcs, Nothing)
+        else
+           return (new_pcs, maybe_rn_stuff)
+    }
 \end{code}
 
-
 \begin{code}
-rename :: RdrNameHsModule
-       -> RnMG (Maybe (Module, RenamedHsModule, InterfaceDetails, RnNameSupply, [ModuleName]), IO ())
-rename this_mod@(HsModule mod_name vers _ imports local_decls mod_deprec loc)
+rename :: Module -> RdrNameHsModule -> RnMG (Maybe ModIface, 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 ->
 
        -- CHECK FOR EARLY EXIT
-    if not (maybeToBool maybe_stuff) then
-       -- Everything is up to date; no need to recompile further
-       rnDump [] []            `thenRn` \ dump_action ->
-       returnRn (Nothing, dump_action)
-    else
-    let
-       Just (export_env, gbl_env, fixity_env, global_avail_env) = maybe_stuff
-        ExportEnv export_avails _ _ = export_env
-    in
+    case maybe_stuff of {
+       Nothing ->      -- Everything is up to date; no need to recompile further
+               rnDump [] []            `thenRn` \ dump_action ->
+               returnRn (Nothing, dump_action) ;
+
+       Just (gbl_env, local_gbl_env, export_avails, global_avail_env) ->
+
+       -- DEAL WITH DEPRECATIONS
+    rnDeprecs local_gbl_env mod_deprec local_decls     `thenRn` \ my_deprecs ->
+
+       -- DEAL WITH LOCAL FIXITIES
+    fixitiesFromLocalDecls local_gbl_env local_decls   `thenRn` \ local_fixity_env ->
 
        -- RENAME THE SOURCE
-    initRnMS gbl_env fixity_env SourceMode (
+    initRnMS gbl_env local_fixity_env SourceMode (
        rnSourceDecls local_decls
     )                                  `thenRn` \ (rn_local_decls, source_fvs) ->
 
        -- SLURP IN ALL THE NEEDED DECLARATIONS
     implicitFVs mod_name rn_local_decls        `thenRn` \ implicit_fvs -> 
     let
-       real_source_fvs = implicit_fvs `plusFV` source_fvs `plusFV` export_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. 
-
                -- 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
-       export_fvs = mkNameSet (map availName export_avails)
-    in
-    slurpImpDecls real_source_fvs      `thenRn` \ rn_imp_decls ->
-    let
-       rn_all_decls       = rn_local_decls ++ rn_imp_decls
+               -- 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
 
-       -- COLLECT ALL DEPRECATIONS
-       deprec_sigs = [ ds | ValD bnds <- rn_local_decls, ds <- collectDeprecs bnds ]
-       deprecs = case mod_deprec of
-          Nothing -> deprec_sigs
-          Just txt -> Deprecation (IEModuleContents undefined) txt : deprec_sigs
+       slurp_fvs       = implicit_fvs `plusFV` real_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
+    slurpImpDecls slurp_fvs            `thenRn` \ rn_imp_decls ->
 
        -- EXIT IF ERRORS FOUND
+    rnDump rn_imp_decls rn_local_decls         `thenRn` \ dump_action ->
     checkErrsRn                                        `thenRn` \ no_errs_so_far ->
     if not no_errs_so_far then
        -- Found errors already, so exit now
-       rnDump rn_imp_decls rn_all_decls        `thenRn` \ dump_action ->
        returnRn (Nothing, dump_action)
     else
 
        -- GENERATE THE VERSION/USAGE INFO
-    getImportVersions mod_name export_env      `thenRn` \ my_usages ->
-    getNameSupplyRn                            `thenRn` \ name_supply ->
-
-       -- REPORT UNUSED NAMES
-    reportUnusedNames mod_name gbl_env global_avail_env
-                     export_env
-                     source_fvs                        `thenRn_`
+    mkImportInfo mod_name imports      `thenRn` \ my_usages ->
 
        -- RETURN THE RENAMED MODULE
+    getNameSupplyRn                    `thenRn` \ name_supply ->
+    getIfacesRn                        `thenRn` \ ifaces ->
     let
-       has_orphans        = any isOrphanDecl rn_local_decls
-       direct_import_mods = [mod | ImportDecl mod _ _ _ _ _ <- imports]
-       renamed_module = HsModule mod_name vers 
-                                 trashed_exports trashed_imports
-                                 rn_all_decls
-                                 mod_deprec
-                                 loc
+       direct_import_mods :: [ModuleName]
+       direct_import_mods = nub [m | ImportDecl m _ _ _ _ _ <- imports]
+
+               -- *don't* just pick the forward edges.  It's entirely possible
+               -- that a module is only reachable via back edges.
+       user_import ImportByUser = True
+       user_import ImportByUserSource = True
+       user_import _ = False
+
+       -- Export only those fixities that are for names that are
+       --      (a) defined in this module
+       --      (b) exported
+       exported_fixities
+         = mkNameEnv [ (name, fixity)
+                     | FixitySig name fixity loc <- nameEnvElts local_fixity_env,
+                       isUserExportedName name
+                     ]
+
+
+       -- Sort the exports to make them easier to compare for versions
+       my_exports = sortAvails export_avails
+       
+       mod_iface = ModIface {  mi_module   = this_module,
+                               mi_version  = panic "mi_version: not filled in yet",
+                               mi_orphan   = any isOrphanDecl rn_local_decls,
+                               mi_exports  = my_exports,
+                               mi_usages   = my_usages,
+                               mi_fixities = exported_fixities,
+                               mi_deprecs  = my_deprecs,
+                               mi_decls    = rn_local_decls ++ rn_imp_decls
+                   }
     in
-    rnDump rn_imp_decls        rn_all_decls            `thenRn` \ dump_action ->
-    returnRn (Just (mkThisModule mod_name,
-                   renamed_module, 
-                   (InterfaceDetails has_orphans my_usages export_env deprecs),
-                   name_supply,
-                   direct_import_mods), dump_action)
+
+       -- 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_`
+
+    returnRn (Just mod_iface, dump_action) }
   where
     trashed_exports  = {-trace "rnSource:trashed_exports"-} Nothing
     trashed_imports  = {-trace "rnSource:trashed_imports"-} []
-
-    collectDeprecs EmptyBinds = []
-    collectDeprecs (ThenBinds x y) = collectDeprecs x ++ collectDeprecs y
-    collectDeprecs (MonoBind _ sigs _) = [ d | DeprecSig d _ <- sigs ]
 \end{code}
 
 @implicitFVs@ forces the renamer to slurp in some things which aren't
@@ -181,11 +212,9 @@ mentioned explicitly, but which might be needed by the type checker.
 
 \begin{code}
 implicitFVs mod_name decls
-  = mapRn lookupImplicitOccRn implicit_occs    `thenRn` \ implicit_names ->
-    returnRn (implicit_main                            `plusFV` 
-             mkNameSet (map getName default_tycons)    `plusFV`
-             mkNameSet thinAirIdNames                  `plusFV`
-             mkNameSet implicit_names)
+  = lookupOrigNames implicit_occs                      `thenRn` \ implicit_names ->
+    returnRn (mkNameSet (map getName default_tycons)   `plusFV`
+             implicit_names)
   where
        -- Add occurrences for Int, and (), because they
        -- are the types to which ambigious type variables may be defaulted by
@@ -199,15 +228,19 @@ implicitFVs mod_name decls
 
        -- Add occurrences for IO or PrimIO
     implicit_main |  mod_name == mAIN_Name
-                 || mod_name == pREL_MAIN_Name = unitFV ioTyCon_NAME
-                 |  otherwise                  = emptyFVs
+                 || mod_name == pREL_MAIN_Name = [ioTyCon_RDR]
+                 |  otherwise                  = []
 
        -- Now add extra "occurrences" for things that
        -- the deriving mechanism, or defaulting, will later need in order to
        -- generate code
-    implicit_occs = foldr ((++) . get) [] 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]
 
-    get (TyClD (TyData _ _ _ _ _ (Just deriv_classes) _ _))
+    get (TyClD (TyData _ _ _ _ _ _ (Just deriv_classes) _ _ _ _))
        = concat (map get_deriv deriv_classes)
     get other = []
 
@@ -224,7 +257,7 @@ isOrphanDecl (InstD (InstDecl inst_ty _ _ _ _))
        -- The decl is an orphan if Baz and T are both not locally defined,
        --      even if Foo *is* locally defined
 
-isOrphanDecl (RuleD (RuleDecl _ _ _ lhs _ _))
+isOrphanDecl (RuleD (HsRule _ _ _ lhs _ _))
   = check lhs
   where
        -- At the moment we just check for common LHS forms
@@ -233,6 +266,7 @@ isOrphanDecl (RuleD (RuleDecl _ _ _ lhs _ _))
     check (HsVar v)      = not (isLocallyDefined v)
     check (HsApp f a)    = check f && check a
     check (HsLit _)      = False
+    check (HsOverLit _)          = False
     check (OpApp l o _ r) = check l && check o && check r
     check (NegApp e _)    = check e
     check (HsPar e)      = check e
@@ -271,8 +305,13 @@ slurpImpDecls source_fvs
     getSlurped                                 `thenRn` \ source_binders ->
     slurpSourceRefs source_binders source_fvs  `thenRn` \ (decls, needed) ->
 
-       -- And finally get everything else
-    closeDecls 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
@@ -307,7 +346,7 @@ slurpSourceRefs source_binders source_fvs
 
     go_outer decls fvs all_gates refs  -- refs are not necessarily slurped yet
        = traceRn (text "go_outer" <+> ppr refs)                `thenRn_`
-         go_inner decls fvs emptyFVs refs                      `thenRn` \ (decls1, fvs1, gates1) ->
+         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)
@@ -315,39 +354,17 @@ slurpSourceRefs source_binders source_fvs
                -- 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 []
-       = returnRn (decls, fvs, gates)
-
-    go_inner decls fvs gates (wanted_name:refs) 
-       | isWiredInName wanted_name
-       = load_home wanted_name         `thenRn_`
-         go_inner decls fvs (gates `plusFV` getWiredInGates wanted_name) refs
-
-       | otherwise
-       = importDecl wanted_name                `thenRn` \ maybe_decl ->
-         case maybe_decl of
-           Nothing   -> go_inner decls fvs gates refs  -- No declaration... (already slurped, or local)
-           Just decl -> rnIfaceDecl decl               `thenRn` \ (new_decl, fvs1) ->
-                        go_inner (new_decl : decls)
-                                 (fvs1 `plusFV` fvs)
-                                 (gates `plusFV` getGates source_fvs new_decl)
-                                 refs
-
-       -- When we find a wired-in name we must load its
-       -- home module so that we find any instance decls therein
-    load_home name 
-       | name `elemNameSet` source_binders = returnRn ()
-               -- When compiling the prelude, a wired-in thing may
-               -- be defined in this module, in which case we don't
-               -- want to load its home module!
-               -- Using 'isLocallyDefined' doesn't work because some of
-               -- the free variables returned are simply 'listTyCon_Name',
-               -- with a system provenance.  We could look them up every time
-               -- but that seems a waste.
-       | otherwise = loadHomeInterface doc name        `thenRn_`
-                     returnRn ()
-        where
-         doc = ptext SLIT("need home module for wired in thing") <+> ppr name
+    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 -> rnIfaceDecl decl           `thenRn` \ (new_decl, fvs1) ->
+                            returnRn (new_decl : decls, 
+                                      fvs1 `plusFV` fvs,
+                                      gates `plusFV` getGates source_fvs new_decl)
 
 rnInstDecls decls fvs gates []
   = returnRn (decls, fvs, gates)
@@ -377,17 +394,6 @@ closeDecls decls needed
                 
 
 -------------------------------------------------------
-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)      
-                       
-
--------------------------------------------------------
 -- Augment decls with any decls needed by needed.
 -- Return also free vars of the new decls (only)
 slurpDecls decls needed
@@ -399,14 +405,72 @@ slurpDecls decls needed
 
 -------------------------------------------------------
 slurpDecl decls fvs wanted_name
-  = importDecl wanted_name             `thenRn` \ maybe_decl ->
-    case maybe_decl of
-       -- No declaration... (wired in thing)
-       Nothing -> returnRn (decls, fvs)
-
+  = importDecl wanted_name             `thenRn` \ import_result ->
+    case import_result of
        -- Found a declaration... rename it
-       Just decl -> rnIfaceDecl decl           `thenRn` \ (new_decl, fvs1) ->
-                    returnRn (new_decl:decls, fvs1 `plusFV` fvs)
+       HereItIs decl -> rnIfaceDecl decl               `thenRn` \ (new_decl, fvs1) ->
+                        returnRn (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)      
+\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 NoDataPragmas loc
+               name1 name2))
+       -- Nuke the context and constructors
+       -- But retain the *number* of constructors!
+       -- Also the tvs will have kinds on them.
+-}
 \end{code}
 
 
@@ -434,13 +498,13 @@ vars of the source program, and extracts from the decl the gate names.
 getGates source_fvs (SigD (IfaceSig _ ty _ _))
   = extractHsTyNames ty
 
-getGates source_fvs (TyClD (ClassDecl ctxt cls tvs _ sigs _ _ _ _ _ _ _))
+getGates source_fvs (TyClD (ClassDecl ctxt cls tvs _ sigs _ _ _ _ ))
   = (delListFromNameSet (foldr (plusFV . get) (extractHsCtxtTyNames ctxt) sigs)
-                      (map getTyVarName tvs)
+                       (hsTyVarNames tvs)
      `addOneToNameSet` cls)
     `plusFV` maybe_double
   where
-    get (ClassOpSig n _ _ ty _) 
+    get (ClassOpSig n _ ty _) 
        | n `elemNameSet` source_fvs = extractHsTyNames ty
        | otherwise                  = emptyFVs
 
@@ -456,12 +520,12 @@ getGates source_fvs (TyClD (ClassDecl ctxt cls tvs _ sigs _ _ _ _ _ _ _))
 
 getGates source_fvs (TyClD (TySynonym tycon tvs ty _))
   = delListFromNameSet (extractHsTyNames ty)
-                      (map getTyVarName tvs)
+                      (hsTyVarNames tvs)
        -- A type synonym type constructor isn't a "gate" for instance decls
 
-getGates source_fvs (TyClD (TyData _ ctxt tycon tvs cons _ _ _))
+getGates source_fvs (TyClD (TyData _ ctxt tycon tvs cons _ _ _ _ _ _))
   = delListFromNameSet (foldr (plusFV . get) (extractHsCtxtTyNames ctxt) cons)
-                      (map getTyVarName tvs)
+                      (hsTyVarNames tvs)
     `addOneToNameSet` tycon
   where
     get (ConDecl n _ tvs ctxt details _)
@@ -469,7 +533,7 @@ getGates source_fvs (TyClD (TyData _ ctxt tycon tvs cons _ _ _))
                -- If the constructor is method, get fvs from all its fields
        = delListFromNameSet (get_details details `plusFV` 
                              extractHsCtxtTyNames ctxt)
-                            (map getTyVarName tvs)
+                            (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
@@ -482,14 +546,11 @@ getGates source_fvs (TyClD (TyData _ ctxt tycon tvs cons _ _ _))
     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_details (NewCon t _)    = extractHsTyNames t
 
     get_field (fs,t) | any (`elemNameSet` source_fvs) fs = get_bang t
                     | otherwise                         = emptyFVs
 
-    get_bang (Banged   t) = extractHsTyNames t
-    get_bang (Unbanged t) = extractHsTyNames t
-    get_bang (Unpacked t) = extractHsTyNames t
+    get_bang bty = extractHsTyNames (getBangType bty)
 
 getGates source_fvs other_decl = emptyFVs
 \end{code}
@@ -500,17 +561,16 @@ rather than a declaration.
 \begin{code}
 getWiredInGates :: Name -> FreeVars
 getWiredInGates name   -- No classes are wired in
-  | is_id               = getWiredInGates_s (namesOfType (idType the_id))
-  | isSynTyCon the_tycon = getWiredInGates_s
-        (delListFromNameSet (namesOfType ty) (map getName tyvars))
-  | otherwise           = unitFV name
-  where
-    maybe_wired_in_id    = maybeWiredInIdName name
-    is_id               = maybeToBool maybe_wired_in_id
-    maybe_wired_in_tycon = maybeWiredInTyConName name
-    Just the_id         = maybe_wired_in_id
-    Just the_tycon      = maybe_wired_in_tycon
-    (tyvars,ty)         = getSynTyConDefn the_tycon
+  = 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}
@@ -523,82 +583,224 @@ getInstDeclGates other                               = emptyFVs
 
 %*********************************************************
 %*                                                      *
+\subsection{Fixities}
+%*                                                      *
+%*********************************************************
+
+\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_`
+    returnRn env
+  where
+    getFixities :: Bool -> LocalFixityEnv -> RdrNameHsDecl -> RnMG LocalFixityEnv
+    getFixities warn_uu acc (FixD fix)
+      = fix_decl warn_uu acc fix
+
+    getFixities warn_uu acc (TyClD (ClassDecl _ _ _ _ sigs _ _ _ _ ))
+      = foldlRn (fix_decl warn_uu) acc [sig | FixSig sig <- sigs]
+               -- Get fixities from class decl sigs too.
+    getFixities warn_uu acc other_decl
+      = returnRn acc
+
+    fix_decl warn_uu acc sig@(FixitySig rdr_name fixity loc)
+       =       -- Check for fixity decl for something not declared
+         case lookupRdrEnv gbl_env rdr_name of {
+           Nothing | warn_uu
+                   -> pushSrcLocRn loc (addWarnRn (unusedFixityDecl rdr_name fixity))
+                      `thenRn_` returnRn acc 
+                   | otherwise -> returnRn acc ;
+       
+           Just ((name,_):_) ->
+
+               -- Check for duplicate fixity decl
+         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))
+         }}
+\end{code}
+
+
+%*********************************************************
+%*                                                      *
+\subsection{Deprecations}
+%*                                                      *
+%*********************************************************
+
+For deprecations, all we do is check that the names are in scope.
+It's only imported deprecations, dealt with in RnIfaces, that we
+gather them together.
+
+\begin{code}
+rnDeprecs :: GlobalRdrEnv -> Maybe DeprecTxt
+          -> [RdrNameHsDecl] -> RnMG [RdrNameDeprecation]
+rnDeprecs gbl_env mod_deprec decls
+ = mapRn rn_deprec deprecs     `thenRn_` 
+   returnRn (extra_deprec ++ deprecs)
+ where
+   deprecs = [d | DeprecD d <- decls]
+   extra_deprec = case mod_deprec of
+                  Nothing  -> []
+                  Just txt -> [Deprecation (IEModuleContents undefined) txt noSrcLoc]
+
+   rn_deprec (Deprecation ie txt loc)
+     = pushSrcLocRn loc                $
+       mapRn check (ieNames ie)
+
+   check n = case lookupRdrEnv gbl_env n of
+               Nothing -> addErrRn (unknownNameErr n)
+               Just _  -> returnRn ()
+\end{code}
+
+
+%*********************************************************
+%*                                                      *
 \subsection{Unused names}
 %*                                                      *
 %*********************************************************
 
 \begin{code}
-reportUnusedNames :: ModuleName -> GlobalRdrEnv -> AvailEnv -> ExportEnv -> NameSet -> RnMG ()
-reportUnusedNames mod_name gbl_env avail_env (ExportEnv export_avails _ _) mentioned_names
-  = let
-       used_names = mentioned_names `unionNameSets` availsToNameSet export_avails
-
-       -- 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`
-         mkNameSet [ availName parent_avail
-                   | sub_name <- nameSetToList used_names
-                   , isValOcc (getOccName sub_name)
-
-                       -- Usually, every used name will appear in avail_env, but there 
-                       -- is one time when it doesn't: tuples and other built in syntax.  When you
-                       -- write (a,b) that gives rise to a *use* of "(,)", so that the
-                       -- instances will get pulled in, but the tycon "(,)" isn't actually
-                       -- in scope.  Hence the isValOcc filter.
-                       --
-                       -- Also, (-x) gives rise to an implicit use of 'negate'; similarly, 
-                       --   3.5 gives rise to an implcit use of :%
-                       -- hence the isUserImportedName filter on the warning
-                     
-                   , let parent_avail 
-                           = case lookupNameEnv avail_env sub_name of
-                               Just avail -> avail
-                               Nothing -> WARN( isUserImportedName sub_name,
-                                                text "reportUnusedName: not in avail_env" <+> ppr sub_name )
-                                          Avail sub_name
-                     
-                   , case parent_avail of { AvailTC _ _ -> True; other -> False }
-                   ]
-
-       defined_names = mkNameSet (concat (rdrEnvElts gbl_env))
-       defined_but_not_used =
-          nameSetToList (defined_names `minusNameSet` really_used_names)
-
-       -- Filter out the ones only defined implicitly
-       bad_locals = [n | n <- defined_but_not_used, isLocallyDefined             n]
-       bad_imps   = [n | n <- defined_but_not_used, isUserImportedExplicitlyName n]
-
-       deprec_used deprec_env = [ (n,txt)
-                                 | n <- nameSetToList mentioned_names,
-                                   not (isLocallyDefined n),
-                                   Just txt <- [lookupNameEnv deprec_env n] ]
-
-       minimal_imports :: FiniteMap Module AvailEnv
-       minimal_imports = foldNameSet add emptyFM really_used_names
-       add n acc = case maybeUserImportedFrom n of
-                       Nothing -> acc
-                       Just m  -> addToFM_C plusAvailEnv acc m
-                                            (unitAvailEnv (mk_avail n))
-       mk_avail n = case lookupNameEnv avail_env n of
-                       Just (AvailTC m _) | n==m      -> AvailTC n [n]
-                                          | otherwise -> AvailTC m [n,m]
-                       Just avail         -> Avail n
-                       Nothing            -> pprPanic "mk_avail" (ppr n)
-    in
+reportUnusedNames :: ModuleName -> [ModuleName] 
+                 -> GlobalRdrEnv -> AvailEnv
+                 -> Avails -> NameSet -> [RenamedHsDecl] 
+                 -> RnMG ()
+reportUnusedNames mod_name direct_import_mods 
+                 gbl_env avail_env 
+                 export_avails mentioned_names
+                 imported_decls
+  = warnUnusedModules unused_imp_mods                          `thenRn_`
     warnUnusedLocalBinds bad_locals                            `thenRn_`
-    warnUnusedImports bad_imps                                 `thenRn_`
+    warnUnusedImports bad_imp_names                            `thenRn_`
     printMinimalImports mod_name minimal_imports               `thenRn_`
-    getIfacesRn                                                        `thenRn` \ ifaces ->
-    (if opt_WarnDeprecations
-       then mapRn_ warnDeprec (deprec_used (iDeprecs ifaces))
-       else returnRn ())
+    warnDeprecations really_used_names                         `thenRn_`
+    returnRn ()
+
+  where
+    used_names = mentioned_names `unionNameSets` availsToNameSet export_avails
+    
+    -- 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`
+      mkNameSet [ parent_name
+               | sub_name <- nameSetToList used_names
+    
+               -- Usually, every used name will appear in avail_env, but there 
+               -- is one time when it doesn't: tuples and other built in syntax.  When you
+               -- write (a,b) that gives rise to a *use* of "(,)", so that the
+               -- instances will get pulled in, but the tycon "(,)" isn't actually
+               -- in scope.  Also, (-x) gives rise to an implicit use of 'negate'; 
+               -- similarly,   3.5 gives rise to an implcit use of :%
+               -- Hence the silent 'False' in all other cases
+             
+               , Just parent_name <- [case lookupNameEnv avail_env sub_name of
+                                       Just (AvailTC n _) -> Just n
+                                       other              -> Nothing]
+           ]
+    
+    defined_names, defined_and_used, defined_but_not_used :: [(Name,Provenance)]
+    defined_names                           = concat (rdrEnvElts gbl_env)
+    (defined_and_used, defined_but_not_used) = partition used defined_names
+    used (name,_)                           = not (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,
+                             not (module_unused mod)]
+    
+    -- inst_mods are directly-imported modules that 
+    -- contain instance decl(s) that the renamer decided to suck in
+    -- It's not necessarily redundant to import such modules.
+    --
+    -- NOTE: Consider 
+    --       module This
+    --         import M ()
+    --
+    --  The import M() is not *necessarily* redundant, even if
+    --          we suck in no instance decls from M (e.g. it contains 
+    --  no instance decls, or This contains no code).  It may be 
+    --  that we import M solely to ensure that M's orphan instance 
+    --  decls (or those in its imports) are visible to people who 
+    --  import This.  Sigh. 
+    --  There's really no good way to detect this, so the error message 
+    --  in RnEnv.warnUnusedModules is weakened instead
+    inst_mods :: [ModuleName]
+    inst_mods = [m | InstD (InstDecl _ _ _ (Just dfun) _) <- imported_decls,
+                let m = moduleName (nameModule dfun),
+                m `elem` direct_import_mods
+           ]
+    
+    -- To figure out the minimal set of imports, start with the things
+    -- that are in scope (i.e. in gbl_env).  Then just combine them
+    -- into a bunch of avails, so they are properly grouped
+    minimal_imports :: FiniteMap ModuleName AvailEnv
+    minimal_imports0 = emptyFM
+    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
+
+    mk_avail n = case lookupNameEnv avail_env n of
+               Just (AvailTC m _) | n==m      -> AvailTC n [n]
+                                  | otherwise -> AvailTC m [n,m]
+               Just avail         -> Avail n
+               Nothing            -> pprPanic "mk_avail" (ppr n)
+    
+    add_inst_mod m acc 
+      | m `elemFM` acc = acc   -- We import something already
+      | otherwise      = addToFM acc m emptyAvailEnv
+       -- Add an empty collection of imports for a module
+       -- from which we have sucked only instance decls
+    
+    -- unused_imp_mods are the directly-imported modules 
+    -- that are not mentioned in minimal_imports
+    unused_imp_mods = [m | m <- direct_import_mods,
+                      not (maybeToBool (lookupFM minimal_imports m)),
+                      m /= pRELUDE_Name]
+    
+    module_unused :: Module -> Bool
+    module_unused mod = moduleName mod `elem` unused_imp_mods
+
+
+warnDeprecations used_names
+  = doptRn Opt_WarnDeprecations                                `thenRn` \ warn_drs ->
+    if not warn_drs then returnRn () else
+
+    getIfacesRn                                                `thenRn` \ ifaces ->
+    getHomeIfaceTableRn                                        `thenRn` \ hit ->
+    let
+       pit     = iPIT ifaces
+       deprecs = [ (n,txt)
+                  | n <- nameSetToList used_names,
+                    Just txt <- [lookup_deprec hit pit n] ]
+    in                   
+    mapRn_ warnDeprec deprecs
+
+  where
+    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
 
 -- ToDo: deal with original imports with 'qualified' and 'as M' clauses
 printMinimalImports mod_name imps
-  | not opt_D_dump_minimal_imports
-  = returnRn ()
-  | otherwise
-  = mapRn to_ies (fmToList imps)               `thenRn` \ mod_ies ->
+  = 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))
        })                                      `thenRn_`
@@ -613,7 +815,7 @@ printMinimalImports mod_name imps
                            parens (fsep (punctuate comma (map ppr ies)))
 
     to_ies (mod, avail_env) = mapRn to_ie (availEnvElts avail_env)     `thenRn` \ ies ->
-                             returnRn (moduleName mod, ies)
+                             returnRn (mod, ies)
 
     to_ie :: AvailInfo -> RnMG (IE Name)
     to_ie (Avail n)       = returnRn (IEVar n)
@@ -627,27 +829,20 @@ printMinimalImports mod_name imps
                              other -> pprTrace "to_ie" (ppr n <+> ppr (nameModule n) <+> ppr other) $
                                       returnRn (IEVar n)
 
-warnDeprec :: (Name, DeprecTxt) -> RnM d ()
-warnDeprec (name, txt)
-  = pushSrcLocRn (getSrcLoc name)      $
-    addWarnRn                          $
-    sep [ text (occNameFlavour (nameOccName name)) <+> ppr name <+>
-          text "is deprecated:", nest 4 (ppr txt) ]
-
-
 rnDump  :: [RenamedHsDecl]     -- Renamed imported decls
        -> [RenamedHsDecl]      -- Renamed local decls
        -> RnMG (IO ())
-rnDump imp_decls decls
-        | opt_D_dump_rn_trace || 
-         opt_D_dump_rn_stats ||
-         opt_D_dump_rn 
-       = getRnStats imp_decls          `thenRn` \ stats_msg ->
-
-         returnRn (printErrs stats_msg >> 
-                   dumpIfSet opt_D_dump_rn "Renamer:" (vcat (map ppr decls)))
-
-       | otherwise = returnRn (return ())
+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 ())
 \end{code}
 
 
@@ -662,9 +857,9 @@ getRnStats :: [RenamedHsDecl] -> RnMG SDoc
 getRnStats imported_decls
   = getIfacesRn                `thenRn` \ ifaces ->
     let
-       n_mods = length [() | (_, _, _, Just _) <- eltsFM (iImpModInfo ifaces)]
+       n_mods = length [() | (_, _, True) <- eltsFM (iImpModInfo ifaces)]
 
-       decls_read     = [decl | (_, avail, True, (_,decl)) <- nameEnvElts (iDecls 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.
@@ -715,3 +910,165 @@ count_decls decls
     inst_decls    = length [() | InstD _  <- decls]
 \end{code}    
 
+
+%************************************************************************
+%*                                                                     *
+\subsection{Errors and warnings}
+%*                                                                     *
+%************************************************************************
+
+\begin{code}
+warnDeprec :: (Name, DeprecTxt) -> RnM d ()
+warnDeprec (name, txt)
+  = pushSrcLocRn (getSrcLoc name)      $
+    addWarnRn                          $
+    sep [ text (occNameFlavour (nameOccName name)) <+> ppr name <+>
+          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,
+         ptext SLIT("and") <+> ppr loc2]
+\end{code}
+
+
+%********************************************************
+%*                                                     *
+\subsection{Checking usage information}
+%*                                                     *
+%********************************************************
+
+\begin{code}
+{-
+checkEarlyExit mod_name
+  = traceRn (text "Considering whether compilation is required...")    `thenRn_`
+
+       -- Read the old interface file, if any, for the module being compiled
+    findAndReadIface doc_str mod_name False {- Not hi-boot -}  `thenRn` \ maybe_iface ->
+
+       -- CHECK WHETHER WE HAVE IT ALREADY
+    case maybe_iface of
+       Left err ->     -- Old interface file not found, so we'd better bail out
+                   traceRn (vcat [ptext SLIT("No old interface file for") <+> ppr mod_name,
+                                  err])                        `thenRn_`
+                   returnRn (outOfDate, Nothing)
+
+       Right iface
+         | panic "checkEarlyExit: ???: not opt_SourceUnchanged"
+         ->    -- Source code changed
+            traceRn (nest 4 (text "source file changed or recompilation check turned off"))    `thenRn_` 
+            returnRn (False, Just iface)
+
+         | otherwise
+         ->    -- Source code unchanged and no errors yet... carry on 
+            checkModUsage (pi_usages iface)    `thenRn` \ up_to_date ->
+            returnRn (up_to_date, Just iface)
+  where
+       -- Only look in current directory, with suffix .hi
+    doc_str = sep [ptext SLIT("need usage info from"), ppr mod_name]
+\end{code}
+       
+\begin{code}
+upToDate  = True
+outOfDate = False
+
+checkModUsage :: [ImportVersion OccName] -> RnMG Bool
+-- Given the usage information extracted from the old
+-- M.hi file for the module being compiled, figure out
+-- whether M needs to be recompiled.
+
+checkModUsage [] = returnRn upToDate           -- Yes!  Everything is up to date!
+
+checkModUsage ((mod_name, _, _, NothingAtAll) : rest)
+       -- If CurrentModule.hi contains 
+       --      import Foo :: ;
+       -- then that simply records that Foo lies below CurrentModule in the
+       -- hierarchy, but CurrentModule doesn't depend in any way on Foo.
+       -- In this case we don't even want to open Foo's interface.
+  = traceRn (ptext SLIT("Nothing used from:") <+> ppr mod_name)        `thenRn_`
+    checkModUsage rest -- This one's ok, so check the rest
+
+checkModUsage ((mod_name, _, _, whats_imported)  : rest)
+  = tryLoadInterface doc_str mod_name ImportBySystem   `thenRn` \ (ifaces, maybe_err) ->
+    case maybe_err of {
+       Just err -> out_of_date (sep [ptext SLIT("Can't find version number for module"), 
+                                     ppr mod_name]) ;
+               -- Couldn't find or parse a module mentioned in the
+               -- old interface file.  Don't complain -- it might just be that
+               -- the current module doesn't need that import and it's been deleted
+
+       Nothing -> 
+    let
+       (_, new_mod_vers, new_fix_vers, new_rule_vers, _, _) 
+               = case lookupFM (iImpModInfo ifaces) mod_name of
+                          Just (_, _, Just stuff) -> stuff
+
+        old_mod_vers = case whats_imported of
+                        Everything v        -> v
+                        Specifically v _ _ _ -> v
+                        -- NothingAtAll case dealt with by previous eqn for checkModUsage
+    in
+       -- If the module version hasn't changed, just move on
+    if new_mod_vers == old_mod_vers then
+       traceRn (sep [ptext SLIT("Module version unchanged:"), ppr mod_name])
+       `thenRn_` checkModUsage rest
+    else
+    traceRn (sep [ptext SLIT("Module version has changed:"), ppr mod_name])
+    `thenRn_`
+       -- Module version changed, so check entities inside
+
+       -- If the usage info wants to say "I imported everything from this module"
+       --     it does so by making whats_imported equal to Everything
+       -- In that case, we must recompile
+    case whats_imported of {   -- NothingAtAll dealt with earlier
+       
+      Everything _ 
+       -> out_of_date (ptext SLIT("...and I needed the whole module")) ;
+
+      Specifically _ old_fix_vers old_rule_vers old_local_vers ->
+
+    if old_fix_vers /= new_fix_vers then
+       out_of_date (ptext SLIT("Fixities changed"))
+    else if old_rule_vers /= new_rule_vers then
+       out_of_date (ptext SLIT("Rules changed"))
+    else       
+       -- Non-empty usage list, so check item by item
+    checkEntityUsage mod_name (iDecls ifaces) old_local_vers   `thenRn` \ up_to_date ->
+    if up_to_date then
+       traceRn (ptext SLIT("...but the bits I use haven't."))  `thenRn_`
+       checkModUsage rest      -- This one's ok, so check the rest
+    else
+       returnRn outOfDate      -- This one failed, so just bail out now
+    }}
+  where
+    doc_str = sep [ptext SLIT("need version info for"), ppr mod_name]
+
+
+checkEntityUsage mod decls [] 
+  = returnRn upToDate  -- Yes!  All up to date!
+
+checkEntityUsage mod decls ((occ_name,old_vers) : rest)
+  = newGlobalName mod occ_name         `thenRn` \ name ->
+    case lookupNameEnv decls name of
+
+       Nothing       ->        -- We used it before, but it ain't there now
+                         out_of_date (sep [ptext SLIT("No longer exported:"), ppr name])
+
+       Just (new_vers,_,_,_)   -- It's there, but is it up to date?
+               | new_vers == old_vers
+                       -- Up to date, so check the rest
+               -> checkEntityUsage mod decls rest
+
+               | otherwise
+                       -- Out of date, so bale out
+               -> out_of_date (sep [ptext SLIT("Out of date:"), ppr name])
+
+out_of_date msg = traceRn msg `thenRn_` returnRn outOfDate
+-}
+\end{code}
+
+