[project @ 1999-05-18 14:56:06 by simonpj]
[ghc-hetmet.git] / ghc / compiler / rename / Rename.lhs
index e116f7e..5720007 100644 (file)
 %
-% (c) The GRASP Project, Glasgow University, 1992-1996
+% (c) The GRASP Project, Glasgow University, 1992-1998
 %
 \section[Rename]{Renaming and dependency analysis passes}
 
 \begin{code}
-#include "HsVersions.h"
-
 module Rename ( renameModule ) where
 
-import PreludeGlaST    ( thenPrimIO, returnPrimIO, fixPrimIO, newVar, MutableVar(..) )
-
-import Ubiq
+#include "HsVersions.h"
 
 import HsSyn
-import RdrHsSyn                ( RdrNameHsModule(..), RdrNameImportDecl(..) )
-import RnHsSyn         ( RnName, RenamedHsModule(..), isRnTyCon, isRnClass )
+import RdrHsSyn                ( RdrNameHsModule )
+import RnHsSyn         ( RenamedHsModule, RenamedHsDecl, extractHsTyNames, extractHsCtxtTyNames )
 
+import CmdLineOpts     ( opt_HiMap, opt_D_dump_rn_trace,
+                         opt_D_dump_rn, opt_D_dump_rn_stats,
+                         opt_WarnUnusedBinds, opt_WarnUnusedImports
+                       )
 import RnMonad
-import RnNames         ( getGlobalNames, GlobalNameInfo(..) )
-import RnSource                ( rnSource )
-import RnIfaces                ( rnInterfaces, finalIfaceInfo, VersionInfo(..), ParsedIface )
-import RnUtils         ( extendGlobalRnEnv, emptyRnEnv, multipleOccWarn )
-import MainMonad
-
-import Bag             ( isEmptyBag, unionBags, bagToList, listToBag )
-import ErrUtils                ( Error(..), Warning(..) )
-import FiniteMap       ( emptyFM, eltsFM )
-import Name            ( getOrigNameRdr, isLocallyDefined, Name, RdrName(..) )
-import PrelInfo                ( BuiltinNames(..), BuiltinKeys(..) )
-import UniqFM          ( emptyUFM, lookupUFM, addListToUFM_C, eltsUFM )
-import UniqSupply      ( splitUniqSupply )
-import Util            ( panic, assertPanic )
-
-findHiFiles :: PrimIO (FiniteMap Module FAST_STRING)
-findHiFiles = returnPrimIO emptyFM
+import RnNames         ( getGlobalNames )
+import RnSource                ( rnSourceDecls, rnDecl )
+import RnIfaces                ( getImportedInstDecls, importDecl, getImportVersions,
+                         getImportedRules, loadHomeInterface, getSlurped
+                       )
+import RnEnv           ( availName, availNames, availsToNameSet, 
+                         warnUnusedTopNames, mapFvRn,
+                         FreeVars, plusFVs, plusFV, unitFV, emptyFVs, isEmptyFVs
+                       )
+import Module           ( Module, ModuleName, pprModule, mkSearchPath, mkThisModule )
+import Name            ( Name, isLocallyDefined,
+                         NamedThing(..), ImportReason(..), Provenance(..),
+                         pprOccName, nameOccName,
+                         getNameProvenance, occNameUserString, 
+                         maybeWiredInTyConName, maybeWiredInIdName, isWiredInName
+                       )
+import Id              ( idType )
+import DataCon         ( dataConTyCon, dataConType )
+import TyCon           ( TyCon, tyConDataCons, isSynTyCon, getSynTyConDefn )
+import RdrName         ( RdrName )
+import NameSet
+import PrelMods                ( mAIN_Name, pREL_MAIN_Name )
+import TysWiredIn      ( unitTyCon, intTyCon, doubleTyCon, boolTyCon )
+import PrelInfo                ( ioTyCon_NAME, thinAirIdNames )
+import Type            ( namesOfType, funTyCon )
+import ErrUtils                ( pprBagOfErrors, pprBagOfWarnings,
+                         doIfSet, dumpIfSet, ghcExit
+                       )
+import BasicTypes      ( NewOrData(..) )
+import Bag             ( isEmptyBag, bagToList )
+import FiniteMap       ( fmToList, delListFromFM, addToFM, sizeFM, eltsFM )
+import UniqSupply      ( UniqSupply )
+import Util            ( equivClasses )
+import Maybes          ( maybeToBool )
+import Outputable
 \end{code}
 
