[project @ 2003-07-17 12:04:50 by simonmar]
[ghc-hetmet.git] / ghc / compiler / main / MkIface.lhs
index de344b7..49d428f 100644 (file)
@@ -6,8 +6,8 @@
 
 \begin{code}
 module MkIface ( 
-       mkFinalIface,
-       pprModDetails, pprIface, pprUsage,
+       showIface, mkIface, mkUsageInfo,
+       pprIface, 
        ifaceTyThing,
   ) where
 
@@ -17,90 +17,161 @@ import HsSyn
 import HsCore          ( HsIdInfo(..), UfExpr(..), toUfExpr, toUfBndr )
 import HsTypes         ( toHsTyVars )
 import TysPrim         ( alphaTyVars )
-import BasicTypes      ( NewOrData(..), Activation(..),
+import BasicTypes      ( NewOrData(..), Activation(..), FixitySig(..),
                          Version, initialVersion, bumpVersion 
                        )
 import NewDemand       ( isTopSig )
-import RnMonad
+import TcRnMonad
+import TcRnTypes       ( ImportAvails(..) )
 import RnHsSyn         ( RenamedInstDecl, RenamedTyClDecl )
-import HscTypes                ( VersionInfo(..), ModIface(..), ModDetails(..),
-                         ModuleLocation(..), GhciMode(..), FixityEnv, lookupFixity,
+import HscTypes                ( VersionInfo(..), ModIface(..), 
+                         ModGuts(..), ModGuts, 
+                         GhciMode(..), HscEnv(..), Dependencies(..),
+                         FixityEnv, lookupFixity, collectFixities,
                          IfaceDecls, mkIfaceDecls, dcl_tycl, dcl_rules, dcl_insts,
-                         TyThing(..), DFunId, Avails, TypeEnv,
-                         WhatsImported(..), GenAvailInfo(..), 
-                         ImportVersion, AvailInfo, Deprecations(..),
-                         lookupVersion, typeEnvIds
+                         TyThing(..), DFunId, 
+                         Avails, AvailInfo, GenAvailInfo(..), availName, 
+                         ExternalPackageState(..),
+                         ParsedIface(..), Usage(..),
+                         Deprecations(..), initialVersionInfo,
+                         lookupVersion, lookupIfaceByModName
                        )
 
 import CmdLineOpts
-import Id              ( idType, idInfo, isImplicitId, idCgInfo,
-                         isLocalId, idName,
-                       )
-import DataCon         ( dataConId, dataConSig, dataConFieldLabels, dataConStrictMarks )
+import Id              ( idType, idInfo, isImplicitId, idCafInfo )
+import DataCon         ( dataConName, dataConSig, dataConFieldLabels, dataConStrictMarks )
 import IdInfo          -- Lots
-import Var              ( Var )
 import CoreSyn         ( CoreRule(..), IdCoreRule )
 import CoreFVs         ( ruleLhsFreeNames )
 import CoreUnfold      ( neverUnfold, unfoldingTemplate )
-import PprCore         ( pprIdCoreRule )
-import Name            ( getName, nameModule, toRdrName, isGlobalName, 
+import Name            ( getName, nameModule, nameModule_maybe, nameOccName,
                          nameIsLocalOrFrom, Name, NamedThing(..) )
 import NameEnv
 import NameSet
-import OccName         ( pprOccName )
-import TyCon
-import Class           ( classExtraBigSig, classTyCon, DefMeth(..) )
+import OccName         ( OccName, pprOccName )
+import TyCon           ( DataConDetails(..), tyConTyVars, tyConDataCons, tyConTheta,
+                         isFunTyCon, isPrimTyCon, isNewTyCon, isClassTyCon, 
+                         isSynTyCon, isAlgTyCon, isForeignTyCon,
+                         getSynTyConDefn, tyConGenInfo, tyConDataConDetails, tyConArity )
+import Class           ( classExtraBigSig, classTyCon )
 import FieldLabel      ( fieldLabelType )
-import TcType          ( tcSplitSigmaTy, tidyTopType, deNoteType, namesOfDFunHead )
+import TcType          ( tcSplitForAllTys, tcFunResultTy, tidyTopType, deNoteType, tyClsNamesOfDFunHead,
+                         mkSigmaTy, mkFunTys, mkTyConApp, mkTyVarTys )
 import SrcLoc          ( noSrcLoc )
+import Module          ( Module, ModuleName, moduleNameFS, moduleName, isHomeModule,
+                         ModLocation(..), mkSysModuleNameFS, 
+                         ModuleEnv, emptyModuleEnv, lookupModuleEnv,
+                         extendModuleEnv_C, moduleEnvElts 
+                       )
 import Outputable
-import Module          ( ModuleName )
-import Util            ( sortLt, dropList )
+import DriverUtil      ( createDirectoryHierarchy, directoryOf )
+import Util            ( sortLt, dropList, seqList )
+import Binary          ( getBinFileWithDict )
+import BinIface                ( writeBinIface, v_IgnoreHiVersion )
 import ErrUtils                ( dumpIfSet_dyn )
+import FiniteMap
+import FastString
 
+import DATA_IOREF      ( writeIORef )
 import Monad           ( when )
-import Maybe           ( catMaybes )
-import IO              ( IOMode(..), openFile, hClose )
+import Maybe           ( catMaybes, isJust, isNothing )
+import Maybes          ( orElse )
+import IO              ( putStrLn )
 \end{code}
 
 
 %************************************************************************
 %*                                                                     *
-\subsection{Completing an interface}
+\subsection{Print out the contents of a binary interface}
 %*                                                                     *
 %************************************************************************
 
 \begin{code}
+showIface :: FilePath -> IO ()
+showIface filename = do
+   -- skip the version check; we don't want to worry about profiled vs.
+   -- non-profiled interfaces, for example.
+   writeIORef v_IgnoreHiVersion True
+   parsed_iface <- Binary.getBinFileWithDict filename
+   let ParsedIface{
+      pi_mod=pi_mod, pi_pkg=pi_pkg, pi_vers=pi_vers,
+      pi_deps=pi_deps,
+      pi_orphan=pi_orphan, pi_usages=pi_usages,
+      pi_exports=pi_exports, pi_decls=pi_decls,
+      pi_fixity=pi_fixity, pi_insts=pi_insts,
+      pi_rules=pi_rules, pi_deprecs=pi_deprecs } = parsed_iface
+   putStrLn (showSDoc (vcat [
+       text "__interface" <+> doubleQuotes (ppr pi_pkg)
+          <+> ppr pi_mod <+> ppr pi_vers 
+          <+> (if pi_orphan then char '!' else empty)
+          <+> ptext SLIT("where"),
+       -- no instance Outputable (WhatsImported):
+       pprExports id (snd pi_exports),
+       pprDeps pi_deps,
+       pprUsages  id pi_usages,
+       hsep (map ppr_fix pi_fixity) <> semi,
+       vcat (map ppr_inst pi_insts),
+       vcat (map ppr_decl pi_decls),
+       ppr pi_rules
+       -- no instance Outputable (Either):
+       -- ppr pi_deprecs
+       ]))
+   where
+    ppr_fix (FixitySig n f _) = ppr f <+> ppr n
+    ppr_inst i  = ppr i <+> semi
+    ppr_decl (v,d)  = int v <+> ppr d <> semi
+\end{code}
 
+%************************************************************************
+%*                                                                     *
+\subsection{Completing an interface}
+%*                                                                     *
+%************************************************************************
 
-
-mkFinalIface :: GhciMode
-            -> DynFlags
-            -> ModuleLocation
-            -> Maybe ModIface          -- The old interface, if we have it
-            -> ModIface                -- The new one, minus the decls and versions
-            -> ModDetails              -- The ModDetails for this module
-            -> IO ModIface             -- The new one, complete with decls and versions
+\begin{code}
+mkIface :: HscEnv
+       -> ModLocation
+       -> Maybe ModIface       -- The old interface, if we have it
+       -> ModGuts              -- The compiled, tidied module
+       -> IO ModIface          -- The new one, complete with decls and versions
 -- mkFinalIface 
 --     a) completes the interface
 --     b) writes it out to a file if necessary
 
-mkFinalIface ghci_mode dflags location maybe_old_iface 
-       new_iface@ModIface{ mi_module=mod }
-       new_details@ModDetails{ md_insts=insts, 
-                               md_rules=rules,
-                               md_types=types }
-  = do { 
-               -- Add the new declarations, and the is-orphan flag
-         let iface_w_decls = new_iface { mi_decls = new_decls,
-                                         mi_orphan = orphan_mod }
+mkIface hsc_env location maybe_old_iface 
+       impl@ModGuts{ mg_module = this_mod,
+                     mg_usages = usages,
+                     mg_deps   = deps,
+                     mg_exports = exports,
+                     mg_rdr_env = rdr_env,
+                     mg_fix_env = fix_env,
+                     mg_deprecs = deprecs,
+                     mg_insts = insts, 
+                     mg_rules = rules,
+                     mg_types = types }
+  = do {       -- Sort the exports to make them easier to compare for versions
+         let { my_exports = groupAvails this_mod exports ;
+
+               iface_w_decls = ModIface { mi_module   = this_mod,
+                                          mi_package  = opt_InPackage,
+                                          mi_version  = initialVersionInfo,
+                                          mi_deps     = deps,
+                                          mi_usages   = usages,
+                                          mi_exports  = my_exports,
+                                          mi_decls    = new_decls,
+                                          mi_orphan   = orphan_mod,
+                                          mi_boot     = False,
+                                          mi_fixities = fix_env,
+                                          mi_globals  = Just rdr_env,
+                                          mi_deprecs  = deprecs } }
 
                -- Add version information
        ; let (final_iface, maybe_diffs) = _scc_ "versioninfo" addVersionInfo maybe_old_iface iface_w_decls
 
                -- Write the interface file, if necessary
-       ; when (must_write_hi_file maybe_diffs)
-              (writeIface hi_file_path final_iface)
+       ; when (must_write_hi_file maybe_diffs) $ do
+               createDirectoryHierarchy (directoryOf hi_file_path)
+               writeBinIface hi_file_path final_iface
 
                -- Debug printing
        ; write_diffs dflags final_iface maybe_diffs
@@ -109,6 +180,9 @@ mkFinalIface ghci_mode dflags location maybe_old_iface
          return final_iface }
 
   where
+     dflags    = hsc_dflags hsc_env
+     ghci_mode = hsc_mode hsc_env
+
      must_write_hi_file Nothing       = False
      must_write_hi_file (Just _diffs) = ghci_mode /= Interactive
                -- We must write a new .hi file if there are some changes
@@ -122,7 +196,7 @@ mkFinalIface ghci_mode dflags location maybe_old_iface
      inst_dcls    = map ifaceInstance insts
      ty_cls_dcls  = foldNameEnv ifaceTyThing_acc [] types
      rule_dcls    = map ifaceRule rules
-     orphan_mod   = isOrphanModule mod new_details
+     orphan_mod   = isOrphanModule impl
 
 write_diffs :: DynFlags -> ModIface -> Maybe SDoc -> IO ()
 write_diffs dflags new_iface Nothing
@@ -135,12 +209,15 @@ write_diffs dflags new_iface (Just sdoc_diffs)
 \end{code}
 
 \begin{code}
-isOrphanModule :: Module -> ModDetails -> Bool
-isOrphanModule this_mod (ModDetails {md_insts = insts, md_rules = rules})
+isOrphanModule :: ModGuts -> Bool
+isOrphanModule (ModGuts {mg_module = this_mod, mg_insts = insts, mg_rules = rules})
   = any orphan_inst insts || any orphan_rule rules
   where
-    orphan_inst dfun_id = no_locals (namesOfDFunHead (idType dfun_id))
+       -- A rule is an orphan if the LHS mentions nothing defined locally
+    orphan_inst dfun_id = no_locals (tyClsNamesOfDFunHead (idType dfun_id))
+       -- A instance is an orphan if its head mentions nothing defined locally
     orphan_rule rule    = no_locals (ruleLhsFreeNames rule)
+
     no_locals names     = isEmptyNameSet (filterNameSet (nameIsLocalOrFrom this_mod) names)
 \end{code}
 
@@ -149,6 +226,8 @@ we miss them out of the accumulating parameter here.
 
 \begin{code}
 ifaceTyThing_acc :: TyThing -> [RenamedTyClDecl] -> [RenamedTyClDecl]
+-- Don't put implicit things into the result
+ifaceTyThing_acc (ADataCon dc) so_far                = so_far
 ifaceTyThing_acc (AnId   id) so_far | isImplicitId id = so_far
 ifaceTyThing_acc (ATyCon id) so_far | isClassTyCon id = so_far
 ifaceTyThing_acc other so_far = ifaceTyThing other : so_far
@@ -167,24 +246,23 @@ ifaceTyThing (AClass clas) = cls_decl
                           tcdFDs       = toHsFDs clas_fds,
                           tcdSigs      = map toClassOpSig op_stuff,
                           tcdMeths     = Nothing, 
-                          tcdSysNames  = sys_names,
                           tcdLoc       = noSrcLoc }
 
     (clas_tyvars, clas_fds, sc_theta, sc_sels, op_stuff) = classExtraBigSig clas
     tycon     = classTyCon clas
     data_con  = head (tyConDataCons tycon)
-    sys_names = mkClassDeclSysNames (getName tycon, getName data_con, 
-                                    getName (dataConId data_con), map getName sc_sels)
 
     toClassOpSig (sel_id, def_meth)
        = ASSERT(sel_tyvars == clas_tyvars)
-         ClassOpSig (getName sel_id) def_meth' (toHsType op_ty) noSrcLoc
+         ClassOpSig (getName sel_id) def_meth (toHsType op_ty) noSrcLoc
        where
-         (sel_tyvars, _, op_ty) = tcSplitSigmaTy (idType sel_id)
-         def_meth' = case def_meth of
-                        NoDefMeth  -> NoDefMeth
-                        GenDefMeth -> GenDefMeth
-                        DefMeth id -> DefMeth (getName id)
+               -- Be careful when splitting the type, because of things
+               -- like         class Foo a where
+               --                op :: (?x :: String) => a -> a
+               -- and          class Baz a where
+               --                op :: (Ord a) => a -> a
+         (sel_tyvars, rho_ty) = tcSplitForAllTys (idType sel_id)
+         op_ty                = tcFunResultTy rho_ty
 
 ifaceTyThing (ATyCon tycon) = ty_decl
   where
@@ -195,15 +273,15 @@ ifaceTyThing (ATyCon tycon) = ty_decl
                          tcdLoc    = noSrcLoc }
 
            | isAlgTyCon tycon
