[project @ 2000-10-24 15:55:35 by simonpj]
[ghc-hetmet.git] / ghc / compiler / rename / RnNames.lhs
index 275f830..a33df88 100644 (file)
@@ -10,46 +10,40 @@ module RnNames (
 
 #include "HsVersions.h"
 
-import CmdLineOpts    ( opt_NoImplicitPrelude, opt_WarnDuplicateExports, 
-                       opt_SourceUnchanged, opt_WarnUnusedBinds
-                     )
-
-import HsSyn   ( HsModule(..), HsDecl(..), TyClDecl(..),
-                 IE(..), ieName, 
-                 ForeignDecl(..), ForKind(..), isDynamicExtName,
-                 FixitySig(..), Sig(..), ImportDecl(..),
-                 collectTopBinders
-               )
-import RdrHsSyn        ( RdrNameIE, RdrNameImportDecl,
-                 RdrNameHsModule, RdrNameHsDecl
-               )
-import RnIfaces        ( getInterfaceExports, getDeclBinders, getDeclSysBinders,
-                 recordSlurp, checkUpToDate
-               )
+import CmdLineOpts     ( DynFlag(..), opt_NoImplicitPrelude )
+
+import HsSyn           ( HsModule(..), HsDecl(..), IE(..), ieName, ImportDecl(..),
+                         ForeignDecl(..), ForKind(..), isDynamicExtName,
+                         collectTopBinders
+                       )
+import RdrHsSyn                ( RdrNameIE, RdrNameImportDecl,
+                         RdrNameHsModule, RdrNameHsDecl
+                       )
+import RnIfaces                ( getInterfaceExports, recordLocalSlurps )
+import RnHiFiles       ( getTyClDeclBinders )
 import RnEnv
 import RnMonad
 
 import FiniteMap
-import PrelMods
-import PrelInfo ( main_RDR )
-import UniqFM  ( lookupUFM )
-import Bag     ( bagToList )
-import Module  ( ModuleName, mkThisModule, pprModuleName, WhereFrom(..) )
+import PrelNames       ( pRELUDE_Name, mAIN_Name, main_RDR )
+import UniqFM          ( lookupUFM )
+import Bag             ( bagToList )
+import Module          ( ModuleName, mkModuleInThisPackage, WhereFrom(..) )
 import NameSet
-import Name    ( Name, ExportFlag(..), ImportReason(..), Provenance(..),
-                 isLocallyDefined, setNameProvenance,
-                 nameOccName, getSrcLoc, pprProvenance, getNameProvenance
-               )
-import RdrName ( RdrName, rdrNameOcc, setRdrNameOcc, mkRdrQual, mkRdrUnqual, isQual )
-import OccName ( setOccNameSpace, dataName )
-import SrcLoc  ( SrcLoc )
-import NameSet ( elemNameSet, emptyNameSet )
+import Name            ( Name, nameSrcLoc,
+                         setLocalNameSort, nameOccName,  nameEnvElts )
+import HscTypes                ( Provenance(..), ImportReason(..), GlobalRdrEnv,
+                         GenAvailInfo(..), AvailInfo, Avails, AvailEnv )
+import RdrName         ( RdrName, rdrNameOcc, setRdrNameOcc, mkRdrQual, mkRdrUnqual, isUnqual )
+import OccName         ( setOccNameSpace, dataName )
+import NameSet         ( elemNameSet, emptyNameSet )
+import SrcLoc          ( SrcLoc )
 import Outputable
-import Maybes  ( maybeToBool, catMaybes, mapMaybe )
-import UniqFM   ( emptyUFM, listToUFM, plusUFM_C )
-import Unique  ( getUnique )
-import Util    ( removeDups, equivClassesByUniq, sortLt )
-import List    ( partition )
+import Maybes          ( maybeToBool, catMaybes, mapMaybe )
+import UniqFM          ( emptyUFM, listToUFM )
+import ListSetOps      ( removeDups )
+import Util            ( sortLt )
+import List            ( partition )
 \end{code}
 
 
@@ -62,27 +56,26 @@ import List ( partition )
 
 \begin{code}
 getGlobalNames :: RdrNameHsModule
-              -> RnMG (Maybe (ExportEnv, 
-                              GlobalRdrEnv,
-                              FixityEnv,        -- Fixities for local decls only
-                              AvailEnv          -- Maps a name to its parent AvailInfo
-                                                -- Just for in-scope things only
+              -> RnMG (Maybe (GlobalRdrEnv,    -- Maps all in-scope things
+                              GlobalRdrEnv,    -- Maps just *local* things
+                              Avails,          -- The exported stuff
+                              AvailEnv         -- Maps a name to its parent AvailInfo
+                                               -- Just for in-scope things only
                               ))
                        -- Nothing => no need to recompile
 
 getGlobalNames (HsModule this_mod _ exports imports decls _ mod_loc)
   =    -- These two fix-loops are to get the right
        -- provenance information into a Name
-    fixRn (\ ~(rec_gbl_env, rec_exported_avails, _) ->
+    fixRn ( \ ~(Just (rec_gbl_env, _, rec_export_avails, _)) ->
 
        let
           rec_unqual_fn :: Name -> Bool        -- Is this chap in scope unqualified?
           rec_unqual_fn = unQualInScope rec_gbl_env
 
-          rec_exp_fn :: Name -> ExportFlag
-          rec_exp_fn = mk_export_fn (availsToNameSet rec_exported_avails)
+          rec_exp_fn :: Name -> Bool
+          rec_exp_fn = mk_export_fn (availsToNameSet rec_export_avails)
        in
-       setModuleRn this_mod                    $
 
                -- PROCESS LOCAL DECLS
                -- Do these *first* so that the correct provenance gets
@@ -113,74 +106,40 @@ getGlobalNames (HsModule this_mod _ exports imports decls _ mod_loc)
 
            all_avails :: ExportAvails
            all_avails = foldr plusExportAvails local_mod_avails (imp_avails_s2 ++ imp_avails_s1)
+           (_, global_avail_env) = all_avails
        in
 
-       -- TRY FOR EARLY EXIT
-       -- We can't go for an early exit before this because we have to check
-       -- for name clashes.  Consider:
-       --
-       --      module A where          module B where
-       --         import B                h = True
-       --         f = h
-       --
-       -- Suppose I've compiled everything up, and then I add a
-       -- new definition to module B, that defines "f".
-       --
-       -- Then I must detect the name clash in A before going for an early
-       -- exit.  The early-exit code checks what's actually needed from B
-       -- to compile A, and of course that doesn't include B.f.  That's
-       -- why we wait till after the plusEnv stuff to do the early-exit.
-      checkEarlyExit this_mod                  `thenRn` \ up_to_date ->
-      if up_to_date then
-       returnRn (gbl_env, junk_exp_fn, Nothing)
-      else
-       -- RECORD BETTER PROVENANCES IN THE CACHE
-       -- The names in the envirnoment have better provenances (e.g. imported on line x)
-       -- than the names in the name cache.  We update the latter now, so that we
-       -- we start renaming declarations we'll get the good names
-       -- The isQual is because the qualified name is always in scope
-      updateProvenances (concat [names | (rdr_name, names) <- rdrEnvToList imp_gbl_env, 
-                                         isQual rdr_name])     `thenRn_`
-
-       -- PROCESS EXPORT LISTS
-      exportsFromAvail this_mod exports all_avails gbl_env      `thenRn` \ exported_avails ->
-
-       -- DONE
-      returnRn (gbl_env, exported_avails, Just all_avails)
-    )          `thenRn` \ (gbl_env, exported_avails, maybe_stuff) ->
-
-    case maybe_stuff of {
-       Nothing -> returnRn Nothing ;
-       Just all_avails ->
-
-       -- DEAL WITH FIXITIES
-   fixitiesFromLocalDecls gbl_env decls                `thenRn` \ local_fixity_env ->
-   let
-       -- Export only those fixities that are for names that are
-       --      (a) defined in this module
-       --      (b) exported
-       exported_fixities :: [(Name,Fixity)]
-       exported_fixities = [(name,fixity)
-                           | FixitySig name fixity _ <- nameEnvElts local_fixity_env,
-                             isLocallyDefined name
-                           ]
-
-       -- CONSTRUCT RESULTS
-       export_mods = case exports of
-                       Nothing -> []
-                       Just es -> [mod | IEModuleContents mod <- es, mod /= this_mod]
-
-       export_env            = ExportEnv exported_avails exported_fixities export_mods
-       (_, global_avail_env) = all_avails
-   in
-   traceRn (text "fixity env" <+> vcat (map ppr (nameEnvElts local_fixity_env)))       `thenRn_`
-
-   returnRn (Just (export_env, gbl_env, local_fixity_env, global_avail_env))
-   }
+               -- TRY FOR EARLY EXIT
+               -- We can't go for an early exit before this because we have to check
+               -- for name clashes.  Consider:
+               --
+               --      module A where          module B where
+               --         import B                h = True
+               --         f = h
+               --
+               -- Suppose I've compiled everything up, and then I add a
+               -- new definition to module B, that defines "f".
+               --
+               -- Then I must detect the name clash in A before going for an early
+               -- exit.  The early-exit code checks what's actually needed from B
+               -- to compile A, and of course that doesn't include B.f.  That's
+               -- why we wait till after the plusEnv stuff to do the early-exit.
+               
+       -- Check For early exit
+       checkErrsRn                             `thenRn` \ no_errs_so_far ->
+        if not no_errs_so_far then
+               -- Found errors already, so exit now
+               returnRn Nothing
+       else
+       
+               -- PROCESS EXPORT LISTS
+       exportsFromAvail this_mod exports all_avails gbl_env    `thenRn` \ export_avails ->
+       
+       
+               -- ALL DONE
+       returnRn (Just (gbl_env, local_gbl_env, export_avails, global_avail_env))
+   )
   where
-    junk_exp_fn = error "RnNames:export_fn"
-
     all_imports = prel_imports ++ imports
 
        -- NB: opt_NoImplicitPrelude is slightly different to import Prelude ();
@@ -203,30 +162,6 @@ getGlobalNames (HsModule this_mod _ exports imports decls _ mod_loc)
 \end{code}
        
 \begin{code}
-checkEarlyExit mod
-  = checkErrsRn                                `thenRn` \ no_errs_so_far ->
-    if not no_errs_so_far then
-       -- Found errors already, so exit now
-       returnRn True
-    else
-
-    traceRn (text "Considering whether compilation is required...")    `thenRn_`
-    if not opt_SourceUnchanged then
-       -- Source code changed and no errors yet... carry on 
-       traceRn (nest 4 (text "source file changed or recompilation check turned off")) `thenRn_` 
-       returnRn False
-    else
-
-       -- Unchanged source, and no errors yet; see if usage info
-       -- up to date, and exit if so
-    checkUpToDate mod                                          `thenRn` \ up_to_date ->
-    (if up_to_date 
-       then putDocRn (text "Compilation IS NOT required")
-       else returnRn ())                                       `thenRn_`
-    returnRn up_to_date
-\end{code}
-       
-\begin{code}
 importsFromImportDecl :: (Name -> Bool)                -- OK to omit qualifier
                      -> RdrNameImportDecl
                      -> RnMG (GlobalRdrEnv, 
@@ -244,33 +179,22 @@ importsFromImportDecl is_unqual (ImportDecl imp_mod_name from qual_only as_mod i
 
     filterImports imp_mod_name import_spec avails   `thenRn` \ (filtered_avails, hides, explicits) ->
 
+    let
+       mk_provenance name = NonLocalDef (UserImport imp_mod iloc (name `elemNameSet` explicits)) 
+                                        (is_unqual name)
+    in
+
     qualifyImports imp_mod_name
                   (not qual_only)      -- Maybe want unqualified names
                   as_mod hides
-                  (improveAvails imp_mod iloc explicits 
-                                 is_unqual filtered_avails)
-
-
-improveAvails imp_mod iloc explicits is_unqual avails
-       -- We 'improve' the provenance by setting
-       --      (a) the import-reason field, so that the Name says how it came into scope
-       --              including whether it's explicitly imported
-       --      (b) the print-unqualified field
-  = map improve_avail avails
-  where
-    improve_avail (Avail n)      = Avail (improve n)
-    improve_avail (AvailTC n ns) = AvailTC n (map improve ns)  -- n doesn't matter
-
-    improve name = setNameProvenance name 
-                       (NonLocalDef (UserImport imp_mod iloc (is_explicit name)) 
-                                    (is_unqual name))
-    is_explicit name  = name `elemNameSet` explicits
+                  mk_provenance
+                  filtered_avails
 \end{code}
 
 
 \begin{code}
 importsFromLocalDecls mod_name rec_exp_fn decls
-  = mapRn (getLocalDeclBinders newLocalName) decls     `thenRn` \ avails_s ->
+  = mapRn (getLocalDeclBinders (newLocalName mod rec_exp_fn)) decls    `thenRn` \ avails_s ->
 
     let
        avails = concat avails_s
@@ -285,82 +209,66 @@ importsFromLocalDecls mod_name rec_exp_fn decls
     mapRn_ (addErrRn . dupDeclErr) dups                `thenRn_` 
 
        -- Record that locally-defined things are available
-    mapRn_ (recordSlurp Nothing) avails                `thenRn_`
+    recordLocalSlurps avails                   `thenRn_`
 
        -- Build the environment
     qualifyImports mod_name 
-                  True         -- Want unqualified names
-                  Nothing      -- no 'as M'
-                  []           -- Hide nothing
+                  True                 -- Want unqualified names
+                  Nothing              -- no 'as M'
+                  []                   -- Hide nothing
+                  (\n -> LocalDef)     -- Provenance is local
                   avails
-
   where
-    mod = mkThisModule mod_name
-
-    newLocalName rdr_name loc 
-       = (if isQual rdr_name then
-               qualNameErr (text "the binding for" <+> quotes (ppr rdr_name)) (rdr_name,loc)
-               -- There should never be a qualified name in a binding position (except in instance decls)
-               -- The parser doesn't check this because the same parser parses instance decls
-           else 
-               returnRn ())                    `thenRn_`
-
-         newLocalTopBinder mod (rdrNameOcc rdr_name) rec_exp_fn loc
+    mod = mkModuleInThisPackage mod_name
 
-
-getLocalDeclBinders :: (RdrName -> SrcLoc -> RnMG Name)        -- New-name function
-                   -> RdrNameHsDecl
-                   -> RnMG Avails
+---------------------------
+getLocalDeclBinders :: (RdrName -> SrcLoc -> RnMG Name)
+                   -> RdrNameHsDecl -> RnMG Avails
 getLocalDeclBinders new_name (ValD binds)
   = mapRn do_one (bagToList (collectTopBinders binds))
   where
     do_one (rdr_name, loc) = new_name rdr_name loc     `thenRn` \ name ->
                             returnRn (Avail name)
 
-getLocalDeclBinders new_name decl
-  = getDeclBinders new_name decl       `thenRn` \ maybe_avail ->
-    case maybe_avail of
-       Nothing    -> returnRn []               -- Instance decls and suchlike
-       Just avail -> getDeclSysBinders new_sys_name decl               `thenRn_`  
-                     returnRn [avail]
-  where
-       -- The getDeclSysBinders is just to get the names of superclass selectors
-       -- etc, into the cache
-    new_sys_name rdr_name loc = newImplicitBinder (rdrNameOcc rdr_name) loc
+getLocalDeclBinders new_name (TyClD tycl_decl)
+  = getTyClDeclBinders new_name tycl_decl      `thenRn` \ avail ->
+    returnRn [avail]
 
-fixitiesFromLocalDecls :: GlobalRdrEnv -> [RdrNameHsDecl] -> RnMG FixityEnv
-fixitiesFromLocalDecls gbl_env decls
-  = foldlRn getFixities emptyNameEnv decls
-  where
-    getFixities :: FixityEnv -> RdrNameHsDecl -> RnMG FixityEnv
-    getFixities acc (FixD fix)
-      = fix_decl acc fix
-
-    getFixities acc (TyClD (ClassDecl _ _ _ _ sigs _ _ _ _ _ _ _))
-      = foldlRn fix_decl acc [sig | FixSig sig <- sigs]
-               -- Get fixities from class decl sigs too.
-    getFixities acc other_decl
-      = returnRn acc
-
-    fix_decl acc sig@(FixitySig rdr_name fixity loc)
-       =       -- Check for fixity decl for something not declared
-         case lookupRdrEnv gbl_env rdr_name of {
-           Nothing | opt_WarnUnusedBinds 
-                   -> pushSrcLocRn loc (addWarnRn (unusedFixityDecl rdr_name fixity))
-                      `thenRn_` returnRn acc 
-                   | otherwise -> returnRn acc ;
-       
-           Just (name:_) ->
+getLocalDeclBinders new_name (ForD (ForeignDecl nm kind _ ext_nm _ loc))
+  | binds_haskell_name kind
+  = new_name nm loc                `thenRn` \ name ->
+    returnRn [Avail name]
 
-               -- Check for duplicate fixity decl
-         case lookupNameEnv acc name of {
-           Just (FixitySig _ _ loc') -> addErrRn (dupFixityDecl rdr_name loc loc')
-                                        `thenRn_` returnRn acc ;
-
-           Nothing -> returnRn (addToNameEnv acc name (FixitySig name fixity loc))
-         }}
+  | otherwise          -- a foreign export
+  = lookupOrigName nm `thenRn_` 
+    returnRn []
+  where
+    binds_haskell_name (FoImport _) = True
+    binds_haskell_name FoLabel      = True
+    binds_haskell_name FoExport     = isDynamicExtName ext_nm
+
+getLocalDeclBinders new_name (FixD _)    = returnRn []
+getLocalDeclBinders new_name (DeprecD _) = returnRn []
+getLocalDeclBinders new_name (DefD _)    = returnRn []
+getLocalDeclBinders new_name (InstD _)   = returnRn []
+getLocalDeclBinders new_name (RuleD _)   = returnRn []
+
+
+---------------------------
+newLocalName mod rec_exp_fn rdr_name loc 
+  = check_unqual rdr_name loc          `thenRn_`
+    newTopBinder mod rdr_name loc      `thenRn` \ name ->
+    returnRn (setLocalNameSort name (rec_exp_fn name))
+  where
+       -- There should never be a qualified name in a binding position (except in instance decls)
+       -- The parser doesn't check this because the same parser parses instance decls
+    check_unqual rdr_name loc
+       | isUnqual rdr_name = returnRn ()
+       | otherwise         = qualNameErr (text "the binding for" <+> quotes (ppr rdr_name)) 
+                                         (rdr_name,loc)
 \end{code}
 
+
 %************************************************************************
 %*                                                                     *
 \subsection{Filtering imports}
@@ -474,10 +382,11 @@ qualifyImports :: ModuleName              -- Imported module
               -> Bool                  -- True <=> want unqualified import
               -> Maybe ModuleName      -- Optional "as M" part 
               -> [AvailInfo]           -- What's to be hidden
+              -> (Name -> Provenance)
               -> Avails                -- Whats imported and how
               -> RnMG (GlobalRdrEnv, ExportAvails)
 
-qualifyImports this_mod unqual_imp as_mod hides avails
+qualifyImports this_mod unqual_imp as_mod hides mk_provenance avails
   = 
        -- Make the name environment.  We're talking about a 
        -- single module here, so there must be no name clashes.
@@ -507,9 +416,10 @@ qualifyImports this_mod unqual_imp as_mod hides avails
        | unqual_imp = env2
        | otherwise  = env1
        where
-         env1 = addOneToGlobalRdrEnv env  (mkRdrQual qual_mod occ) name
-         env2 = addOneToGlobalRdrEnv env1 (mkRdrUnqual occ)        name
+         env1 = addOneToGlobalRdrEnv env  (mkRdrQual qual_mod occ) (name,prov)
+         env2 = addOneToGlobalRdrEnv env1 (mkRdrUnqual occ)        (name,prov)
          occ  = nameOccName name
+         prov = mk_provenance name
 
     del_avail env avail = foldl delOneFromGlobalRdrEnv env rdr_names
                        where
@@ -606,8 +516,10 @@ exportsFromAvail this_mod Nothing export_avails global_name_env
 exportsFromAvail this_mod (Just export_items) 
                 (mod_avail_env, entity_avail_env)
                 global_name_env
-  = foldlRn exports_from_item
-           ([], emptyFM, emptyAvailEnv) export_items   `thenRn` \ (_, _, export_avail_map) ->
+  = doptRn Opt_WarnDuplicateExports            `thenRn` \ warn_dup_exports ->
+    foldlRn (exports_from_item warn_dup_exports)
+           ([], emptyFM, emptyAvailEnv) export_items
+                                               `thenRn` \ (_, _, export_avail_map) ->
     let
        export_avails :: [AvailInfo]
        export_avails   = nameEnvElts export_avail_map
@@ -615,12 +527,11 @@ exportsFromAvail this_mod (Just export_items)
     returnRn export_avails
 
   where
-    exports_from_item :: ExportAccum -> RdrNameIE -> RnMG ExportAccum
+    exports_from_item :: Bool -> ExportAccum -> RdrNameIE -> RnMG ExportAccum
 
-    exports_from_item acc@(mods, occs, avails) ie@(IEModuleContents mod)
+    exports_from_item warn_dups acc@(mods, occs, avails) ie@(IEModuleContents mod)
        | mod `elem` mods       -- Duplicate export of M
-       = warnCheckRn opt_WarnDuplicateExports
-                     (dupModuleExport mod)     `thenRn_`
+       = warnCheckRn warn_dups (dupModuleExport mod)   `thenRn_`
          returnRn acc
 
        | otherwise
@@ -633,12 +544,12 @@ exportsFromAvail this_mod (Just export_items)
                                   in
                                   returnRn (mod:mods, occs', avails')
 
-    exports_from_item acc@(mods, occs, avails) ie
+    exports_from_item warn_dups acc@(mods, occs, avails) ie
        | not (maybeToBool maybe_in_scope) 
        = failWithRn acc (unknownNameErr (ieName ie))
 
        | not (null dup_names)
-       = addNameClashErrRn rdr_name (name:dup_names)   `thenRn_`
+       = addNameClashErrRn rdr_name ((name,prov):dup_names)    `thenRn_`
          returnRn acc
 
 #ifdef DEBUG
@@ -662,7 +573,7 @@ exportsFromAvail this_mod (Just export_items)
        where
          rdr_name        = ieName ie
           maybe_in_scope  = lookupFM global_name_env rdr_name
-         Just (name:dup_names) = maybe_in_scope
+         Just ((name,prov):dup_names) = maybe_in_scope
          maybe_avail        = lookupUFM entity_avail_env name
          Just avail         = maybe_avail
          maybe_export_avail = filterAvail ie avail
@@ -677,14 +588,15 @@ exportsFromAvail this_mod (Just export_items)
 
 check_occs :: RdrNameIE -> ExportOccMap -> AvailInfo -> RnMG ExportOccMap
 check_occs ie occs avail 
-  = foldlRn check occs (availNames avail)
+  = doptRn Opt_WarnDuplicateExports    `thenRn` \ warn_dup_exports ->
+    foldlRn (check warn_dup_exports) occs (availNames avail)
   where
-    check occs name
+    check warn_dup occs name
       = case lookupFM occs name_occ of
          Nothing           -> returnRn (addToFM occs name_occ (name, ie))
          Just (name', ie') 
            | name == name' ->  -- Duplicate export
-                               warnCheckRn opt_WarnDuplicateExports
+                               warnCheckRn warn_dup
                                            (dupExportWarn name_occ ie ie')
                                `thenRn_` returnRn occs
 
@@ -693,11 +605,8 @@ check_occs ie occs avail
       where
        name_occ = nameOccName name
        
-mk_export_fn :: NameSet -> (Name -> ExportFlag)
-mk_export_fn exported_names
-  = \name -> if name `elemNameSet` exported_names
-            then Exported
-            else NotExported
+mk_export_fn :: NameSet -> (Name -> Bool)      -- True => exported
+mk_export_fn exported_names = \name ->  name `elemNameSet` exported_names
 \end{code}
 
 %************************************************************************
@@ -708,7 +617,7 @@ mk_export_fn exported_names
 
 \begin{code}
 badImportItemErr mod ie
-  = sep [ptext SLIT("Module"), quotes (pprModuleName mod), 
+  = sep [ptext SLIT("Module"), quotes (ppr mod), 
         ptext SLIT("does not export"), quotes (ppr ie)]
 
 dodgyImportWarn mod item = dodgyMsg (ptext SLIT("import")) item
@@ -720,7 +629,7 @@ dodgyMsg kind item@(IEThingAll tc)
          ptext SLIT("but it has none; it is a type synonym or abstract type or class") ]
          
 modExportErr mod
-  = hsep [ ptext SLIT("Unknown module in export list: module"), quotes (pprModuleName mod)]
+  = hsep [ ptext SLIT("Unknown module in export list: module"), quotes (ppr mod)]
 
 exportItemErr export_item
   = sep [ ptext SLIT("The export item") <+> quotes (ppr export_item),
@@ -733,13 +642,10 @@ exportClashErr occ_name ie1 ie2
 
 dupDeclErr (n:ns)
   = vcat [ptext SLIT("Multiple declarations of") <+> quotes (ppr n),
-         nest 4 (vcat (map pp sorted_ns))]
+         nest 4 (vcat (map ppr sorted_locs))]
   where
-    sorted_ns = sortLt occ'ed_before (n:ns)
-
-    occ'ed_before a b = LT == compare (getSrcLoc a) (getSrcLoc b)
-
-    pp n      = pprProvenance (getNameProvenance n)
+    sorted_locs = sortLt occ'ed_before (map nameSrcLoc (n:ns))
+    occ'ed_before a b = LT == compare a b
 
 dupExportWarn occ_name ie1 ie2
   = hsep [quotes (ppr occ_name), 
@@ -748,14 +654,6 @@ dupExportWarn occ_name ie1 ie2
 
 dupModuleExport mod
   = hsep [ptext SLIT("Duplicate"),
-         quotes (ptext SLIT("Module") <+> pprModuleName mod), 
+         quotes (ptext SLIT("Module") <+> ppr mod), 
           ptext SLIT("in export list")]
-
-unusedFixityDecl rdr_name fixity
-  = hsep [ptext SLIT("Unused fixity declaration for"), quotes (ppr rdr_name)]
-
-dupFixityDecl rdr_name loc1 loc2
-  = vcat [ptext SLIT("Multiple fixity declarations for") <+> quotes (ppr rdr_name),
-         ptext SLIT("at ") <+> ppr loc1,
-         ptext SLIT("and") <+> ppr loc2]
 \end{code}