+
+
 \begin{code}
-renameModule :: BuiltinNames
-            -> BuiltinKeys
-            -> UniqSupply
+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
+                     ))
 
-            -> MainIO
-               (
-                RenamedHsModule,  -- output, after renaming
-                [Module],         -- imported modules; for profiling
+renameModule us this_mod@(HsModule mod_name vers exports imports local_decls loc)
+  =    -- Initialise the renamer monad
+    initRn mod_name us (mkSearchPath opt_HiMap) loc
+          (rename this_mod)                            >>=
+       \ (maybe_rn_stuff, rn_errs_bag, rn_warns_bag) ->
 
-                VersionInfo,      -- version info; for usage
-                [Module],         -- instance modules; for iface
+       -- Check for warnings
+    doIfSet (not (isEmptyBag rn_warns_bag))
+           (printErrs (pprBagOfWarnings rn_warns_bag)) >>
 
-                Bag Error,
-                Bag Warning
-               )
+       -- Check for errors; exit if so
+    doIfSet (not (isEmptyBag rn_errs_bag))
+           (printErrs (pprBagOfErrors rn_errs_bag)      >>
+            ghcExit 1
+           )                                            >>
+
+       -- Dump output, if any
+    (case maybe_rn_stuff of
+       Nothing  -> return ()
+       Just results@(_, rn_mod, _, _, _)
+                -> dumpIfSet opt_D_dump_rn "Renamer:"
+                             (ppr rn_mod)
+    )                                                  >>
+
+       -- Return results
+    return maybe_rn_stuff
 \end{code}
 
-ToDo: May want to arrange to return old interface for this module!
-ToDo: Return OrigName RnEnv to rename derivings etc with.
-ToDo: Builtin names which must be read.
-ToDo: Deal with instances (instance version, this module on instance list ???)
 
 \begin{code}
-renameModule b_names b_keys us
-            input@(HsModule mod _ _ imports _ _ _ _ _ _ _ _ _ _)
-  = findHiFiles                        `thenPrimIO` \ hi_files ->
-    newVar (emptyFM, hi_files) `thenPrimIO` \ iface_var ->
+rename this_mod@(HsModule mod_name vers exports imports local_decls loc)
+  =    -- FIND THE GLOBAL NAME ENVIRONMENT
+    getGlobalNames this_mod                    `thenRn` \ maybe_stuff ->
 
