[project @ 1999-03-02 17:12:54 by sof]
[ghc-hetmet.git] / ghc / compiler / rename / Rename.lhs
index 2fab42e..5474e17 100644 (file)
@@ -1,5 +1,5 @@
 %
-% (c) The GRASP Project, Glasgow University, 1992-1996
+% (c) The GRASP Project, Glasgow University, 1992-1998
 %
 \section[Rename]{Renaming and dependency analysis passes}
 
@@ -9,7 +9,7 @@ module Rename ( renameModule ) where
 #include "HsVersions.h"
 
 import HsSyn
-import RdrHsSyn                ( RdrName(..), RdrNameHsModule )
+import RdrHsSyn                ( RdrNameHsModule )
 import RnHsSyn         ( RenamedHsModule, RenamedHsDecl, extractHsTyNames )
 
 import CmdLineOpts     ( opt_HiMap, opt_D_show_rn_trace,
@@ -18,22 +18,27 @@ import CmdLineOpts  ( opt_HiMap, opt_D_show_rn_trace,
                        )
 import RnMonad
 import RnNames         ( getGlobalNames )
-import RnSource                ( rnDecl )
+import RnSource                ( rnIfaceDecl, rnSourceDecls )
 import RnIfaces                ( getImportedInstDecls, importDecl, getImportVersions, getSpecialInstModules,
                          getDeferredDataDecls,
                          mkSearchPath, getSlurpedNames, getRnStats
                        )
-import RnEnv           ( addImplicitOccsRn, availNames )
-import Name            ( Name, PrintUnqualified, Provenance, isLocallyDefined,
-                         NameSet,
-                           nameSetToList, minusNameSet,
-                         NamedThing(..),
-                          nameModule, pprModule, pprOccName, nameOccName
+import RnEnv           ( addImplicitOccsRn, availName, availNames, availsToNameSet, 
+                         warnUnusedTopNames
                        )
-import TysWiredIn      ( unitTyCon, intTyCon, doubleTyCon )
+import Module           ( pprModule )
+import Name            ( Name, isLocallyDefined,
+                         NamedThing(..), ImportReason(..), Provenance(..),
+                         nameModule, pprOccName, nameOccName,
+                         getNameProvenance, occNameUserString, 
+                       )
+import RdrName         ( RdrName )
+import NameSet
 import TyCon           ( TyCon )
 import PrelMods                ( mAIN, pREL_MAIN )
-import PrelInfo                ( ioTyCon_NAME )
+import TysWiredIn      ( unitTyCon, intTyCon, doubleTyCon )
+import PrelInfo                ( ioTyCon_NAME, thinAirIdNames )
+import Type            ( funTyCon )
 import ErrUtils                ( pprBagOfErrors, pprBagOfWarnings,
                          doIfSet, dumpIfSet, ghcExit
                        )
@@ -42,7 +47,6 @@ import FiniteMap      ( fmToList, delListFromFM )
 import UniqSupply      ( UniqSupply )
 import Util            ( equivClasses )
 import Maybes          ( maybeToBool )
-import List            ( partition )
 import Outputable
 \end{code}
 
@@ -58,7 +62,7 @@ renameModule :: UniqSupply
                      , [Module]          -- Imported modules; for profiling
                      ))
 