-           = TyData {  tcdND     = new_or_data,
-                       tcdCtxt   = toHsContext (tyConTheta tycon),
-                       tcdName   = getName tycon,
-                       tcdTyVars = toHsTyVars tyvars,
-                       tcdCons   = map ifaceConDecl (tyConDataCons tycon),
-                       tcdNCons  = tyConFamilySize tycon,
-                       tcdDerivs = Nothing,
-                       tcdSysNames  = map getName (tyConGenIds tycon),
-                       tcdLoc       = noSrcLoc }
+           = TyData {  tcdND      = new_or_data,
+                       tcdCtxt    = toHsContext (tyConTheta tycon),
+                       tcdName    = getName tycon,
+                       tcdTyVars  = toHsTyVars tyvars,
+                       tcdCons    = ifaceConDecls (tyConDataConDetails tycon),
+                       tcdDerivs  = Nothing,
+                       tcdGeneric = Just (isJust (tyConGenInfo tycon)),
+                               -- Just True <=> has generic stuff
+                       tcdLoc     = noSrcLoc }
 
            | isForeignTyCon tycon
            = ForeignType { tcdName    = getName tycon,
@@ -217,10 +295,9 @@ ifaceTyThing (ATyCon tycon) = ty_decl
                        tcdCtxt   = [],
                        tcdName   = getName tycon,
                        tcdTyVars = toHsTyVars (take (tyConArity tycon) alphaTyVars),
-                       tcdCons   = [],
-                       tcdNCons  = 0,
+                       tcdCons   = Unknown,
                        tcdDerivs = Nothing,
-                       tcdSysNames  = [],
+                       tcdGeneric  = Just False,
                        tcdLoc       = noSrcLoc }
 
            | otherwise = pprPanic "ifaceTyThing" (ppr tycon)