-    fixPrimIO ( \ ~(_, _, _, _, rec_occ_fm, rec_export_fn) ->
+       -- CHECK FOR EARLY EXIT
+    if not (maybeToBool maybe_stuff) then
+       -- Everything is up to date; no need to recompile further
+       rnStats []              `thenRn_`
+       returnRn Nothing
+    else
     let
-       rec_occ_fn :: Name -> [RdrName]
-       rec_occ_fn n = case lookupUFM rec_occ_fm n of
-                        Nothing        -> []
-                        Just (rn,occs) -> occs
+       Just (export_env, gbl_env, fixity_env, global_avail_env) = maybe_stuff
+    in
+
+       -- RENAME THE SOURCE
+    initRnMS gbl_env fixity_env SourceMode (
+       rnSourceDecls local_decls
+    )                                  `thenRn` \ (rn_local_decls, source_fvs) ->
 
-       global_name_info = (b_names, b_keys, rec_export_fn, rec_occ_fn)
+       -- SLURP IN ALL THE NEEDED DECLARATIONS
+    let
+       real_source_fvs = implicitFVs mod_name `plusFV` source_fvs
+               -- It's important to do the "plus" this way round, so that
+               -- when compiling the prelude, locally-defined (), Bool, etc
+               -- override the implicit ones. 
     in
-    getGlobalNames iface_var global_name_info us1 input
-               `thenPrimIO` \ (occ_env, imp_mods, imp_fixes, top_errs, top_warns) ->
+    slurpImpDecls real_source_fvs      `thenRn` \ rn_imp_decls ->
 
-    if not (isEmptyBag top_errs) then
-       returnPrimIO (rn_panic, rn_panic, top_errs, top_warns, emptyUFM, rn_panic)
+       -- EXIT IF ERRORS FOUND
+    checkErrsRn                                `thenRn` \ no_errs_so_far ->
+    if not no_errs_so_far then
+       -- Found errors already, so exit now
+       rnStats []              `thenRn_`
+       returnRn Nothing
     else
 
-    -- No top-level name errors so rename source ...
-    case initRn True mod occ_env us2
-               (rnSource imp_mods imp_fixes input) of {
-       ((rn_module, export_fn, src_occs), src_errs, src_warns) ->
+       -- GENERATE THE VERSION/USAGE INFO
+    getImportVersions mod_name exports                 `thenRn` \ my_usages ->
+    getNameSupplyRn                                    `thenRn` \ name_supply ->
 
+       -- REPORT UNUSED NAMES
+    reportUnusedNames gbl_env global_avail_env
+                     export_env
+                     source_fvs                        `thenRn_`
+
+       -- RETURN THE RENAMED MODULE
     let
-       occ_fm :: UniqFM (RnName, [RdrName])
+       has_orphans        = any isOrphanDecl rn_local_decls
+       direct_import_mods = [mod | ImportDecl mod _ _ _ _ _ <- imports]
+       rn_all_decls       = rn_imp_decls ++ rn_local_decls 
+       renamed_module = HsModule mod_name vers 
+                                 trashed_exports trashed_imports
+                                 rn_all_decls
+                                 loc
+    in
+    rnStats rn_imp_decls       `thenRn_`
+    returnRn (Just (mkThisModule mod_name,
+                   renamed_module, 
+                   (has_orphans, my_usages, export_env),
+                   name_supply,
+                   direct_import_mods))
+  where
+    trashed_exports  = {-trace "rnSource:trashed_exports"-} Nothing
+    trashed_imports  = {-trace "rnSource:trashed_imports"-} []
+\end{code}
 
-       occ_list = [ (rn,(rn,[occ])) | (rn,occ) <- bagToList src_occs]
-        occ_fm = addListToUFM_C insert_occ emptyUFM occ_list
+@implicitFVs@ forces the renamer to slurp in some things which aren't
+mentioned explicitly, but which might be needed by the type checker.
 
-       insert_occ (rn,olds) (rn',[new]) = (rn, insert new olds)
+\begin{code}
+implicitFVs mod_name
+  = implicit_main              `plusFV` 
+    mkNameSet default_tys      `plusFV`
+    mkNameSet thinAirIdNames
+  where
+       -- Add occurrences for Int, Double, and (), because they
+       -- are the types to which ambigious type variables may be defaulted by
+       -- the type checker; so they won't always appear explicitly.
+       -- [The () one is a GHC extension for defaulting CCall results.]
+       -- ALSO: funTyCon, since it occurs implicitly everywhere!
+       --       (we don't want to be bothered with making funTyCon a
+       --        free var at every function application!)
+    default_tys = [getName intTyCon, getName doubleTyCon,
+                  getName unitTyCon, getName funTyCon, getName boolTyCon]
 
-        insert new []         = [new]
-        insert new xxs@(x:xs) = case cmp new x of LT_  -> new : xxs
-                                                 EQ_  -> xxs
-                                                 GT__ -> x : insert new xs
+       -- Add occurrences for IO or PrimIO
+    implicit_main |  mod_name == mAIN_Name
+                 || mod_name == pREL_MAIN_Name = unitFV ioTyCon_NAME
+                 |  otherwise                  = emptyFVs
+\end{code}
 
-       occ_warns = map multipleOccWarn (filter multiple_occs (eltsUFM occ_fm))
-       multiple_occs (rn, (o1:o2:_)) = True
-       multiple_occs _               = False
-    in
-    returnPrimIO (rn_module, imp_mods,
-                 top_errs  `unionBags` src_errs,
-                 top_warns `unionBags` src_warns `unionBags` listToBag occ_warns,
-                 occ_fm, export_fn)
+\begin{code}
+isOrphanDecl (InstD (InstDecl inst_ty _ _ _ _))
+  = not (foldNameSet ((||) . isLocallyDefined) False (extractHsTyNames inst_ty))
+isOrphanDecl (RuleD (RuleDecl _ _ _ lhs _ _))
+  = check lhs
+  where
+    check (HsVar v)   = not (isLocallyDefined v)
+    check (HsApp f a) = check f && check a
+    check other              = True
+isOrphanDecl other = False
+\end{code}
 
-    }) `thenPrimIO` \ (rn_module, imp_mods, errs_so_far, warns_so_far, occ_fm, _) ->
 
-    if not (isEmptyBag errs_so_far) then
-       returnMn (rn_panic, rn_panic, rn_panic, rn_panic,
-                 errs_so_far, warns_so_far)
-    else
+%*********************************************************
+%*                                                      *
+\subsection{Slurping declarations}
+%*                                                      *
+%*********************************************************
+
+\begin{code}
+-------------------------------------------------------
+slurpImpDecls source_fvs
+  = traceRn (text "slurpImp" <+> fsep (map ppr (nameSetToList source_fvs))) `thenRn_`
+       -- The current slurped-set records all local things
+    getSlurped                                 `thenRn` \ local_binders ->
 
-    -- No errors renaming source so rename the interfaces ...
+    slurpSourceRefs source_fvs                 `thenRn` \ (decls1, needed1, wired_in) ->
     let
-        imports_used = [ rn | (rn,_) <- eltsUFM occ_fm, not (isLocallyDefined rn) ]
-       (import_tcs, import_vals) = partition (\ rn -> isRnTyCon rn || isRnClass rn) imports_used
+       inst_gates1 = foldr (plusFV . getWiredInGates)     source_fvs  wired_in
+       inst_gates2 = foldr (plusFV . getGates source_fvs) inst_gates1 decls1
+    in
+       -- Do this first slurpDecls before the getImportedInstDecls,
+       -- so that the home modules of all the inst_gates will be sure to be loaded
+    slurpDecls decls1 needed1                  `thenRn` \ (decls2, needed2) -> 
+    mapRn_ (load_home local_binders) wired_in  `thenRn_`
+
+       -- Now we can get the instance decls
+    getImportedInstDecls inst_gates2           `thenRn` \ inst_decls ->
+    rnIfaceDecls decls2 needed2 inst_decls     `thenRn` \ (decls3, needed3) ->
+    closeDecls  decls3 needed3
+  where
+    load_home local_binders name 
+       | name `elemNameSet` local_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
+
+-------------------------------------------------------
+slurpSourceRefs :: FreeVars                    -- Variables referenced in source
+               -> RnMG ([RenamedHsDecl],
+                        FreeVars,              -- Un-satisfied needs
+                        [Name])                -- Those variables referenced in the source
+                                               -- that turned out to be wired in things
+
+slurpSourceRefs source_fvs
+  = go [] emptyFVs [] (nameSetToList source_fvs)
+  where
+    go decls fvs wired []
+       = returnRn (decls, fvs, wired)
+    go decls fvs wired (wanted_name:refs) 
+       | isWiredInName wanted_name
+       = go decls fvs (wanted_name:wired) refs
+       | otherwise
+       = importDecl wanted_name                `thenRn` \ maybe_decl ->
+         case maybe_decl of
+               -- No declaration... (already slurped, or local)
+           Nothing   -> go decls fvs wired refs
+           Just decl -> rnIfaceDecl decl               `thenRn` \ (new_decl, fvs1) ->
+                        go (new_decl : decls) (fvs1 `plusFV` fvs) wired
+                           (extraGates new_decl ++ refs)
+
+-- Hack alert.  If we suck in a class 
+--     class Ord a => Baz a where ...
+-- then Eq is also a 'gate'.  Why?  Because Eq is a superclass of Ord,
+-- and hence may be needed during context reduction even though
+-- Eq is never mentioned explicitly.  So we snaffle out the super-classes
+-- right now, so that slurpSourceRefs will heave them in
+--
+-- Similarly the RHS of type synonyms
+extraGates (TyClD (ClassDecl ctxt _ tvs _ _ _ _ _ _ _))
+  = nameSetToList (delListFromNameSet (extractHsCtxtTyNames ctxt) (map getTyVarName tvs))
+extraGates (TyClD (TySynonym _ tvs ty _))
+  = nameSetToList (delListFromNameSet (extractHsTyNames ty) (map getTyVarName tvs))
+extraGates other = []
+
+-------------------------------------------------------
+-- closeDecls keeps going until the free-var set is empty
+closeDecls decls needed
+  | not (isEmptyFVs needed)
+  = slurpDecls decls needed    `thenRn` \ (decls1, needed1) ->
+    closeDecls decls1 needed1
+
+  | otherwise
+  = getImportedRules                   `thenRn` \ rule_decls ->
+    case rule_decls of
+       []    -> returnRn decls -- No new rules, so we are done
+       other -> rnIfaceDecls decls emptyFVs rule_decls         `thenRn` \ (decls1, needed1) ->
+                closeDecls decls1 needed1
+                
+
+-------------------------------------------------------
+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
+  = go decls emptyFVs (nameSetToList needed) 
+  where
+    go decls fvs []         = returnRn (decls, fvs)
+    go decls fvs (ref:refs) = slurpDecl decls fvs ref  `thenRn` \ (decls1, fvs1) ->
+                             go decls1 fvs1 refs
+
+-------------------------------------------------------
+slurpDecl decls fvs wanted_name
+  = importDecl wanted_name             `thenRn` \ maybe_decl ->
+    case maybe_decl of
+       -- No declaration... (wired in thing)
+       Nothing -> returnRn (decls, fvs)
+
+       -- Found a declaration... rename it
+       Just decl -> rnIfaceDecl decl           `thenRn` \ (new_decl, fvs1) ->
+                    returnRn (new_decl:decls, fvs1 `plusFV` fvs)
+\end{code}
+
+
+%*********************************************************
+%*                                                      *
+\subsection{Extracting the 'gates'}
+%*                                                      *
+%*********************************************************
+
+When we import a declaration like
+
+       data T = T1 Wibble | T2 Wobble
 
-       (orig_env, orig_dups) = extendGlobalRnEnv emptyRnEnv (map pair_orig import_vals)
-                                                            (map pair_orig import_tcs)
-        pair_orig rn = (getOrigNameRdr rn, rn)
+we don't want to treat Wibble and Wobble as gates *unless* T1, T2
+respectively are mentioned by the user program.  If only T is mentioned
+we want only T to be a gate; that way we don't suck in useless instance
+decls for (say) Eq Wibble, when they can't possibly be useful.
 
-       -- ToDo: Do we need top-level names from this module in orig_env ???
+@getGates@ takes a newly imported (and renamed) decl, and the free
+vars of the source program, and extracts from the decl the gate names.
+
+\begin{code}
+getGates source_fvs (SigD (IfaceSig _ ty _ _))
+  = extractHsTyNames ty
+
+getGates source_fvs (TyClD (ClassDecl ctxt cls tvs sigs _ _ _ _ _ _))
+  = delListFromNameSet (foldr (plusFV . get) (extractHsCtxtTyNames ctxt) sigs)
+                      (map getTyVarName tvs)
+    `addOneToNameSet` cls
+  where
+    get (ClassOpSig n _ ty _) 
+       | n `elemNameSet` source_fvs = extractHsTyNames ty
+       | otherwise                  = emptyFVs
+
+getGates source_fvs (TyClD (TySynonym tycon tvs ty _))
+  = delListFromNameSet (extractHsTyNames ty)
+                      (map getTyVarName tvs)
+    `addOneToNameSet` tycon
+
+getGates source_fvs (TyClD (TyData _ ctxt tycon tvs cons _ _ _))
+  = delListFromNameSet (foldr (plusFV . get) (extractHsCtxtTyNames ctxt) cons)
+                      (map getTyVarName tvs)
+    `addOneToNameSet` tycon
+  where
+    get (ConDecl n tvs ctxt details _)
+       | n `elemNameSet` source_fvs
+               -- If the constructor is method, get fvs from all its fields
+       = delListFromNameSet (get_details details `plusFV` 
+                             extractHsCtxtTyNames ctxt)
+                            (map getTyVarName tvs)
+    get (ConDecl n tvs ctxt (RecCon fields) _)
+               -- Even if the constructor isn't mentioned, the fields
+               -- might be, as selectors.  They can't mention existentially
+               -- bound tyvars (typechecker checks for that) so no need for 
+               -- the deleteListFromNameSet part
+       = foldr (plusFV . get_field) emptyFVs fields
+       
+    get other_con = emptyFVs
+
+    get_details (VanillaCon tys) = plusFVs (map get_bang tys)
+    get_details (InfixCon t1 t2) = get_bang t1 `plusFV` get_bang t2
+    get_details (RecCon fields)  = plusFVs [get_bang t | (_, t) <- fields]
+    get_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
+
+getGates source_fvs other_decl = emptyFVs
+\end{code}
+
+getWiredInGates is just like getGates, but it sees a wired-in Name
+rather than a declaration.
+
+\begin{code}
+getWiredInGates name | is_tycon  = get_wired_tycon the_tycon
+                    | otherwise = get_wired_id the_id
+  where
+    maybe_wired_in_tycon = maybeWiredInTyConName name
+    is_tycon            = maybeToBool maybe_wired_in_tycon
+    maybe_wired_in_id    = maybeWiredInIdName name
+    Just the_tycon      = maybe_wired_in_tycon
+    Just the_id         = maybe_wired_in_id
+
+get_wired_id id = namesOfType (idType id)
+
+get_wired_tycon tycon 
+  | isSynTyCon tycon
+  = namesOfType ty `minusNameSet` mkNameSet (map getName tyvars)
+
+  | otherwise          -- data or newtype
+  = foldr (unionNameSets . namesOfType . dataConType) emptyNameSet data_cons
+  where
+    (tyvars,ty) = getSynTyConDefn tycon
+    data_cons   = tyConDataCons tycon
+\end{code}
+
+
+%*********************************************************
+%*                                                      *
+\subsection{Unused names}
+%*                                                      *
+%*********************************************************
+
+\begin{code}
+reportUnusedNames gbl_env avail_env (ExportEnv export_avails _) mentioned_names
+  | not (opt_WarnUnusedBinds || opt_WarnUnusedImports)
+  = returnRn ()
+
+  | otherwise
+  = 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 avail 
+                                     | sub_name <- nameSetToList used_names,
+                                       let avail = case lookupNameEnv avail_env sub_name of
+                                                       Just avail -> avail
+                                                       Nothing -> pprTrace "r.u.n" (ppr sub_name) $
+                                                                  Avail sub_name
+                                     ]
+
+       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_guys = filter reportableUnusedName defined_but_not_used
     in