-renameModule us this_mod@(HsModule mod_name vers exports imports fixities local_decls loc)
+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)                            >>=
@@ -88,7 +92,7 @@ renameModule us this_mod@(HsModule mod_name vers exports imports fixities local_
 
 
 \begin{code}
-rename this_mod@(HsModule mod_name vers exports imports fixities local_decls loc)
+rename this_mod@(HsModule mod_name vers exports imports local_decls loc)
   =    -- FIND THE GLOBAL NAME ENVIRONMENT
     getGlobalNames this_mod                    `thenRn` \ maybe_stuff ->
 
@@ -99,17 +103,17 @@ rename this_mod@(HsModule mod_name vers exports imports fixities local_decls loc
        returnRn Nothing
     else
     let
-       Just (export_env, rn_env, explicit_info, print_unqual) = maybe_stuff
+       Just (export_env, rn_env, global_avail_env) = maybe_stuff
     in
 
        -- RENAME THE SOURCE
-    initRnMS rn_env mod_name SourceMode (
+    initRnMS rn_env SourceMode (
        addImplicits mod_name                           `thenRn_`
-       mapRn rnDecl local_decls
-    )                                                  `thenRn` \ rn_local_decls ->
+       rnSourceDecls local_decls
+    )                                                  `thenRn` \ (rn_local_decls, fvs) ->
 
        -- SLURP IN ALL THE NEEDED DECLARATIONS
-    slurpDecls print_unqual rn_local_decls             `thenRn` \ rn_all_decls ->
+    slurpDecls rn_local_decls          `thenRn` \ rn_all_decls ->
 
        -- EXIT IF ERRORS FOUND
     checkErrsRn                                `thenRn` \ no_errs_so_far ->
@@ -124,7 +128,9 @@ rename this_mod@(HsModule mod_name vers exports imports fixities local_decls loc
     getNameSupplyRn                                    `thenRn` \ name_supply ->
 
        -- REPORT UNUSED NAMES
-    reportUnusedNames export_env explicit_info         `thenRn_`
+    reportUnusedNames rn_env global_avail_env
+                     export_env
+                     fvs                               `thenRn_`
 
        -- GENERATE THE SPECIAL-INSTANCE MODULE LIST
        -- The "special instance" modules are those modules that contain instance
@@ -142,10 +148,10 @@ rename this_mod@(HsModule mod_name vers exports imports fixities local_decls loc
     
        -- RETURN THE RENAMED MODULE
     let
-       import_mods = [mod | ImportDecl mod _ _ _ _ _ <- imports]
+       import_mods = [mod | ImportDecl mod _ _ _ _ <- imports]
 
        renamed_module = HsModule mod_name vers 
-                                 trashed_exports trashed_imports trashed_fixities
+                                 trashed_exports trashed_imports
                                  rn_all_decls
                                  loc
     in
@@ -157,7 +163,6 @@ rename this_mod@(HsModule mod_name vers exports imports fixities local_decls loc
   where
     trashed_exports  = {-trace "rnSource:trashed_exports"-} Nothing
     trashed_imports  = {-trace "rnSource:trashed_imports"-} []
-    trashed_fixities = []
 \end{code}
 
 @addImplicits@ forces the renamer to slurp in some things which aren't
@@ -165,23 +170,27 @@ mentioned explicitly, but which might be needed by the type checker.
 
 \begin{code}
 addImplicits mod_name
-  = addImplicitOccsRn (implicit_main ++ default_tys)
+  = addImplicitOccsRn (implicit_main ++ default_tys ++ 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 every appear explicitly.
+       -- the type checker; so they won't always appear explicitly.
        -- [The () one is a GHC extension for defaulting CCall results.]
-    default_tys = [getName intTyCon, getName doubleTyCon, getName unitTyCon ]
+       -- ALSO: funTyCon, since it occurs implicitly everywhere!
+       --       (we don't want to be bothered with addImplicitOcc at every
+       --        function application)
+    default_tys = [getName intTyCon, getName doubleTyCon,
+                  getName unitTyCon, getName funTyCon]
 
        -- Add occurrences for IO or PrimIO
     implicit_main |  mod_name == mAIN
                  || mod_name == pREL_MAIN = [ioTyCon_NAME]
-                 |  otherwise            = []
+                 |  otherwise             = []
 \end{code}
 
 
 \begin{code}
-slurpDecls print_unqual decls
+slurpDecls decls
   =    -- First of all, get all the compulsory decls
     slurp_compulsories decls   `thenRn` \ decls1 ->
 
@@ -196,8 +205,8 @@ slurpDecls print_unqual decls
     returnRn (rn_data_decls ++ decls2)
 
   where
-    compulsory_mode = InterfaceMode Compulsory print_unqual
-    optional_mode   = InterfaceMode Optional   print_unqual
+    compulsory_mode = InterfaceMode Compulsory
+    optional_mode   = InterfaceMode Optional
 
        -- The "slurp_compulsories" function is a loop that alternates
        -- between slurping compulsory decls and slurping the instance
@@ -227,7 +236,7 @@ slurpDecls print_unqual decls
 \end{code}
 
 \begin{code}
-closeDecls :: RnSMode
+closeDecls :: RnMode
           -> [RenamedHsDecl]                   -- Declarations got so far
           -> RnMG [RenamedHsDecl]              -- input + extra decls slurped
        -- The monad includes a list of possibly-unresolved Names
@@ -257,57 +266,55 @@ closeDecls mode decls
                           mod_name = nameModule (fst name_w_loc)
 
 rn_iface_decl mod_name mode decl
-  = initRnMS emptyRnEnv mod_name mode (rnDecl decl)
+  = setModuleRn mod_name $
+    initRnMS emptyRnEnv mode (rnIfaceDecl decl)
                                        
-rn_inst_decl mode (mod_name,decl)      = rn_iface_decl mod_name mode (InstD decl)
-rn_data_decl mode (tycon_name,ty_decl) = rn_iface_decl mod_name mode (TyD ty_decl)
-                                      where
-                                        mod_name = nameModule tycon_name
+rn_inst_decl mode (mod_name,decl)    = rn_iface_decl mod_name mode (InstD decl)
+rn_data_decl mode (mod_name,ty_decl) = rn_iface_decl mod_name mode (TyClD ty_decl)
 \end{code}
 
 \begin{code}
-reportUnusedNames (ExportEnv export_avails _) explicit_info
+reportUnusedNames (RnEnv gbl_env _) avail_env (ExportEnv export_avails _) mentioned_names
   | not (opt_WarnUnusedBinds || opt_WarnUnusedImports)
   = returnRn ()
 
   | otherwise
-  = getSlurpedNames                    `thenRn` \ slurped_names ->
-    let
-       unused_info :: FiniteMap Name HowInScope
-       unused_info = foldl delListFromFM
-                           (delListFromFM explicit_info (nameSetToList slurped_names))
-                           (map availNames export_avails)
-       unused_list = fmToList unused_info
-
-       groups = filter wanted (equivClasses cmp unused_list)
-              where
-                (name1, his1) `cmp` (name2, his2) = his1 `cmph` his2
-                
-                (FromLocalDefn _)     `cmph` (FromImportDecl _ _)  = LT
-                (FromLocalDefn _)     `cmph` (FromLocalDefn _)     = EQ
-                (FromImportDecl m1 _) `cmph` (FromImportDecl m2 _) = m1 `compare` m2
-                h1                    `cmph` h2                    = GT
-
-       wanted ((_,FromImportDecl _ _) : _) = opt_WarnUnusedImports
-       wanted ((_,FromLocalDefn _)    : _) = opt_WarnUnusedImports
-
-       pp_imp = sep [text "Warning: the following are unused:",
-                     nest 4 (vcat (map pp_group groups))]
-
-       pp_group group = sep [msg <> char ':',
-                             nest 4 (sep (map (pprOccName . nameOccName . fst) group))]
-                      where
-                        his = case group of
-                                 ((_,his) : _) -> his
-
-                        msg = case his of
-                                 FromImportDecl m _ -> text "Imported from" <+> pprModule m
-                                 FromLocalDefn _    -> text "Locally defined"   
-
+  = 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
-    if null groups
-    then returnRn ()
-    else addWarnRn pp_imp
+    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 all_decls