@@ -230,8 +307,12 @@ ifaceTyThing (ATyCon tycon) = ty_decl
     new_or_data | isNewTyCon tycon = NewType
                | otherwise        = DataType
 
+    ifaceConDecls Unknown       = Unknown
+    ifaceConDecls (HasCons n)   = HasCons n
+    ifaceConDecls (DataCons cs) = DataCons (map ifaceConDecl cs)
+
     ifaceConDecl data_con 
-       = ConDecl (getName data_con) (getName (dataConId data_con))
+       = ConDecl (dataConName data_con)
                  (toHsTyVars ex_tyvars)
                  (toHsContext ex_theta)
                  details noSrcLoc
@@ -243,26 +324,25 @@ ifaceTyThing (ATyCon tycon) = ty_decl
                                -- includes the existential dictionaries
          details | null field_labels
                  = ASSERT( tycon == tycon1 && tyvars == tyvars1 )
-                   VanillaCon (zipWith BangType strict_marks (map toHsType arg_tys))
+                   PrefixCon (zipWith BangType strict_marks (map toHsType arg_tys))
 
                  | otherwise
                  = RecCon (zipWith mk_field strict_marks field_labels)
 
     mk_field strict_mark field_label
-       = ([getName field_label], BangType strict_mark (toHsType (fieldLabelType field_label)))
+       = (getName field_label, BangType strict_mark (toHsType (fieldLabelType field_label)))
 
 ifaceTyThing (AnId id) = iface_sig
   where
     iface_sig = IfaceSig { tcdName   = getName id, 
                           tcdType   = toHsType id_type,
                           tcdIdInfo = hs_idinfo,
-                          tcdLoc    =  noSrcLoc }
+                          tcdLoc    = noSrcLoc }
 
     id_type = idType id
     id_info = idInfo id