-    ASSERT (isEmptyBag orig_dups)
-    rnInterfaces iface_var orig_env us3 rn_module imports_used
-               `thenPrimIO` \ (rn_module_with_imports,
-                               (implicit_val_fm, implicit_tc_fm),
-                               iface_errs, iface_warns) ->
+    warnUnusedTopNames bad_guys        `thenRn_`
+    returnRn ()
+
+reportableUnusedName :: Name -> Bool
+reportableUnusedName name
+  = explicitlyImported (getNameProvenance name) &&
+    not (startsWithUnderscore (occNameUserString (nameOccName name)))
+  where
+    explicitlyImported (LocalDef _ _)                       = True     -- Report unused defns of local vars
+    explicitlyImported (NonLocalDef (UserImport _ _ expl) _) = expl    -- Report unused explicit imports
+    explicitlyImported other                                = False    -- Don't report others
+   
+       -- Haskell 98 encourages compilers to suppress warnings about
+       -- unused names in a pattern if they start with "_".
+    startsWithUnderscore ('_' : _) = True      -- Suppress warnings for names starting
+    startsWithUnderscore other     = False     -- with an underscore
+
+rnStats :: [RenamedHsDecl] -> RnMG ()
+rnStats imp_decls
+        | opt_D_dump_rn_trace || 
+         opt_D_dump_rn_stats ||
+         opt_D_dump_rn 
+       = getRnStats imp_decls          `thenRn` \ msg ->
+         ioToRnM (printErrs msg)       `thenRn_`
+         returnRn ()
+
+       | otherwise = returnRn ()
+\end{code}
+
+
+
+%*********************************************************
+%*                                                     *
+\subsection{Statistics}
+%*                                                     *
+%*********************************************************
+
+\begin{code}
+getRnStats :: [RenamedHsDecl] -> RnMG SDoc
+getRnStats imported_decls
+  = getIfacesRn                `thenRn` \ ifaces ->
     let
