[project @ 2002-09-13 15:02:25 by simonpj]
[ghc-hetmet.git] / ghc / compiler / rename / RnIfaces.lhs
index c591bb3..9e7c53a 100644 (file)
@@ -1,17 +1,12 @@
 %
 % (c) The GRASP/AQUA Project, Glasgow University, 1992-1998
-%
-\section[RnIfaces]{Cacheing and Renaming of Interfaces}
+section
+\%[RnIfaces]{Cacheing and Renaming of Interfaces}
 
 \begin{code}
 module RnIfaces
-     (
-       recordLocalSlurps, 
-       mkImportInfo, 
-
-       slurpImpDecls, closeDecls,
-
-       RecompileRequired, outOfDate, upToDate, recompileRequired
+     ( slurpImpDecls, importSupportingDecls,
+       RecompileRequired, outOfDate, upToDate, checkVersions
        )
 where
 
@@ -19,237 +14,40 @@ where
 
 import CmdLineOpts     ( opt_IgnoreIfacePragmas, opt_NoPruneDecls )
 import HscTypes
-import HsSyn           ( HsDecl(..), Sig(..), TyClDecl(..), ConDecl(..), ConDetails(..),
+import HsSyn           ( HsDecl(..), Sig(..), TyClDecl(..), ConDecl(..), HsConDetails(..),
                          InstDecl(..), HsType(..), hsTyVarNames, getBangType
                        )
-import HsImpExp                ( ImportDecl(..) )
 import RdrHsSyn                ( RdrNameTyClDecl, RdrNameInstDecl, RdrNameRuleDecl )
 import RnHsSyn         ( RenamedHsDecl, RenamedTyClDecl,
                          extractHsTyNames, extractHsCtxtTyNames, 
-                         tyClDeclFVs, ruleDeclFVs, instDeclFVs
-                       )
-import RnHiFiles       ( tryLoadInterface, loadHomeInterface, 
-                         loadOrphanModules
+                         tyClDeclFVs, ruleDeclFVs, impDeclFVs
                        )
+import RnHiFiles       ( loadInterface, loadHomeInterface, loadOrphanModules )
 import RnSource                ( rnTyClDecl, rnInstDecl, rnIfaceRuleDecl )
-import RnEnv
-import RnMonad
+import TcEnv           ( getInGlobalScope, tcLookupGlobal_maybe )
+import TcRnMonad
 import Id              ( idType, idName, globalIdDetails )
 import IdInfo          ( GlobalIdDetails(..) )
-import TcType          ( namesOfType )
+import TcType          ( tyClsNamesOfType, classNamesOfTheta )
 import FieldLabel      ( fieldLabelTyCon )
 import DataCon         ( dataConTyCon )
 import TyCon           ( visibleDataCons, isSynTyCon, getSynTyConDefn, tyConClass_maybe, tyConName )
-import Class           ( className )
-import Name            ( Name {-instance NamedThing-}, nameOccName,
-                         nameModule, isInternalName, NamedThing(..)
+import Class           ( className, classSCTheta )
+import Name            ( Name {-instance NamedThing-}, isWiredInName, isInternalName, nameModule, NamedThing(..)
                         )
-import NameEnv                 ( elemNameEnv, delFromNameEnv, lookupNameEnv )
+import NameEnv                 ( delFromNameEnv, lookupNameEnv )
 import NameSet
-import Module          ( Module, ModuleEnv, 
-                         moduleName, isHomeModule,
-                         ModuleName, WhereFrom(..),
-                         emptyModuleEnv, 
-                         extendModuleEnv_C, foldModuleEnv, lookupModuleEnv,
-                         elemModuleSet, extendModuleSet
-                       )
-import PrelInfo                ( wiredInThingEnv, hasKey, fractionalClassKey, numClassKey, 
+import Module          ( Module, isHomeModule, extendModuleSet )
+import PrelInfo                ( hasKey, fractionalClassKey, numClassKey, 
                          integerTyConName, doubleTyConName )
-import Maybe           ( isJust )
 import FiniteMap
 import Outputable
 import Bag
-import Util            ( sortLt, seqList )
+import Maybe( fromJust )
 \end{code}
 
 
 %*********************************************************
-%*                                                     *
-\subsection{Keeping track of what we've slurped, and version numbers}
-%*                                                     *
-%*********************************************************
-
-mkImportInfo figures out what the ``usage information'' for this
-moudule is; that is, what it must record in its interface file as the
-things it uses.  
-
-We produce a line for every module B below the module, A, currently being
-compiled:
-       import B <n> ;
-to record the fact that A does import B indirectly.  This is used to decide
-to look to look for B.hi rather than B.hi-boot when compiling a module that
-imports A.  This line says that A imports B, but uses nothing in it.
-So we'll get an early bale-out when compiling A if B's version changes.
-
-The usage information records:
-
-\begin{itemize}
-\item  (a) anything reachable from its body code
-\item  (b) any module exported with a @module Foo@
-\item   (c) anything reachable from an exported item
-\end{itemize}
-
-Why (b)?  Because if @Foo@ changes then this module's export list
-will change, so we must recompile this module at least as far as
-making a new interface file --- but in practice that means complete
-recompilation.
-
-Why (c)?  Consider this:
-\begin{verbatim}
-       module A( f, g ) where  |       module B( f ) where
-         import B( f )         |         f = h 3
-         g = ...               |         h = ...
-\end{verbatim}
-
-Here, @B.f@ isn't used in A.  Should we nevertheless record @B.f@ in
-@A@'s usages?  Our idea is that we aren't going to touch A.hi if it is
-*identical* to what it was before.  If anything about @B.f@ changes
-than anyone who imports @A@ should be recompiled in case they use
-@B.f@ (they'll get an early exit if they don't).  So, if anything
-about @B.f@ changes we'd better make sure that something in A.hi
-changes, and the convenient way to do that is to record the version
-number @B.f@ in A.hi in the usage list.  If B.f changes that'll force a
-complete recompiation of A, which is overkill but it's the only way to 
-write a new, slightly different, A.hi.
-
-But the example is tricker.  Even if @B.f@ doesn't change at all,
-@B.h@ may do so, and this change may not be reflected in @f@'s version
-number.  But with -O, a module that imports A must be recompiled if
-@B.h@ changes!  So A must record a dependency on @B.h@.  So we treat
-the occurrence of @B.f@ in the export list *just as if* it were in the
-code of A, and thereby haul in all the stuff reachable from it.
-
-       *** Conclusion: if A mentions B.f in its export list,
-           behave just as if A mentioned B.f in its source code,
-           and slurp in B.f and all its transitive closure ***
-
-[NB: If B was compiled with -O, but A isn't, we should really *still*
-haul in all the unfoldings for B, in case the module that imports A *is*
-compiled with -O.  I think this is the case.]
-
-\begin{code}
-mkImportInfo :: ModuleName                     -- Name of this module
-            -> [ImportDecl n]                  -- The import decls
-            -> RnMG [ImportVersion Name]
-
-mkImportInfo this_mod imports
-  = getIfacesRn                                        `thenRn` \ ifaces ->
-    getHomeIfaceTableRn                                `thenRn` \ hit -> 
-    let
-       (imp_pkg_mods, imp_home_names) = iVSlurp ifaces
-       pit                            = iPIT    ifaces
-
-       import_all_mods :: [ModuleName]
-               -- Modules where we imported all the names
-               -- (apart from hiding some, perhaps)
-       import_all_mods = [ m | ImportDecl m _ _ _ imp_list _ <- imports,
-                               import_all imp_list ]
-         where
-          import_all (Just (False, _)) = False -- Imports are spec'd explicitly
-          import_all other             = True  -- Everything is imported
-
-       -- mv_map groups together all the things imported and used
-       -- from a particular module in this package
-       -- We use a finite map because we want the domain
-       mv_map :: ModuleEnv [Name]
-       mv_map  = foldNameSet add_mv emptyModuleEnv imp_home_names
-        add_mv name mv_map = extendModuleEnv_C add_item mv_map mod [name]
-                          where
-                            mod = nameModule name
-                            add_item names _ = name:names
-
-       -- In our usage list we record
-       --
-       --      a) Specifically: Detailed version info for imports
-       --         from modules in this package Gotten from iVSlurp plus
-       --         import_all_mods
-       --
-       --      b) Everything: Just the module version for imports
-       --         from modules in other packages Gotten from iVSlurp plus
-       --         import_all_mods
-       --
-       --      c) NothingAtAll: The name only of modules, Baz, in
-       --         this package that are 'below' us, but which we didn't need
-       --         at all (this is needed only to decide whether to open Baz.hi
-       --         or Baz.hi-boot higher up the tree).  This happens when a
-       --         module, Foo, that we explicitly imported has 'import Baz' in
-       --         its interface file, recording that Baz is below Foo in the
-       --         module dependency hierarchy.  We want to propagate this
-       --         info.  These modules are in a combination of HIT/PIT and
-       --         iImpModInfo
-       --
-       --      d) NothingAtAll: The name only of all orphan modules
-       --         we know of (this is needed so that anyone who imports us can
-       --         find the orphan modules) These modules are in a combination
-       --         of HIT/PIT and iImpModInfo
-
-       import_info0 = foldModuleEnv mk_imp_info  []           pit
-       import_info1 = foldModuleEnv mk_imp_info  import_info0 hit
-       import_info  = not_even_opened_imports ++ import_info1
-
-               -- Recall that iImpModInfo describes modules that have
-               -- been mentioned in the import lists of interfaces we
-               -- have opened, but which we have not even opened when
-               -- compiling this module
-       not_even_opened_imports =
-         [ (mod_name, orphans, is_boot, NothingAtAll) 
-         | (mod_name, (orphans, is_boot)) <- fmToList (iImpModInfo ifaces) ]
-
-       
-       mk_imp_info :: ModIface -> [ImportVersion Name] -> [ImportVersion Name]
-       mk_imp_info iface so_far
-
-         | Just ns <- lookupModuleEnv mv_map mod       -- Case (a)
-         = go_for_it (Specifically mod_vers maybe_export_vers 
-                                   (mk_import_items ns) rules_vers)
-
-         | mod `elemModuleSet` imp_pkg_mods            -- Case (b)
-         = go_for_it (Everything mod_vers)
-
-         | import_all_mod                              -- Case (a) and (b); the import-all part
-         = if is_home_pkg_mod then
-               go_for_it (Specifically mod_vers (Just export_vers) [] rules_vers)
-               -- Since the module isn't in the mv_map, presumably we
-               -- didn't actually import anything at all from it
-           else
-               go_for_it (Everything mod_vers)
-               
-         | is_home_pkg_mod || has_orphans              -- Case (c) or (d)
-         = go_for_it NothingAtAll
-
-         | otherwise = so_far
-         where
-           go_for_it exports = (mod_name, has_orphans, mi_boot iface, exports) : so_far
-
-           mod             = mi_module iface
-           mod_name        = moduleName mod
-           is_home_pkg_mod = isHomeModule mod
-           version_info    = mi_version iface
-           version_env     = vers_decls   version_info
-           mod_vers        = vers_module  version_info
-           rules_vers      = vers_rules   version_info
-           export_vers     = vers_exports version_info
-           import_all_mod  = mod_name `elem` import_all_mods
-           has_orphans     = mi_orphan iface
-           
-               -- The sort is to put them into canonical order
-           mk_import_items ns = [(n,v) | n <- sortLt lt_occ ns, 
-                                         let v = lookupVersion version_env n
-                                ]
-                        where
-                          lt_occ n1 n2 = nameOccName n1 < nameOccName n2
-
-           maybe_export_vers | import_all_mod = Just (vers_exports version_info)
-                             | otherwise      = Nothing
-    in
-
-    -- seq the list of ImportVersions returned: occasionally these
-    -- don't get evaluated for a while and we can end up hanging on to
-    -- the entire collection of Ifaces.
-    seqList import_info (returnRn import_info)
-\end{code}
-
-%*********************************************************
 %*                                                      *
 \subsection{Slurping declarations}
 %*                                                      *
@@ -257,27 +55,31 @@ mkImportInfo this_mod imports
 
 \begin{code}
 -------------------------------------------------------
+slurpImpDecls :: FreeVars -> TcRn m [RenamedHsDecl]
 slurpImpDecls source_fvs
-  = traceRn (text "slurpImp" <+> fsep (map ppr (nameSetToList source_fvs))) `thenRn_`
+  = traceRn (text "slurpImp" <+> fsep (map ppr (nameSetToList source_fvs))) `thenM_`
 
-       -- The current slurped-set records all local things
-    slurpSourceRefs source_fvs `thenRn` \ (decls, needed) ->
+       -- Slurp in things which might be 'gates' for instance
+       -- declarations, plus the instance declarations themselves
+    slurpSourceRefs source_fvs                         `thenM` \ (gate_decls, bndrs) ->
 
        -- Then get everything else
-    closeDecls decls needed
+    let
+       needed = foldr (plusFV . impDeclFVs) emptyFVs gate_decls
+    in 
+    import_supporting_decls (gate_decls, bndrs) needed
 
 
 -------------------------------------------------------
 slurpSourceRefs :: FreeVars                    -- Variables referenced in source
-               -> RnMG ([RenamedHsDecl],
-                        FreeVars)              -- Un-satisfied needs
--- The declaration (and hence home module) of each gate has
--- already been loaded
+               -> TcRn m ([RenamedHsDecl],     -- Needed declarations
+                        NameSet)               -- Names bound by those declarations
+-- Slurp imported declarations needed directly by the source code;
+-- and some of the ones they need.  The goal is to find all the 'gates'
+-- for instance declarations.
 
 slurpSourceRefs source_fvs
-  = go_outer []                        -- Accumulating decls
-            emptyFVs                   -- Unsatisfied needs
-            emptyFVs                   -- Accumulating gates
+  = go_outer [] emptyFVs               -- Accumulating decls
             (nameSetToList source_fvs) -- Things whose defn hasn't been loaded yet
   where
        -- The outer loop repeatedly slurps the decls for the current gates
@@ -294,137 +96,203 @@ slurpSourceRefs source_fvs
        -- 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 ->
-         rnIfaceInstDecls 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
-               -- and we get an infinite loop
-
-    go_inner (decls, fvs, gates) wanted_name
-       = importDecl wanted_name                `thenRn` \ import_result ->
+    go_outer decls bndrs [] = returnM (decls, bndrs)
+
+    go_outer decls bndrs refs          -- 'refs' are not necessarily slurped yet
+       = traceRn (text "go_outer" <+> ppr refs)        `thenM_`
+         foldlM go_inner (decls, bndrs, emptyFVs) refs `thenM` \ (decls1, bndrs1, gates1) ->
+         getImportedInstDecls gates1                   `thenM` \ (inst_decls, new_gates) ->
+         rnIfaceDecls rnInstDecl inst_decls            `thenM` \ inst_decls' ->
+         go_outer (map InstD inst_decls' ++ decls1) 
+                  bndrs1
+                  (nameSetToList (new_gates `plusFV` plusFVs (map getInstDeclGates inst_decls')))
+               -- NB: we go round again to fetch the decls for any gates of any decls
+               --     we have loaded.  For example, if we mention
+               --              print :: Show a => a -> String
+               --     then we must load the decl for Show before stopping, to ensure
+               --     that instances from its home module are available
+
+    go_inner (decls, bndrs, gates) wanted_name
+       = importDecl bndrs wanted_name          `thenM` \ import_result ->
          case import_result of
-           AlreadySlurped     -> returnRn (decls, fvs, gates)
-           InTypeEnv ty_thing -> returnRn (decls, fvs, gates `plusFV` getWiredInGates ty_thing)
-                       
-           HereItIs decl -> rnIfaceTyClDecl decl               `thenRn` \ (new_decl, fvs1) ->
-                            returnRn (TyClD new_decl : decls, 
-                                      fvs1 `plusFV` fvs,
-                                      gates `plusFV` getGates source_fvs new_decl)
+           AlreadySlurped -> returnM (decls, bndrs, gates)
+
+           InTypeEnv ty_thing 
+               -> returnM (decls, 
+                           bndrs `addOneFV` wanted_name,       -- Avoid repeated calls to getWiredInGates
+                           gates `plusFV` getWiredInGates ty_thing)
+
+           HereItIs decl new_bndrs 
+               -> rnIfaceDecl rnTyClDecl decl          `thenM` \ new_decl ->
+                  returnM (TyClD new_decl : decls, 
+                           bndrs `plusFV` new_bndrs,
+                           gates `plusFV` getGates source_fvs new_decl)
 \end{code}
 
-
 \begin{code}
 -------------------------------------------------------
--- closeDecls keeps going until the free-var set is empty
-closeDecls decls needed
-  = slurpIfaceDecls decls needed       `thenRn` \ decls1 ->
-    getImportedRules                   `thenRn` \ rule_decls ->
+-- import_supporting_decls keeps going until the free-var set is empty
+importSupportingDecls needed
+ = import_supporting_decls ([], emptyNameSet) needed
+
+import_supporting_decls 
+       :: ([RenamedHsDecl], NameSet)   -- Some imported decls, with their binders
+       -> FreeVars                     -- Remaining un-slurped names
+       -> TcRn m [RenamedHsDecl]
+import_supporting_decls decls needed
+  = slurpIfaceDecls decls needed       `thenM` \ (decls1, bndrs1) ->
+    getImportedRules bndrs1            `thenM` \ rule_decls ->
     case rule_decls of
-       []    -> returnRn decls1        -- No new rules, so we are done
-       other -> rnIfaceDecls rnIfaceRuleDecl rule_decls        `thenRn` \ rule_decls' ->
+       []    -> returnM decls1 -- No new rules, so we are done
+       other -> rnIfaceDecls rnIfaceRuleDecl rule_decls        `thenM` \ rule_decls' ->
                 let
-                       rule_fvs = plusFVs (map ruleDeclFVs rule_decls')
+                   rule_fvs = plusFVs (map ruleDeclFVs rule_decls')
+                   decls2   = decls1 ++ map RuleD rule_decls'
                 in
                 traceRn (text "closeRules" <+> ppr rule_decls' $$ 
-                         fsep (map ppr (nameSetToList rule_fvs)))      `thenRn_`
-                closeDecls (map RuleD rule_decls' ++ decls1) rule_fvs
-                
+                         fsep (map ppr (nameSetToList rule_fvs)))      `thenM_`
+                import_supporting_decls (decls2, bndrs1) rule_fvs
+
 
 -------------------------------------------------------
 -- Augment decls with any decls needed by needed,
 -- and so on transitively
-slurpIfaceDecls :: [RenamedHsDecl] -> FreeVars -> RnMG [RenamedHsDecl]
-slurpIfaceDecls decls needed
-  = slurp decls (nameSetToList needed) 
+slurpIfaceDecls :: ([RenamedHsDecl], NameSet)  -- Already slurped
+               -> FreeVars                     -- Still needed
+               -> TcRn m ([RenamedHsDecl], NameSet)
+slurpIfaceDecls (decls, bndrs) needed
+  = slurp decls bndrs (nameSetToList needed) 
   where
-    slurp decls []     = returnRn decls
-    slurp decls (n:ns) = slurp_one decls n     `thenRn` \ decls1 ->
-                        slurp decls1 ns
-
-    slurp_one decls wanted_name
-      = importDecl wanted_name                 `thenRn` \ import_result ->
+    slurp decls bndrs [] = returnM (decls, bndrs)
+    slurp decls bndrs (n:ns) 
+      = importDecl bndrs n             `thenM` \ import_result ->
        case import_result of
-         HereItIs decl ->      -- Found a declaration... rename it
-                               -- and get the things it needs
-                  rnIfaceTyClDecl decl         `thenRn` \ (new_decl, fvs) ->
-                  slurp (TyClD new_decl : decls) (nameSetToList fvs)
+         HereItIs decl new_bndrs       -- Found a declaration... rename it
+           ->  rnIfaceDecl rnTyClDecl decl     `thenM` \ new_decl ->
+               slurp (TyClD new_decl : decls) 
+                     (bndrs `plusFV` new_bndrs)
+                     (nameSetToList (tyClDeclFVs new_decl) ++ ns)
   
          
          other ->      -- No declaration... (wired in thing, or deferred, 
-                       --      or already slurped)
-                  returnRn decls
-
+                       --                    or already slurped)
+               slurp decls (bndrs `addOneFV` n) ns
 
 -------------------------------------------------------
-rnIfaceDecls rn decls     = mapRn (rnIfaceDecl rn) decls
-rnIfaceDecl rn (mod, decl) = initIfaceRnMS mod (rn decl)       
+rnIfaceDecls rn decls     = mappM (rnIfaceDecl rn) decls
+rnIfaceDecl rn (mod, decl) = initRn (InterfaceMode mod) (rn decl)      
+\end{code}
 
-rnIfaceInstDecls decls fvs gates inst_decls
-  = rnIfaceDecls rnInstDecl inst_decls `thenRn` \ inst_decls' ->
-    returnRn (map InstD inst_decls' ++ decls,
-             fvs `plusFV` plusFVs (map instDeclFVs inst_decls'),
-             gates `plusFV` plusFVs (map getInstDeclGates inst_decls'))
 
-rnIfaceTyClDecl (mod, decl) = initIfaceRnMS mod (rnTyClDecl decl)      `thenRn` \ decl' ->
-                             returnRn (decl', tyClDeclFVs decl')
+\begin{code}
+       -- Tiresomely, we must get the "main" name for the 
+       -- thing, because that's what VSlurp contains, and what
+       -- is recorded in the usage information
+get_main_name (AClass cl) = className cl
+get_main_name (ATyCon tc)
+  | Just clas <- tyConClass_maybe tc = get_main_name (AClass clas)
+  | otherwise                       = tyConName tc
+get_main_name (AnId id)
+  = case globalIdDetails id of
+       DataConId     dc -> get_main_name (ATyCon (dataConTyCon dc))
+       DataConWrapId dc -> get_main_name (ATyCon (dataConTyCon dc))
+       RecordSelId lbl  -> get_main_name (ATyCon (fieldLabelTyCon lbl))
+       other            -> idName id
+
+
+recordUsage :: Name -> TcRn m ()
+-- Record that the Name has been used, for 
+-- later generation of usage info in the interface file
+recordUsage name = updUsages (upd_usg name)
+
+upd_usg name usages
+  | isHomeModule mod = usages { usg_home = addOneToNameSet (usg_home usages) name }
+  | otherwise        = usages { usg_ext  = extendModuleSet (usg_ext usages)  mod }
+  where
+    mod = nameModule name
 \end{code}
 
 
+%*********************************************************
+%*                                                     *
+\subsection{Getting in a declaration}
+%*                                                     *
+%*********************************************************
+
 \begin{code}
-recordDeclSlurp ifaces@(Ifaces { iDecls  = (decls_map, n_slurped),
-                                iSlurp  = slurped_names, 
-                                iVSlurp = vslurp })
-           avail
-  = ASSERT2( not (isInternalName (availName avail)), ppr avail )
-    ifaces { iDecls = (new_decls_map, n_slurped+1),
-            iSlurp  = new_slurped_names, 
-            iVSlurp = updateVSlurp vslurp (availName avail) }
-  where
-    new_decls_map     = foldl delFromNameEnv decls_map (availNames avail)
-    new_slurped_names = addAvailToNameSet slurped_names avail
+importDecl :: NameSet -> Name -> TcRn m ImportDeclResult
 
+data ImportDeclResult
+  = AlreadySlurped
+  | InTypeEnv TyThing
+  | HereItIs (Module, RdrNameTyClDecl) NameSet 
+       -- The NameSet is the bunch of names bound by this decl
+
+importDecl already_slurped name
+  =    -- STEP 0: Check if it's from this module
+       -- Doing this catches a common case quickly
+    getModule                          `thenM` \ this_mod ->
+    if isInternalName name || nameModule name == this_mod then
+       -- Variables defined on the GHCi command line (e.g. let x = 3)
+       -- are Internal names (which don't have a Module)
+       returnM AlreadySlurped
+    else
 
--- recordTypeEnvSlurp is used when we slurp something that's
--- already in the type environment, that was not slurped in an earlier compilation.
--- We record it in the iVSlurp set, because that's used to
--- generate usage information
+       -- STEP 1: Check if we've slurped it in while compiling this module
+    if name `elemNameSet` already_slurped then 
+       returnM AlreadySlurped  
+    else
 
-recordTypeEnvSlurp ifaces ty_thing
-  = ifaces { iVSlurp = updateVSlurp (iVSlurp ifaces) (get_main_name ty_thing) }
-  where
-       -- Tiresomely, we must get the "main" name for the 
-       -- thing, because that's what VSlurp contains, and what
-       -- is recorded in the usage information
-    get_main_name (AClass cl) = className cl
-    get_main_name (ATyCon tc)
-       | Just clas <- tyConClass_maybe tc = get_main_name (AClass clas)
-        | otherwise                       = tyConName tc
-    get_main_name (AnId id)
-       = case globalIdDetails id of
-           DataConId     dc -> get_main_name (ATyCon (dataConTyCon dc))
-           DataConWrapId dc -> get_main_name (ATyCon (dataConTyCon dc))
-           RecordSelId lbl  -> get_main_name (ATyCon (fieldLabelTyCon lbl))
-           other            -> idName id
-
-updateVSlurp (imp_mods, imp_names) main_name
-  | isHomeModule mod = (imp_mods,                    addOneToNameSet imp_names main_name)
-  | otherwise        = (extendModuleSet imp_mods mod, imp_names)
+       -- STEP 2: Check if it's already in the type environment
+    tcLookupGlobal_maybe name          `thenM` \ maybe_thing ->
+    case maybe_thing of {
+
+      Just ty_thing 
+       | isWiredInName name 
+       ->  -- When we find a wired-in name we must load its home
+           -- module so that we find any instance decls lurking therein
+           loadHomeInterface wi_doc name       `thenM_`
+           returnM (InTypeEnv ty_thing)
+
+       | otherwise
+       ->  -- We have slurp something that's already in the type environment, 
+           -- that was not slurped in an earlier compilation.
+           -- Must still record it in the Usages info, because that's used to
+           -- generate usage information
+
+           traceRn (text "not wired in" <+> ppr name)  `thenM_`
+           recordUsage (get_main_name ty_thing)        `thenM_`
+           returnM (InTypeEnv ty_thing) ;
+
+       Nothing -> 
+
+       -- STEP 4: OK, we have to slurp it in from an interface file
+       --         First load the interface file
+    traceRn nd_doc                     `thenM_`
+    loadHomeInterface nd_doc name      `thenM_`
+
+       -- STEP 4: Get the declaration out
+    getEps                             `thenM` \ eps ->
+    let
+       (decls_map, n_slurped) = eps_decls eps
+    in
+    case lookupNameEnv decls_map name of
+      Just (avail,_,decl) -> setEps eps'                       `thenM_` 
+                            recordUsage (availName avail)      `thenM_`
+                            returnM (HereItIs decl (mkFVs avail_names))
+       where
+          avail_names   = availNames avail
+          new_decls_map = foldl delFromNameEnv decls_map avail_names
+          eps'          = eps { eps_decls = (new_decls_map, n_slurped+1) }
+
+      Nothing -> addErr (getDeclErr name)      `thenM_` 
+                returnM AlreadySlurped
+    }
   where
-    mod = nameModule main_name
-  
-recordLocalSlurps new_names
-  = getIfacesRn        `thenRn` \ ifaces ->
-    setIfacesRn (ifaces { iSlurp  = iSlurp ifaces `unionNameSets` new_names })
-\end{code}
+    wi_doc = ptext SLIT("need home module for wired in thing") <+> ppr name
+    nd_doc = ptext SLIT("need decl for") <+> ppr name
 
+\end{code}
 
 
 %*********************************************************
@@ -449,7 +317,11 @@ Each has its set of 'gates': {C, T1, T2} in the above example.
 More precisely, the gates of a module are the types and classes 
 that are mentioned in:
 
-       a) the source code
+       a) the source code      [Note: in fact these don't seem
+                               to be treated as gates, perhaps
+                               because no imported instance decl
+                               can mention them; mutter mutter
+                               recursive modules.]
        b) the type of an Id that's mentioned in the source code
           [includes constructors and selectors]
        c) the RHS of a type synonym that is a gate
@@ -458,28 +330,34 @@ that are mentioned in:
 
 We slurp in an instance decl from the gated instance pool iff
        
-       all its gates are either in the gates of the module, 
-       or are a previously-loaded tycon or class.  
+       all its gates are either in the gates of the module,
+       or the gates of a previously-loaded module
 
 The latter constraint is because there might have been an instance
 decl slurped in during an earlier compilation, like this:
 
        instance Foo a => Baz (Maybe a) where ...
 
-In the module being compiled we might need (Baz (Maybe T)), where T
-is defined in this module, and hence we need (Foo T).  So @Foo@ becomes
-a gate.  But there's no way to 'see' that.  More generally, types
-might be involved as well:
+In the module being compiled we might need (Baz (Maybe T)), where T is
+defined in this module, and hence we need the instance for (Foo T).
+So @Foo@ becomes a gate.  But there's no way to 'see' that.  More
+generally, types might be involved as well:
 
-       instance Foo2 T a => Baz2 a where ...
+       instance Foo2 S a => Baz2 a where ...
 
-Now we must treat T as a gate too, as well as Foo.  So the solution
+Now we must treat S as a gate too, as well as Foo2.  So the solution
 we adopt is:
 
-       we simply treat all previously-loaded 
-       tycons and classes as gates.
+       we simply treat the gates of all previously-loaded 
+       modules as gates of this one
+
+So the gates are remembered across invocations of the renamer in the
+PersistentRenamerState.  This gloss mainly affects ghc --make and ghc
+--interactive.
 
-This gloss only affects ghc --make and ghc --interactive.
+(We used to use the persistent type environment for this purpose,
+but it has too much.  For a start, it contains all tuple types, 
+because they are in the wired-in type env!)
 
 
 Consructors and class operations
@@ -515,7 +393,6 @@ getGates source_fvs decl
 
 get_gates is_used (ForeignType {tcdName = tycon}) = unitNameSet tycon
 get_gates is_used (IfaceSig    {tcdType = ty})    = extractHsTyNames ty
-get_gates is_used (CoreDecl    {tcdType = ty})    = extractHsTyNames ty
 
 get_gates is_used (ClassDecl { tcdCtxt = ctxt, tcdName = cls, tcdTyVars = tvs, tcdSigs = sigs})
   = (super_cls_and_sigs `addOneToNameSet` cls) `unionNameSets` 
@@ -537,13 +414,13 @@ get_gates is_used (TyData {tcdCtxt = ctxt, tcdName = tycon, tcdTyVars = tvs, tcd
                       (hsTyVarNames tvs)
     `addOneToNameSet` tycon
   where
-    get (ConDecl n _ tvs ctxt details _)
+    get (ConDecl n tvs ctxt details _)
        | is_used n
                -- 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) _)
+    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 
@@ -552,12 +429,12 @@ get_gates is_used (TyData {tcdCtxt = ctxt, tcdName = tycon, tcdTyVars = tvs, tcd
        
     get other_con = emptyFVs
 
-    get_details (VanillaCon tys) = plusFVs (map get_bang tys)
+    get_details (PrefixCon 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 is_used fs = get_bang t
-                    | otherwise      = emptyFVs
+    get_field (f,t) | is_used f = get_bang t
+                   | otherwise = emptyFVs
 
     get_bang bty = extractHsTyNames (getBangType bty)
 
@@ -586,85 +463,89 @@ getWiredInGates :: TyThing -> FreeVars
 -- The TyThing is one that we already have in our type environment, either
 --     a) because the TyCon or Id is wired in, or
 --     b) from a previous compile
+--
 -- Either way, we might have instance decls in the (persistent) collection
 -- of parsed-but-not-slurped instance decls that should be slurped in.
 -- This might be the first module that mentions both the type and the class
 -- for that instance decl, even though both the type and the class were
 -- mentioned in other modules, and hence are in the type environment
 
-getWiredInGates (AnId the_id) = namesOfType (idType the_id)
-getWiredInGates (AClass cl)   = implicitClassGates (getName cl)
-       -- The superclasses must also be previously
-       -- loaded, and hence are automatically gates
-       -- All previously-loaded classes are automatically gates
-       -- See "The gating story" above
+getWiredInGates (AClass cl)
+  = unitFV (getName cl) `plusFV` mkFVs super_classes
+  where
+    super_classes = classNamesOfTheta (classSCTheta cl)
+
+getWiredInGates (AnId the_id) = tyClsNamesOfType (idType the_id)
 getWiredInGates (ATyCon tc)
-  | isSynTyCon tc = delListFromNameSet (namesOfType ty) (map getName tyvars)
+  | isSynTyCon tc = tyClsNamesOfType ty
   | otherwise    = unitFV (getName tc)
   where
-    (tyvars,ty)  = getSynTyConDefn tc
+    (_,ty)  = getSynTyConDefn tc
 
 getInstDeclGates (InstDecl inst_ty _ _ _ _) = extractHsTyNames inst_ty
 \end{code}
 
 \begin{code}
-getImportedInstDecls :: NameSet -> RnMG [(Module,RdrNameInstDecl)]
+getImportedInstDecls :: NameSet -> TcRn m ([(Module,RdrNameInstDecl)], NameSet)
+       -- Returns the gates that are new since last time
 getImportedInstDecls gates
   =            -- First, load any orphan-instance modules that aren't aready loaded
        -- Orphan-instance modules are recorded in the module dependecnies
-    getIfacesRn                                        `thenRn` \ ifaces ->
+    getEps                                     `thenM` \ eps ->
     let
-       orphan_mods =
-         [mod | (mod, (True, _)) <- fmToList (iImpModInfo ifaces)]
+       old_gates = eps_inst_gates eps
+       new_gates = gates `minusNameSet` old_gates
+       all_gates = new_gates `unionNameSets` old_gates
+       orphan_mods = [mod | (mod, (True, _)) <- fmToList (eps_imp_mods eps)]
     in
-    loadOrphanModules orphan_mods                      `thenRn_` 
+    loadOrphanModules orphan_mods                      `thenM_` 
 
        -- Now we're ready to grab the instance declarations
        -- Find the un-gated ones and return them, 
-       -- removing them from the bag kept in Ifaces
-    getIfacesRn                                        `thenRn` \ ifaces ->
-    getTypeEnvRn                                       `thenRn` \ lookup ->
+       -- removing them from the bag kept in EPS
+       -- Don't foget to get the EPS a second time... 
+       --      loadOrphanModules may have side-effected it!
+    getEps                                     `thenM` \ eps ->
     let
-       available n = n `elemNameSet` gates || isJust (lookup n)
-               -- See "The gating story" above for the isJust thing
-
-       (decls, new_insts) = selectGated available (iInsts ifaces)
+       available n        = n `elemNameSet` all_gates 
+       (decls, new_insts) = selectGated available (eps_insts eps)
     in
-    setIfacesRn (ifaces { iInsts = new_insts })                `thenRn_`
+    setEps (eps { eps_insts = new_insts,
+                 eps_inst_gates = all_gates })         `thenM_`
 
     traceRn (sep [text "getImportedInstDecls:", 
-                 nest 4 (fsep (map ppr gate_list)),
+                 nest 4 (fsep (map ppr (nameSetToList gates))),
+                 nest 4 (fsep (map ppr (nameSetToList all_gates))),
+                 nest 4 (fsep (map ppr (nameSetToList new_gates))),
                  text "Slurped" <+> int (length decls) <+> text "instance declarations",
-                 nest 4 (vcat (map ppr_brief_inst_decl decls))])       `thenRn_`
-    returnRn decls
-  where
-    gate_list      = nameSetToList gates
+                 nest 4 (vcat (map ppr_brief_inst_decl decls))])       `thenM_`
+    returnM (decls, new_gates)
 
 ppr_brief_inst_decl (mod, InstDecl inst_ty _ _ _ _)
   = case inst_ty of
        HsForAllTy _ _ tau -> ppr tau
        other              -> ppr inst_ty
 
-getImportedRules :: RnMG [(Module,RdrNameRuleDecl)]
-getImportedRules 
-  | opt_IgnoreIfacePragmas = returnRn []
+getImportedRules :: NameSet    -- Slurped already
+                -> TcRn m [(Module,RdrNameRuleDecl)]
+getImportedRules slurped
+  | opt_IgnoreIfacePragmas = returnM []
   | otherwise
-  = getIfacesRn        `thenRn` \ ifaces ->
-    getTypeEnvRn       `thenRn` \ lookup ->
+  = getEps             `thenM` \ eps ->
+    getInGlobalScope   `thenM` \ in_type_env ->
     let
                -- Slurp rules for anything that is slurped, 
-               -- either now or previously
-       gates              = iSlurp ifaces      
-       available n        = n `elemNameSet` gates || isJust (lookup n)
-       (decls, new_rules) = selectGated available (iRules ifaces)
+               -- either now, or previously
+       available n        = n `elemNameSet` slurped || in_type_env n
+       (decls, new_rules) = selectGated available (eps_rules eps)
     in
     if null decls then
-       returnRn []
+       returnM []
     else
-    setIfacesRn (ifaces { iRules = new_rules })                     `thenRn_`
+    setEps (eps { eps_rules = new_rules })                  `thenM_`
     traceRn (sep [text "getImportedRules:", 
-                 text "Slurped" <+> int (length decls) <+> text "rules"])   `thenRn_`
-    returnRn decls
+                 text "Slurped" <+> int (length decls) <+> text "rules"])   `thenM_`
+    returnM decls
 
 selectGated :: (Name->Bool) -> GatedDecls d
            -> ([(Module,d)], GatedDecls d)
@@ -688,70 +569,6 @@ selectGated available (decl_bag, n_slurped)
 \end{code}
 
 
-%*********************************************************
-%*                                                     *
-\subsection{Getting in a declaration}
-%*                                                     *
-%*********************************************************
-
-\begin{code}
-importDecl :: Name -> RnMG ImportDeclResult
-
-data ImportDeclResult
-  = AlreadySlurped
-  | InTypeEnv TyThing
-  | HereItIs (Module, RdrNameTyClDecl)
-
-importDecl name
-  =    -- STEP 1: Check if we've slurped it in while compiling this module
-    getIfacesRn                                `thenRn` \ ifaces ->
-    if name `elemNameSet` iSlurp ifaces then   
-       returnRn AlreadySlurped 
-    else
-
-
-       -- STEP 2: Check if it's already in the type environment
-    getTypeEnvRn                       `thenRn` \ lookup ->
-    case lookup name of {
-       Just ty_thing 
-           |   name `elemNameEnv` wiredInThingEnv
-           ->  -- When we find a wired-in name we must load its home
-               -- module so that we find any instance decls lurking therein
-               loadHomeInterface wi_doc name   `thenRn_`
-               returnRn (InTypeEnv ty_thing)
-
-           |   otherwise
-           ->  -- Very important: record that we've seen it
-               -- See comments with recordTypeEnvSlurp
-               setIfacesRn (recordTypeEnvSlurp ifaces ty_thing)        `thenRn_`
-               returnRn (InTypeEnv ty_thing) ;
-
-       Nothing -> 
-
-       -- STEP 3: OK, we have to slurp it in from an interface file
-       --         First load the interface file
-    traceRn nd_doc                     `thenRn_`
-    loadHomeInterface nd_doc name      `thenRn_`
-    getIfacesRn                                `thenRn` \ ifaces ->
-
-       -- STEP 4: Get the declaration out
-    let
-       (decls_map, _) = iDecls ifaces
-    in
-    case lookupNameEnv decls_map name of
-      Just (avail,_,decl) -> setIfacesRn (recordDeclSlurp ifaces avail)        `thenRn_`
-                            returnRn (HereItIs decl)
-
-      Nothing -> addErrRn (getDeclErr name)    `thenRn_` 
-                returnRn AlreadySlurped
-    }
-  where
-    wi_doc = ptext SLIT("need home module for wired in thing") <+> ppr name
-    nd_doc = ptext SLIT("need decl for") <+> ppr name
-
-\end{code}
-
-
 %********************************************************
 %*                                                     *
 \subsection{Checking usage information}
@@ -768,26 +585,30 @@ type RecompileRequired = Bool
 upToDate  = False      -- Recompile not required
 outOfDate = True       -- Recompile required
 
-recompileRequired :: FilePath          -- Only needed for debug msgs
-                 -> ModIface           -- Old interface
-                 -> RnMG RecompileRequired
-recompileRequired iface_path iface
-  = traceHiDiffsRn (text "Considering whether compilation is required for" <+> text iface_path <> colon)       `thenRn_`
+checkVersions :: Bool          -- True <=> source unchanged
+             -> ModIface       -- Old interface
+             -> TcRn m RecompileRequired
+checkVersions source_unchanged iface
+  | not source_unchanged
+  = returnM outOfDate
+  | otherwise
+  = traceHiDiffs (text "Considering whether compilation is required for" <+> 
+                       ppr (mi_module iface) <> colon) `thenM_`
 
        -- Source code unchanged and no errors yet... carry on 
     checkList [checkModUsage u | u <- mi_usages iface]
 
-checkList :: [RnMG RecompileRequired] -> RnMG RecompileRequired
-checkList []            = returnRn upToDate
-checkList (check:checks) = check       `thenRn` \ recompile ->
+checkList :: [TcRn m RecompileRequired] -> TcRn m RecompileRequired
+checkList []            = returnM upToDate
+checkList (check:checks) = check       `thenM` \ recompile ->
                           if recompile then 
-                               returnRn outOfDate
+                               returnM outOfDate
                           else
                                checkList checks
 \end{code}
        
 \begin{code}
-checkModUsage :: ImportVersion Name -> RnMG RecompileRequired
+checkModUsage :: ImportVersion Name -> TcRn m RecompileRequired
 -- Given the usage information extracted from the old
 -- M.hi file for the module being compiled, figure out
 -- whether M needs to be recompiled.
@@ -804,76 +625,81 @@ checkModUsage (mod_name, _, is_boot, whats_imported)
   =    -- Load the imported interface is possible
        -- We use tryLoadInterface, because failure is not an error
        -- (might just be that the old .hi file for this module is out of date)
-       -- We use ImportByUser/ImportByUserSource as the 'from' flag, 
-       --      a) because we need to know whether to load the .hi-boot file
-       --      b) because loadInterface things matters are amiss if we 
-       --         ImportBySystem an interface it knows nothing about
     let
        doc_str = sep [ptext SLIT("need version info for"), ppr mod_name]
-       from    | is_boot   = ImportByUserSource
-               | otherwise = ImportByUser
+       from    = ImportForUsage is_boot
     in
-    traceHiDiffsRn (text "Checking usages for module" <+> ppr mod_name) `thenRn_`
-    tryLoadInterface doc_str mod_name from     `thenRn` \ (iface, maybe_err) ->
+    traceHiDiffs (text "Checking usages for module" <+> ppr mod_name) `thenM_`
 
-    case maybe_err of {
-       Just err -> out_of_date (sep [ptext SLIT("Can't find version number for module"), 
-                                     ppr mod_name]) ;
+    recoverM (returnM Nothing)
+            (loadInterface doc_str mod_name from       `thenM` \ iface ->
+             returnM (Just iface))                     `thenM` \ mb_iface ->
+
+    case mb_iface of {
+       Nothing ->  (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 -> 
+       Just iface -> 
     let
-       new_vers      = mi_version iface
-       new_decl_vers = vers_decls new_vers
+       new_vers        = mi_version iface
+       new_mod_vers    = vers_module  new_vers
+       new_decl_vers   = vers_decls   new_vers
+       new_export_vers = vers_exports new_vers
+       new_rule_vers   = vers_rules   new_vers
     in
     case whats_imported of {   -- NothingAtAll dealt with earlier
 
-      Everything old_mod_vers -> checkModuleVersion old_mod_vers new_vers      `thenRn` \ recompile ->
+      Everything old_mod_vers -> checkModuleVersion old_mod_vers new_mod_vers  `thenM` \ recompile ->
                                 if recompile then
                                        out_of_date (ptext SLIT("...and I needed the whole module"))
                                 else
-                                       returnRn upToDate ;
+                                       returnM upToDate ;
 
       Specifically old_mod_vers maybe_old_export_vers old_decl_vers old_rule_vers ->
 
        -- CHECK MODULE
-    checkModuleVersion old_mod_vers new_vers   `thenRn` \ recompile ->
+    checkModuleVersion old_mod_vers new_mod_vers       `thenM` \ recompile ->
     if not recompile then
-       returnRn upToDate
+       returnM upToDate
     else
                                 
        -- CHECK EXPORT LIST
-    if checkExportList maybe_old_export_vers new_vers then
-       out_of_date (ptext SLIT("Export list changed"))
+    if checkExportList maybe_old_export_vers new_export_vers then
+       out_of_date_vers (ptext SLIT("  Export list changed"))
+                        (fromJust maybe_old_export_vers) 
+                        new_export_vers
     else
 
        -- CHECK RULES
-    if old_rule_vers /= vers_rules new_vers then
-       out_of_date (ptext SLIT("Rules changed"))
+    if old_rule_vers /= new_rule_vers then
+       out_of_date_vers (ptext SLIT("  Rules changed")) 
+                        old_rule_vers new_rule_vers
     else
 
        -- CHECK ITEMS ONE BY ONE
-    checkList [checkEntityUsage new_decl_vers u | u <- old_decl_vers]  `thenRn` \ recompile ->
+    checkList [checkEntityUsage new_decl_vers u | u <- old_decl_vers]  `thenM` \ recompile ->
     if recompile then
-       returnRn outOfDate      -- This one failed, so just bail out now
+       returnM outOfDate       -- This one failed, so just bail out now
     else
-       up_to_date (ptext SLIT("...but the bits I use haven't."))
+       up_to_date (ptext SLIT("  Great!  The bits I use are up to date"))
 
     }}
 
 ------------------------
-checkModuleVersion old_mod_vers new_vers
-  | vers_module new_vers == old_mod_vers
+checkModuleVersion old_mod_vers new_mod_vers
+  | new_mod_vers == old_mod_vers
   = up_to_date (ptext SLIT("Module version unchanged"))
 
   | otherwise
-  = out_of_date (ptext SLIT("Module version has changed"))
+  = out_of_date_vers (ptext SLIT("  Module version has changed"))
+                    old_mod_vers new_mod_vers
 
 ------------------------
 checkExportList Nothing  new_vers = upToDate
-checkExportList (Just v) new_vers = v /= vers_exports new_vers
+checkExportList (Just v) new_vers = v /= new_vers
 
 ------------------------
 checkEntityUsage new_vers (name,old_vers)
@@ -883,13 +709,15 @@ checkEntityUsage new_vers (name,old_vers)
                          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 -> traceHiDiffsRn (text "Up to date" <+> ppr name <+> parens (ppr new_vers)) `thenRn_`
-                                   returnRn upToDate
-         | otherwise            -> out_of_date (sep [ptext SLIT("Out of date:"), ppr name, ppr 
-                                                     old_vers, ptext SLIT("->"), ppr new_vers])
-
-up_to_date  msg = traceHiDiffsRn msg `thenRn_` returnRn upToDate
-out_of_date msg = traceHiDiffsRn msg `thenRn_` returnRn outOfDate
+         | new_vers == old_vers -> traceHiDiffs (text "  Up to date" <+> ppr name <+> parens (ppr new_vers)) `thenM_`
+                                   returnM upToDate
+         | otherwise            -> out_of_date_vers (ptext SLIT("  Out of date:") <+> ppr name)
+                                                    old_vers new_vers
+
+up_to_date  msg = traceHiDiffs msg `thenM_` returnM upToDate
+out_of_date msg = traceHiDiffs msg `thenM_` returnM outOfDate
+out_of_date_vers msg old_vers new_vers 
+  = out_of_date (hsep [msg, ppr old_vers, ptext SLIT("->"), ppr new_vers])
 \end{code}