-    cg_info = idCgInfo id
     arity_info = arityInfo id_info
-    caf_info   = cgCafInfo cg_info
+    caf_info   = idCafInfo id
 
     hs_idinfo | opt_OmitInterfacePragmas
              = []
@@ -302,6 +382,23 @@ ifaceTyThing (AnId id) = iface_sig
     unfold_hsinfo |  neverUnfold unfold_info 
                  || has_worker = Nothing
                  | otherwise   = Just (HsUnfold inline_prag (toUfExpr rhs))
+
+
+ifaceTyThing (ADataCon dc)
+       -- This case only happens in the call to ifaceThing in InteractiveUI
+       -- Otherwise DataCons are filtered out in ifaceThing_acc
+ = IfaceSig { tcdName   = getName dc, 
+             tcdType   = toHsType full_ty,
+             tcdIdInfo = [],
+             tcdLoc    = noSrcLoc }
+ where
+    (tvs, stupid_theta, ex_tvs, ex_theta, arg_tys, tycon) = dataConSig dc
+
+       -- The "stupid context" isn't part of the wrapper-Id type
+       -- (for better or worse -- see note in DataCon.lhs), so we
+       -- have to make it up here
+    full_ty = mkSigmaTy (tvs ++ ex_tvs) (stupid_theta ++ ex_theta) 
+                       (mkFunTys arg_tys (mkTyConApp tycon (mkTyVarTys tvs)))
 \end{code}
 
 \begin{code}