-        all_imports_used = imports_used ++ eltsFM implicit_tc_fm ++ eltsFM implicit_val_fm
-    in
-    finalIfaceInfo iface_var all_imports_used imp_mods
-               `thenPrimIO` \ (version_info, instance_mods) ->
+       n_mods = length [() | (_, _, Just _) <- eltsFM (iImpModInfo ifaces)]
+
+       decls_read     = [decl | (_, avail, True, (_,decl)) <- nameEnvElts (iDecls ifaces),
+                                       -- Data, newtype, and class decls are in the decls_fm
+                                       -- under multiple names; the tycon/class, and each
+                                       -- constructor/class op too.
+                                       -- The 'True' selects just the 'main' decl
+                                not (isLocallyDefined (availName avail))
+                            ]
 
-    returnMn (rn_module_with_imports, imp_mods, 
-             version_info, instance_mods, 
-             errs_so_far  `unionBags` iface_errs,
-             warns_so_far `unionBags` iface_warns)
+       (cd_rd, dd_rd, nd_rd, sd_rd, vd_rd,     _) = count_decls decls_read
+       (cd_sp, dd_sp, nd_sp, sd_sp, vd_sp, id_sp) = count_decls imported_decls
 
+       unslurped_insts       = iInsts ifaces
+       inst_decls_unslurped  = length (bagToList unslurped_insts)
+       inst_decls_read       = id_sp + inst_decls_unslurped
+
+       stats = vcat 
+               [int n_mods <+> text "interfaces read",
+                hsep [ int cd_sp, text "class decls imported, out of", 
+                       int cd_rd, text "read"],
+                hsep [ int dd_sp, text "data decls imported, out of",  
+                       int dd_rd, text "read"],
+                hsep [ int nd_sp, text "newtype decls imported, out of",  
+                       int nd_rd, text "read"],
+                hsep [int sd_sp, text "type synonym decls imported, out of",  
+                       int sd_rd, text "read"],
+                hsep [int vd_sp, text "value signatures imported, out of",  
+                       int vd_rd, text "read"],
+                hsep [int id_sp, text "instance decls imported, out of",  
+                       int inst_decls_read, text "read"],
+                text "cls dcls slurp" <+> fsep (map (ppr . tyClDeclName) 
+                                          [d | TyClD d <- imported_decls, isClassDecl d]),
+                text "cls dcls read"  <+> fsep (map (ppr . tyClDeclName) 
+                                          [d | TyClD d <- decls_read, isClassDecl d])]
+    in
+    returnRn (hcat [text "Renamer stats: ", stats])
+
+count_decls decls
+  = (class_decls, 
+     data_decls, 
+     newtype_decls,
+     syn_decls, 
+     val_decls, 
+     inst_decls)
   where
-    rn_panic = panic "renameModule: aborted with errors"
+    tycl_decls = [d | TyClD d <- decls]
+    (class_decls, data_decls, newtype_decls, syn_decls) = countTyClDecls tycl_decls
+
+    val_decls     = length [() | SigD _          <- decls]
+    inst_decls    = length [() | InstD _  <- decls]
+\end{code}    
 
-    (us1, us') = splitUniqSupply us
-    (us2, us3) = splitUniqSupply us'
-\end{code}