@@ -320,7 +417,7 @@ ifaceInstance dfun_id
                -- and this instance decl wouldn't get imported into a module
                -- that mentioned T but not Tibble.
 
-ifaceRule :: IdCoreRule -> RuleDecl Name pat
+ifaceRule :: IdCoreRule -> RuleDecl Name
 ifaceRule (id, BuiltinRule _ _)
   = pprTrace "toHsRule: builtin" (ppr id) (bogusIfaceRule id)
 
@@ -328,12 +425,194 @@ ifaceRule (id, Rule name act bndrs args rhs)
   = IfaceRule name act (map toUfBndr bndrs) (getName id)
              (map toUfExpr args) (toUfExpr rhs) noSrcLoc
 
-bogusIfaceRule :: (NamedThing a) => a -> RuleDecl Name pat
+bogusIfaceRule :: (NamedThing a) => a -> RuleDecl Name
 bogusIfaceRule id
-  = IfaceRule SLIT("bogus") NeverActive [] (getName id) [] (UfVar (getName id)) noSrcLoc
+  = IfaceRule FSLIT("bogus") NeverActive [] (getName id) [] (UfVar (getName id)) noSrcLoc
 \end{code}
 
 
+%*********************************************************
+%*                                                     *
+\subsection{Keeping track of what we've slurped, and version numbers}
+%*                                                     *
+%*********************************************************
+
+mkUsageInfo 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}
+mkUsageInfo :: HscEnv -> ExternalPackageState
+           -> ImportAvails -> EntityUsage
+           -> [Usage Name]
+
+mkUsageInfo hsc_env eps
+           (ImportAvails { imp_mods = dir_imp_mods,
+                           imp_dep_mods = dep_mods })
+           used_names
+  = -- seq the list of Usages returned: occasionally these
+    -- don't get evaluated for a while and we can end up hanging on to
+    -- the entire collection of Ifaces.
+    usages `seqList` usages
+  where
+    usages = catMaybes [ mkUsage mod_name 
+                      | (mod_name,_) <- moduleEnvElts dep_mods]
+       -- ToDo: do we need to sort into canonical order?
+
+    hpt = hsc_HPT hsc_env
+    pit = eps_PIT eps
+    
+    import_all mod = case lookupModuleEnv dir_imp_mods mod of
+                       Just (_,imp_all) -> imp_all
+                       Nothing          -> False
+    
+    -- ent_map groups together all the things imported and used
+    -- from a particular module in this package
+    ent_map :: ModuleEnv [Name]
+    ent_map  = foldNameSet add_mv emptyModuleEnv used_names
+    add_mv name mv_map = extendModuleEnv_C add_item mv_map mod [name]
+                  where
+                    mod = nameModule name
+                    add_item names _ = name:names
+    
+    -- We want to create a Usage for a home module if 
+    -- a) we used something from; has something in used_names
+    -- b) we imported all of it, even if we used nothing from it
+    --         (need to recompile if its export list changes: export_vers)
+    -- c) is a home-package orphan module (need to recompile if its
+    --         instance decls change: rules_vers)
+    mkUsage :: ModuleName -> Maybe (Usage Name)
+    mkUsage mod_name
+      |  isNothing maybe_iface -- We can't depend on it if we didn't
+      || not (isHomeModule mod)        -- even open the interface!
+      || (null used_names
+         && not all_imported
+         && not orphan_mod)
+      = Nothing                        -- Record no usage info
+    
+      | otherwise      
+      = Just (Usage { usg_name     = moduleName mod,
+                     usg_mod      = mod_vers,
+                     usg_exports  = export_vers,
+                     usg_entities = ent_vers,
+                     usg_rules    = rules_vers })
+      where
+       maybe_iface  = lookupIfaceByModName hpt pit mod_name
+               -- In one-shot mode, the interfaces for home-package 
+               -- modules accumulate in the PIT not HPT.  Sigh.
+
+        Just iface   = maybe_iface
+        mod         = mi_module iface
+        version_info = mi_version iface
+       orphan_mod   = mi_orphan iface
+        version_env  = vers_decls   version_info
+        mod_vers     = vers_module  version_info
+        rules_vers   = vers_rules   version_info
+        all_imported = import_all mod 
+        export_vers | all_imported = Just (vers_exports version_info)
+                   | otherwise    = Nothing
+    
+       -- The sort is to put them into canonical order
+        used_names = lookupModuleEnv ent_map mod `orElse` []
+        ent_vers = [(n, lookupVersion version_env n) 
+                  | n <- sortLt lt_occ used_names ]
+        lt_occ n1 n2 = nameOccName n1 < nameOccName n2
+       -- ToDo: is '<' on OccNames the right thing; may differ between runs?
+\end{code}
+
+\begin{code}
+groupAvails :: Module -> Avails -> [(ModuleName, Avails)]
+  -- Group by module and sort by occurrence
+  -- This keeps the list in canonical order
+groupAvails this_mod avails 
+  = [ (mkSysModuleNameFS fs, sortLt lt avails)
+    | (fs,avails) <- fmToList groupFM
+    ]
+  where
+    groupFM :: FiniteMap FastString Avails
+       -- Deliberately use the FastString so we
+       -- get a canonical ordering
+    groupFM = foldl add emptyFM avails
+
+    add env avail = addToFM_C combine env mod_fs [avail']
+                 where
+                   mod_fs = moduleNameFS (moduleName avail_mod)
+                   avail_mod = case nameModule_maybe (availName avail) of
+                                         Just m  -> m
+                                         Nothing -> this_mod
+                   combine old _ = avail':old
+                   avail'        = sortAvail avail
+
+    a1 `lt` a2 = occ1 < occ2
+              where
+                occ1  = nameOccName (availName a1)
+                occ2  = nameOccName (availName a2)
+
+sortAvail :: AvailInfo -> AvailInfo
+-- Sort the sub-names into canonical order.
+-- The canonical order has the "main name" at the beginning 
+-- (if it's there at all)
+sortAvail (Avail n) = Avail n
+sortAvail (AvailTC n ns) | n `elem` ns = AvailTC n (n : sortLt lt (filter (/= n) ns))
+                        | otherwise   = AvailTC n (    sortLt lt ns)
+                        where
+                          n1 `lt` n2 = nameOccName n1 < nameOccName n2
+\end{code}
+
 %************************************************************************
 %*                                                                     *
 \subsection{Checking if the new interface is up to date
@@ -385,6 +664,7 @@ addVersionInfo (Just old_iface@(ModIface { mi_version  = old_version,
 
     no_export_change = mi_exports old_iface == mi_exports new_iface            -- Kept sorted
     no_rule_change   = dcl_rules old_decls  == dcl_rules  new_decls            -- Ditto
+                    && dcl_insts old_decls == dcl_insts  new_decls
     no_deprec_change = old_deprecs         == new_deprecs
 
        -- Fill in the version number on the new declarations by looking at the old declarations.
@@ -445,88 +725,26 @@ diffDecls (VersionInfo { vers_module = old_mod_vers, vers_decls = old_decls_vers
 \end{code}
 
 
-
-%************************************************************************
-%*                                                                     *
-\subsection{Writing ModDetails}
-%*                                                                     *
-%************************************************************************
-
-\begin{code}
-pprModDetails :: ModDetails -> SDoc
-pprModDetails (ModDetails { md_types = type_env, md_insts = dfun_ids, md_rules = rules })
-  = vcat [ dump_types dfun_ids type_env
-        , dump_insts dfun_ids
-        , dump_rules rules]
-         
-dump_types :: [Var] -> TypeEnv -> SDoc
-dump_types dfun_ids type_env
-  = text "TYPE SIGNATURES" $$ nest 4 (dump_sigs ids)
-  where
-    ids = [id | id <- typeEnvIds type_env, want_sig id]
-    want_sig id | opt_PprStyle_Debug = True
-               | otherwise          = isLocalId id && 
-                                      isGlobalName (idName id) && 
-                                      not (id `elem` dfun_ids)
-       -- isLocalId ignores data constructors, records selectors etc
-       -- The isGlobalName ignores local dictionary and method bindings
-       -- that the type checker has invented.  User-defined things have
-       -- Global names.
-
-dump_insts :: [Var] -> SDoc
-dump_insts []       = empty
-dump_insts dfun_ids = text "INSTANCES" $$ nest 4 (dump_sigs dfun_ids)
-
-dump_sigs :: [Var] -> SDoc
-dump_sigs ids
-       -- Print type signatures
-       -- Convert to HsType so that we get source-language style printing
-       -- And sort by RdrName
-  = vcat $ map ppr_sig $ sortLt lt_sig $
-    [ (toRdrName id, toHsType (idType id))
-    | id <- ids ]
-  where
-    lt_sig (n1,_) (n2,_) = n1 < n2
-    ppr_sig (n,t)        = ppr n <+> dcolon <+> ppr t
-
-dump_rules :: [IdCoreRule] -> SDoc
-dump_rules [] = empty
-dump_rules rs = vcat [ptext SLIT("{-# RULES"),
-                     nest 4 (vcat (map pprIdCoreRule rs)),
-                     ptext SLIT("#-}")]
-\end{code}
-
-
-%************************************************************************
+b%************************************************************************
 %*                                                                     *
 \subsection{Writing an interface file}
 %*                                                                     *
 %************************************************************************
 
 \begin{code}
-writeIface :: FilePath -> ModIface -> IO ()
-writeIface hi_path mod_iface
-  = do { if_hdl <- openFile hi_path WriteMode
-       ; printForIface if_hdl from_this_mod (pprIface mod_iface)
-       ; hClose if_hdl
-       }
-  where
-       -- Print names unqualified if they are from this module
-    from_this_mod n = nameModule n == this_mod
-    this_mod = mi_module mod_iface
-        
 pprIface :: ModIface -> SDoc
 pprIface iface
  = vcat [ ptext SLIT("__interface")
-               <+> doubleQuotes (ptext (mi_package iface))
+               <+> doubleQuotes (ftext (mi_package iface))
                <+> ppr (mi_module iface) <+> ppr (vers_module version_info)
                <+> pp_sub_vers
                <+> (if mi_orphan iface then char '!' else empty)
                <+> int opt_HiVersion
                <+> ptext SLIT("where")
 
-       , vcat (map pprExport (mi_exports iface))
-       , vcat (map pprUsage (mi_usages iface))
+       , pprExports nameOccName (mi_exports iface)
+       , pprDeps    (mi_deps iface)
+       , pprUsages  nameOccName (mi_usages iface)
 
        , pprFixities (mi_fixities iface) (dcl_tycl decls)
        , pprIfaceDecls (vers_decls version_info) decls
@@ -536,6 +754,7 @@ pprIface iface
     version_info = mi_version iface
     decls       = mi_decls iface
     exp_vers     = vers_exports version_info
+
     rule_vers   = vers_rules version_info
 
     pp_sub_vers | exp_vers == initialVersion && rule_vers == initialVersion = empty
@@ -548,18 +767,22 @@ When printing export lists, we print like this:
        AvailTC C [x, y]        C!(x,y)         -- Exporting x, y but not C
 
 \begin{code}
-pprExport :: (ModuleName, Avails) -> SDoc
-pprExport (mod, items)
+pprExports :: Eq a => (a -> OccName) -> [(ModuleName, [GenAvailInfo a])] -> SDoc
+pprExports getOcc exports = vcat (map (pprExport getOcc) exports)
+
+pprExport :: Eq a => (a -> OccName) -> (ModuleName, [GenAvailInfo a]) -> SDoc
+pprExport getOcc (mod, items)
  = hsep [ ptext SLIT("__export "), ppr mod, hsep (map pp_avail items) ] <> semi
   where
-    pp_avail :: AvailInfo -> SDoc
-    pp_avail (Avail name)                   = pprOcc name
+    --pp_avail :: GenAvailInfo a -> SDoc
+    pp_avail (Avail name)                   = ppr (getOcc name)
     pp_avail (AvailTC _ [])                 = empty
-    pp_avail (AvailTC n (n':ns)) | n==n'     = pprOcc n                    <> pp_export ns
-                                | otherwise = pprOcc n <> char '|' <> pp_export (n':ns)
+    pp_avail (AvailTC n (n':ns)) 
+       | n==n'     = ppr (getOcc n) <> pp_export ns
+       | otherwise = ppr (getOcc n) <> char '|' <> pp_export (n':ns)
     
     pp_export []    = empty
-    pp_export names = braces (hsep (map pprOcc names))
+    pp_export names = braces (hsep (map (ppr.getOcc) names))
 
 pprOcc :: Name -> SDoc -- Print the occurrence name only
 pprOcc n = pprOccName (nameOccName n)
@@ -567,26 +790,35 @@ pprOcc n = pprOccName (nameOccName n)
 
 
 \begin{code}
-pprUsage :: ImportVersion Name -> SDoc
-pprUsage (m, has_orphans, is_boot, whats_imported)
-  = hsep [ptext SLIT("import"), ppr m, 
-         pp_orphan, pp_boot,
-         pp_versions whats_imported
+pprUsages :: (a -> OccName) -> [Usage a] -> SDoc
+pprUsages getOcc usages = vcat (map (pprUsage getOcc) usages)
+
+pprUsage :: (a -> OccName) -> Usage a -> SDoc
+pprUsage getOcc usage
+  = hsep [ptext SLIT("import"), ppr (usg_name usage), 
+         int (usg_mod usage), 
+         pp_export_version (usg_exports usage),
+         int (usg_rules usage),
+         pp_versions (usg_entities usage)
     ] <> semi
   where
-    pp_orphan | has_orphans = char '!'
-             | otherwise   = empty
-    pp_boot   | is_boot     = char '@'
-              | otherwise   = empty
-
-       -- Importing the whole module is indicated by an empty list
-    pp_versions NothingAtAll                       = empty
-    pp_versions (Everything v)                     = dcolon <+> int v
-    pp_versions (Specifically vm ve nvs vr) = dcolon <+> int vm <+> pp_export_version ve <+> int vr 
-                                             <+> hsep [ pprOcc n <+> int v | (n,v) <- nvs ]
+    pp_versions nvs = hsep [ ppr (getOcc n) <+> int v | (n,v) <- nvs ]
 
     pp_export_version Nothing  = empty
     pp_export_version (Just v) = int v
+
+
+pprDeps :: Dependencies -> SDoc
+pprDeps (Deps { dep_mods = mods, dep_pkgs = pkgs, dep_orphs = orphs})
+  = vcat [ptext SLIT("module dependencies:") <+> fsep (map ppr_mod mods),
+         ptext SLIT("package dependencies:") <+> fsep (map ppr pkgs), 
+         ptext SLIT("orphans:") <+> fsep (map ppr orphs)
+       ]
+  where
+    ppr_mod (mod_name, boot) = ppr mod_name <+> ppr_boot boot
+   
+    ppr_boot   True  = text "[boot]"
+    ppr_boot   False = empty
 \end{code}
 
 \begin{code}
@@ -605,15 +837,12 @@ pprIfaceDecls version_map decls
 \end{code}
 
 \begin{code}
-pprFixities :: (Outputable a)
-           => NameEnv a
-           -> [TyClDecl Name pat]
+pprFixities :: FixityEnv
+           -> [TyClDecl Name]
            -> SDoc
 pprFixities fixity_map decls
   = hsep [ ppr fix <+> ppr n 
-        | d <- decls, 
-          (n,_) <- tyClDeclNames d, 
-          Just fix <- [lookupNameEnv fixity_map n]] <> semi
+        | FixitySig n fix _ <- collectFixities fixity_map decls ] <> semi
 
 -- Disgusting to print these two together, but that's 
 -- the way the interface parser currently expects them.
@@ -629,11 +858,11 @@ pprRulesAndDeprecs rules deprecs
     pp_deprecs deprecs   = ptext SLIT("__D") <+> guts
                          where
                            guts = case deprecs of
-                                       DeprecAll txt  -> doubleQuotes (ptext txt)
+                                       DeprecAll txt  -> doubleQuotes (ftext txt)
                                        DeprecSome env -> ppr_deprec_env env
 
-ppr_deprec_env :: NameEnv (Name, FAST_STRING) -> SDoc
+ppr_deprec_env :: NameEnv (Name, FastString) -> SDoc
 ppr_deprec_env env = vcat (punctuate semi (map pp_deprec (nameEnvElts env)))
                   where
-                    pp_deprec (name, txt) = pprOcc name <+> doubleQuotes (ptext txt)
+                    pp_deprec (name, txt) = pprOcc name <+> doubleQuotes (ftext txt)
 \end{code}