[project @ 2001-03-08 12:07:38 by simonpj]
[ghc-hetmet.git] / ghc / compiler / rename / RnEnv.lhs
index 0dc76fe..c8090f9 100644 (file)
@@ -8,26 +8,45 @@ module RnEnv where            -- Export everything
 
 #include "HsVersions.h"
 
+import {-# SOURCE #-} RnHiFiles
+
+import HscTypes                ( ModIface(..) )
 import HsSyn
+import RnHsSyn         ( RenamedHsDecl )
 import RdrHsSyn                ( RdrNameIE )
 import RdrName         ( RdrName, rdrNameModule, rdrNameOcc, isQual, isUnqual, isOrig,
                          mkRdrUnqual, mkRdrQual, qualifyRdrName, lookupRdrEnv, foldRdrEnv
                        )
 import HsTypes         ( hsTyVarName, replaceTyVarName )
 import HscTypes                ( Provenance(..), pprNameProvenance, hasBetterProv,
-                         ImportReason(..), GlobalRdrEnv, AvailEnv,
-                         AvailInfo, Avails, GenAvailInfo(..), OrigNameEnv(..) )
+                         ImportReason(..), GlobalRdrEnv, GlobalRdrElt(..), AvailEnv,
+                         AvailInfo, Avails, GenAvailInfo(..), NameSupply(..), 
+                         Deprecations(..), lookupDeprec,
+                         extendLocalRdrEnv
+                       )
 import RnMonad
-import Name            ( Name, NamedThing(..),
+import Name            ( Name,
                          getSrcLoc, 
                          mkLocalName, mkGlobalName,
                          mkIPName, nameOccName, nameModule_maybe,
                          setNameModuleAndLoc
                        )
-import Name            ( extendNameEnv_C, plusNameEnv_C, nameEnvElts )
+import NameEnv
 import NameSet
 import OccName         ( OccName, occNameUserString, occNameFlavour )
-import Module          ( ModuleName, moduleName, mkVanillaModule, mkSysModuleNameFS, moduleNameFS )
+import Module          ( ModuleName, moduleName, mkVanillaModule, 
+                         mkSysModuleNameFS, moduleNameFS, WhereFrom(..) )
+import PrelNames       ( mkUnboundName, syntaxList, SyntaxMap, vanillaSyntaxMap,
+                         derivingOccurrences,
+                         mAIN_Name, pREL_MAIN_Name, 
+                         ioTyConName, integerTyConName, doubleTyConName, intTyConName, 
+                         boolTyConName, funTyConName,
+                         unpackCStringName, unpackCStringFoldrName, unpackCStringUtf8Name,
+                         eqStringName, printName, 
+                         hasKey, fractionalClassKey, numClassKey,
+                         bindIOName, returnIOName, failIOName
+                       )
+import TysWiredIn      ( unitTyCon )   -- A little odd
 import FiniteMap
 import UniqSupply
 import SrcLoc          ( SrcLoc, noSrcLoc )
@@ -35,7 +54,8 @@ import Outputable
 import ListSetOps      ( removeDups, equivClasses )
 import Util            ( sortLt )
 import List            ( nub )
-import PrelNames       ( mkUnboundName )
+import UniqFM          ( lookupWithDefaultUFM )
+import Maybes          ( orElse )
 import CmdLineOpts
 import FastString      ( FastString )
 \end{code}
@@ -57,7 +77,6 @@ newTopBinder :: Module -> RdrName -> SrcLoc -> RnM d Name
 
 newTopBinder mod rdr_name loc
   =    -- First check the cache
-    -- traceRn (text "newTopBinder" <+> ppr mod <+> ppr loc) `thenRn_`
 
        -- 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
@@ -71,7 +90,7 @@ newTopBinder mod rdr_name loc
     let 
        occ = rdrNameOcc rdr_name
        key = (moduleName mod, occ)
-       cache = origNames name_supply
+       cache = nsNames name_supply
     in
     case lookupFM cache key of
 
@@ -86,8 +105,8 @@ newTopBinder mod rdr_name loc
                        new_name  = setNameModuleAndLoc name mod loc
                        new_cache = addToFM cache key new_name
                     in
-                    setNameSupplyRn (name_supply {origNames = new_cache})      `thenRn_`
-                    traceRn (text "newTopBinder: overwrite" <+> ppr new_name) `thenRn_`
+                    setNameSupplyRn (name_supply {nsNames = new_cache})        `thenRn_`
+--                  traceRn (text "newTopBinder: overwrite" <+> ppr new_name) `thenRn_`
                     returnRn new_name
                     
        -- Miss in the cache!
@@ -95,13 +114,13 @@ newTopBinder mod rdr_name loc
        -- Even for locally-defined names we use implicitImportProvenance; 
        -- updateProvenances will set it to rights
        Nothing -> let
-                       (us', us1) = splitUniqSupply (origNS name_supply)
+                       (us', us1) = splitUniqSupply (nsUniqs name_supply)
                        uniq       = uniqFromSupply us1
                        new_name   = mkGlobalName uniq mod occ loc
                        new_cache  = addToFM cache key new_name
                   in
-                  setNameSupplyRn (name_supply {origNS = us', origNames = new_cache})  `thenRn_`
-                  traceRn (text "newTopBinder: new" <+> ppr new_name) `thenRn_`
+                  setNameSupplyRn (name_supply {nsUniqs = us', nsNames = new_cache})   `thenRn_`
+--                traceRn (text "newTopBinder: new" <+> ppr new_name) `thenRn_`
                   returnRn new_name
 
 
@@ -127,17 +146,17 @@ newGlobalName mod_name occ
   = getNameSupplyRn            `thenRn` \ name_supply ->
     let
        key = (mod_name, occ)
-       cache = origNames name_supply
+       cache = nsNames name_supply
     in
     case lookupFM cache key of
        Just name -> -- traceRn (text "newGlobalName: hit" <+> ppr name) `thenRn_`
                     returnRn name
 
-       Nothing   -> setNameSupplyRn (name_supply {origNS = us', origNames = new_cache})  `thenRn_`
+       Nothing   -> setNameSupplyRn (name_supply {nsUniqs = us', nsNames = new_cache})  `thenRn_`
                     -- traceRn (text "newGlobalName: new" <+> ppr name)                  `thenRn_`
                     returnRn name
                  where
-                    (us', us1) = splitUniqSupply (origNS name_supply)
+                    (us', us1) = splitUniqSupply (nsUniqs name_supply)
                     uniq       = uniqFromSupply us1
                     mod        = mkVanillaModule mod_name
                     name       = mkGlobalName uniq mod occ noSrcLoc
@@ -146,14 +165,14 @@ newGlobalName mod_name occ
 newIPName rdr_name
   = getNameSupplyRn            `thenRn` \ name_supply ->
     let
-       ipcache = origIParam name_supply
+       ipcache = nsIPs name_supply
     in
     case lookupFM ipcache key of
        Just name -> returnRn name
-       Nothing   -> setNameSupplyRn (name_supply {origNS = us', origIParam = new_ipcache})     `thenRn_`
+       Nothing   -> setNameSupplyRn (name_supply {nsUniqs = us', nsIPs = new_ipcache}) `thenRn_`
                     returnRn name
                  where
-                    (us', us1)  = splitUniqSupply (origNS name_supply)
+                    (us', us1)  = splitUniqSupply (nsUniqs name_supply)
                     uniq        = uniqFromSupply us1
                     name        = mkIPName uniq key
                     new_ipcache = addToFM ipcache key name
@@ -177,13 +196,12 @@ lookupBndrRn rdr_name
 
 lookupTopBndrRn rdr_name
   = getModeRn  `thenRn` \ mode ->
-    case mode of 
-       InterfaceMode -> lookupIfaceName rdr_name       
-
-       SourceMode    -> -- Source mode, so look up a *qualified* version
-                        -- of the name, so that we get the right one even
-                        -- if there are many with the same occ name
-                        -- There must *be* a binding
+    if isInterfaceMode mode
+       then lookupIfaceName rdr_name   
+       else     -- Source mode, so look up a *qualified* version
+                -- of the name, so that we get the right one even
+                -- if there are many with the same occ name
+                -- There must *be* a binding
                getModuleRn             `thenRn` \ mod ->
                getGlobalNameEnv        `thenRn` \ global_env ->
                lookupSrcName global_env (qualifyRdrName (moduleName mod) rdr_name)
@@ -216,11 +234,46 @@ lookupOccRn rdr_name
 
 lookupGlobalOccRn rdr_name
   = getModeRn          `thenRn` \ mode ->
-    case mode of 
-       SourceMode    -> getGlobalNameEnv                       `thenRn` \ global_env ->
-                        lookupSrcName global_env rdr_name
+    if (isInterfaceMode mode)
+       then lookupIfaceName rdr_name
+       else 
 
-       InterfaceMode -> lookupIfaceName rdr_name
+    getGlobalNameEnv   `thenRn` \ global_env ->
+    case mode of 
+       SourceMode -> lookupSrcName global_env rdr_name
+
+       CmdLineMode
+        | not (isQual rdr_name) -> 
+               lookupSrcName global_env rdr_name
+
+               -- We allow qualified names on the command line to refer to 
+               -- *any* name exported by any module in scope, just as if 
+               -- there was an "import qualified M" declaration for every 
+               -- module.
+               --
+               -- First look up the name in the normal environment.  If
+               -- it isn't there, we manufacture a new occurrence of an
+               -- original name.
+        | otherwise -> 
+               case lookupRdrEnv global_env rdr_name of
+                      Just _  -> lookupSrcName global_env rdr_name
+                      Nothing -> lookupQualifiedName rdr_name
+
+-- a qualified name on the command line can refer to any module at all: we
+-- try to load the interface if we don't already have it.
+lookupQualifiedName :: RdrName -> RnM d Name
+lookupQualifiedName rdr_name
+ = let 
+       mod = rdrNameModule rdr_name
+       occ = rdrNameOcc rdr_name
+   in
+   loadInterface (ppr rdr_name) mod ImportBySystem `thenRn` \ iface ->
+   case  [ name | (_,avails) <- mi_exports iface,
+          avail             <- avails,
+          name              <- availNames avail,
+          nameOccName name == occ ] of
+      (n:ns) -> ASSERT (null ns) returnRn n
+      _      -> failWithRn (mkUnboundName rdr_name) (unknownNameErr rdr_name)
 
 lookupSrcName :: GlobalRdrEnv -> RdrName -> RnM d Name
 -- NB: passed GlobalEnv explicitly, not necessarily in RnMS monad
@@ -230,11 +283,13 @@ lookupSrcName global_env rdr_name
 
   | otherwise
   = case lookupRdrEnv global_env rdr_name of
-       Just [(name,_)]         -> returnRn name
-       Just stuff@((name,_):_) -> addNameClashErrRn rdr_name stuff     `thenRn_`
-                                  returnRn name
-       Nothing                 -> failWithRn (mkUnboundName rdr_name)
-                                             (unknownNameErr rdr_name)
+       Just [GRE name _ Nothing]       -> returnRn name
+       Just [GRE name _ (Just deprec)] -> warnDeprec name deprec       `thenRn_`
+                                          returnRn name
+       Just stuff@(GRE name _ _ : _)   -> addNameClashErrRn rdr_name stuff     `thenRn_`
+                                          returnRn name
+       Nothing                         -> failWithRn (mkUnboundName rdr_name)
+                                                     (unknownNameErr rdr_name)
 
 lookupOrigName :: RdrName -> RnM d Name 
 lookupOrigName rdr_name
@@ -270,7 +325,6 @@ calls it at all I think).
 
   \fbox{{\em Jan 98: this comment is wrong: @rnHsType@ uses it quite a bit.}}
 
-
 \begin{code}
 lookupOrigNames :: [RdrName] -> RnM d NameSet
 lookupOrigNames rdr_names
@@ -278,10 +332,10 @@ lookupOrigNames rdr_names
     returnRn (mkNameSet names)
 \end{code}
 
-lookupSysBinder is used for the "system binders" of a type, class, or instance decl.
-It ensures that the module is set correctly in the name cache, and sets the provenance
-on the returned name too.  The returned name will end up actually in the type, class,
-or instance.
+lookupSysBinder is used for the "system binders" of a type, class, or
+instance decl.  It ensures that the module is set correctly in the
+name cache, and sets the provenance on the returned name too.  The
+returned name will end up actually in the type, class, or instance.
 
 \begin{code}
 lookupSysBinder rdr_name
@@ -292,6 +346,119 @@ lookupSysBinder rdr_name
 \end{code}
 
 
+%*********************************************************
+%*                                                     *
+\subsection{Implicit free vars and sugar names}
+%*                                                     *
+%*********************************************************
+
+@addImplicitFVs@ forces the renamer to slurp in some things which aren't
+mentioned explicitly, but which might be needed by the type checker.
+
+\begin{code}
+addImplicitFVs :: GlobalRdrEnv
+              -> Maybe (ModuleName, [RenamedHsDecl])   -- Nothing when compling an expression
+              -> FreeVars                              -- Free in the source
+              -> RnMG (FreeVars, SyntaxMap)            -- Augmented source free vars
+
+addImplicitFVs gbl_env maybe_mod source_fvs
+  =    -- Find out what re-bindable names to use for desugaring
+     rnSyntaxNames gbl_env source_fvs          `thenRn` \ (source_fvs1, sugar_map) ->
+
+       -- Find implicit FVs thade
+    extra_implicits maybe_mod          `thenRn` \ extra_fvs ->
+    
+    let
+       implicit_fvs = ubiquitousNames `plusFV` extra_fvs
+       slurp_fvs    = implicit_fvs `plusFV` source_fvs1
+               -- 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
+    returnRn (slurp_fvs, sugar_map)
+
+  where
+    extra_implicits Nothing            -- Compiling a statement
+      = returnRn (mkFVs [printName, bindIOName, returnIOName, failIOName])
+               -- These are all needed implicitly when compiling a statement
+               -- See TcModule.tc_stmts
+
+    extra_implicits (Just (mod_name, decls))   -- Compiling a module
+      = lookupOrigNames deriv_occs             `thenRn` \ deriving_names ->
+       returnRn (deriving_names `plusFV` implicit_main)
+      where
+       -- Add occurrences for IO or PrimIO
+       implicit_main |  mod_name == mAIN_Name
+                     || mod_name == pREL_MAIN_Name = unitFV ioTyConName
+                     |  otherwise                  = emptyFVs
+
+       deriv_occs = [occ | TyClD (TyData {tcdDerivs = Just deriv_classes}) <- decls,
+                           cls <- deriv_classes,
+                           occ <- lookupWithDefaultUFM derivingOccurrences [] cls ]
+
+-- ubiquitous_names are loaded regardless, because 
+-- they are needed in virtually every program
+ubiquitousNames 
+  = mkFVs [unpackCStringName, unpackCStringFoldrName, 
+          unpackCStringUtf8Name, eqStringName]
+       -- Virtually every program has error messages in it somewhere
+
+  `plusFV`
+    mkFVs [getName unitTyCon, funTyConName, boolTyConName, intTyConName]
+       -- Add occurrences for very frequently used types.
+       --       (e.g. we don't want to be bothered with making funTyCon a
+       --        free var at every function application!)
+\end{code}
+
+\begin{code}
+implicitGates :: Name -> FreeVars      
+-- If we load class Num, add Integer to the gates
+-- This takes account of the fact that Integer might be needed for
+-- defaulting, but we don't want to load Integer (and all its baggage)
+-- if there's no numeric stuff needed.
+-- Similarly for class Fractional and Double
+--
+-- NB: If we load (say) Floating, we'll end up loading Fractional too,
+--     since Fractional is a superclass of Floating
+implicitGates cls | cls `hasKey` numClassKey       = unitFV integerTyConName
+                 | cls `hasKey` fractionalClassKey = unitFV doubleTyConName
+                 | otherwise                       = emptyFVs
+\end{code}
+
+\begin{code}
+rnSyntaxNames :: GlobalRdrEnv -> FreeVars -> RnMG (FreeVars, SyntaxMap)
+-- Look up the re-bindable syntactic sugar names
+-- Any errors arising from these lookups may surprise the
+-- programmer, since they aren't explicitly mentioned, and
+-- the src line will be unhelpful (ToDo)
+
+rnSyntaxNames gbl_env source_fvs
+  = doptRn Opt_NoImplicitPrelude       `thenRn` \ no_prelude -> 
+    if not no_prelude then
+       returnRn (source_fvs, vanillaSyntaxMap)
+    else
+
+       -- There's a -fno-implicit-prelude flag,
+       -- so build the re-mapping function
+    let
+       reqd_syntax_list = filter is_reqd syntaxList
+       is_reqd (n,_)    = n `elemNameSet` source_fvs
+       lookup (n,rn)    = lookupSrcName gbl_env rn     `thenRn` \ rn' ->
+                          returnRn (n,rn')
+    in
+    mapRn lookup reqd_syntax_list      `thenRn` \ rn_syntax_list ->
+    let
+       -- Delete the proxies and add the actuals
+       proxies = map fst rn_syntax_list
+       actuals = map snd rn_syntax_list
+       new_source_fvs = (proxies `delFVs` source_fvs) `plusFV` mkFVs actuals
+
+       syntax_env   = mkNameEnv rn_syntax_list
+       syntax_map n = lookupNameEnv syntax_env n `orElse` n
+    in   
+    returnRn (new_source_fvs, syntax_map)
+\end{code}
+
 
 %*********************************************************
 %*                                                     *
@@ -306,13 +473,13 @@ newLocalsRn rdr_names_w_loc
  =  getNameSupplyRn            `thenRn` \ name_supply ->
     let
        n          = length rdr_names_w_loc
-       (us', us1) = splitUniqSupply (origNS name_supply)
+       (us', us1) = splitUniqSupply (nsUniqs name_supply)
        uniqs      = uniqsFromSupply n us1
        names      = [ mkLocalName uniq (rdrNameOcc rdr_name) loc
                     | ((rdr_name,loc), uniq) <- rdr_names_w_loc `zip` uniqs
                     ]
     in
-    setNameSupplyRn (name_supply {origNS = us'})       `thenRn_`
+    setNameSupplyRn (name_supply {nsUniqs = us'})      `thenRn_`
     returnRn names
 
 
@@ -360,11 +527,11 @@ bindCoreLocalRn rdr_name enclosed_scope
     getLocalNameEnv            `thenRn` \ name_env ->
     getNameSupplyRn            `thenRn` \ name_supply ->
     let
-       (us', us1) = splitUniqSupply (origNS name_supply)
+       (us', us1) = splitUniqSupply (nsUniqs name_supply)
        uniq       = uniqFromSupply us1
        name       = mkLocalName uniq (rdrNameOcc rdr_name) loc
     in
-    setNameSupplyRn (name_supply {origNS = us'})       `thenRn_`
+    setNameSupplyRn (name_supply {nsUniqs = us'})      `thenRn_`
     let
        new_name_env = extendRdrEnv name_env rdr_name name
     in
@@ -377,10 +544,8 @@ bindCoreLocalsRn (b:bs) thing_inside = bindCoreLocalRn b   $ \ name' ->
 
 bindLocalNames names enclosed_scope
   = getLocalNameEnv            `thenRn` \ name_env ->
-    setLocalNameEnv (addListToRdrEnv name_env pairs)
+    setLocalNameEnv (extendLocalRdrEnv name_env names)
                    enclosed_scope
-  where
-    pairs = [(mkRdrUnqual (nameOccName n), n) | n <- names]
 
 bindLocalNamesFV names enclosed_scope
   = bindLocalNames names $
@@ -410,7 +575,7 @@ bindLocalsFVRn doc rdr_names enclosed_scope
 
 -------------------------------------
 extendTyVarEnvFVRn :: [Name] -> RnMS (a, FreeVars) -> RnMS (a, FreeVars)
-       -- This tiresome function is used only in rnDecl on InstDecl
+       -- This tiresome function is used only in rnSourceDecl on InstDecl
 extendTyVarEnvFVRn tyvars enclosed_scope
   = bindLocalNames tyvars enclosed_scope       `thenRn` \ (thing, fvs) -> 
     returnRn (thing, delListFromNameSet fvs tyvars)
@@ -493,13 +658,16 @@ checkDupNames doc_str rdr_names_w_loc
 \begin{code}
 mkGlobalRdrEnv :: ModuleName           -- Imported module (after doing the "as M" name change)
               -> Bool                  -- True <=> want unqualified import
+              -> Bool                  -- True <=> want qualified import
               -> [AvailInfo]           -- What's to be hidden (but only the unqualified 
                                        --      version is hidden)
               -> (Name -> Provenance)
               -> Avails                -- Whats imported and how
+              -> Deprecations
               -> GlobalRdrEnv
 
-mkGlobalRdrEnv this_mod unqual_imp hides mk_provenance avails
+mkGlobalRdrEnv this_mod unqual_imp qual_imp hides 
+              mk_provenance avails deprecs
   = gbl_env2
   where
        -- Make the name environment.  We're talking about a 
@@ -517,13 +685,16 @@ mkGlobalRdrEnv this_mod unqual_imp hides mk_provenance avails
     add_avail env avail = foldl add_name env (availNames avail)
 
     add_name env name
-       | unqual_imp = env2
-       | otherwise  = env1
+       | qual_imp && unqual_imp = env3
+       | unqual_imp             = env2
+       | qual_imp               = env1
+       | otherwise              = env
        where
-         env1 = addOneToGlobalRdrEnv env  (mkRdrQual this_mod occ) (name,prov)
-         env2 = addOneToGlobalRdrEnv env1 (mkRdrUnqual occ)        (name,prov)
+         env1 = addOneToGlobalRdrEnv env  (mkRdrQual this_mod occ) elt
+         env2 = addOneToGlobalRdrEnv env  (mkRdrUnqual occ)        elt
+         env3 = addOneToGlobalRdrEnv env1 (mkRdrUnqual occ)        elt
          occ  = nameOccName name
-         prov = mk_provenance name
+         elt  = GRE name (mk_provenance name) (lookupDeprec deprecs name)
 
     del_avail env avail = foldl delOneFromGlobalRdrEnv env rdr_names
                        where
@@ -537,22 +708,24 @@ mkIfaceGlobalRdrEnv :: [(ModuleName,Avails)] -> GlobalRdrEnv
 mkIfaceGlobalRdrEnv m_avails
   = foldl add emptyRdrEnv m_avails
   where
-    add env (mod,avails) = plusGlobalRdrEnv env (mkGlobalRdrEnv mod True [] (\n -> LocalDef) avails)
+    add env (mod,avails) = plusGlobalRdrEnv env (mkGlobalRdrEnv mod True False [] 
+                                                               (\n -> LocalDef) avails NoDeprecs)
+               -- The NoDeprecs is a bit of a hack I suppose
 \end{code}
 
 \begin{code}
 plusGlobalRdrEnv :: GlobalRdrEnv -> GlobalRdrEnv -> GlobalRdrEnv
 plusGlobalRdrEnv env1 env2 = plusFM_C combine_globals env1 env2
 
-addOneToGlobalRdrEnv :: GlobalRdrEnv -> RdrName -> (Name,Provenance) -> GlobalRdrEnv
+addOneToGlobalRdrEnv :: GlobalRdrEnv -> RdrName -> GlobalRdrElt -> GlobalRdrEnv
 addOneToGlobalRdrEnv env rdr_name name = addToFM_C combine_globals env rdr_name [name]
 
 delOneFromGlobalRdrEnv :: GlobalRdrEnv -> RdrName -> GlobalRdrEnv 
 delOneFromGlobalRdrEnv env rdr_name = delFromFM env rdr_name
 
-combine_globals :: [(Name,Provenance)]         -- Old
-               -> [(Name,Provenance)]  -- New
-               -> [(Name,Provenance)]
+combine_globals :: [GlobalRdrElt]      -- Old
+               -> [GlobalRdrElt]       -- New
+               -> [GlobalRdrElt]
 combine_globals ns_old ns_new  -- ns_new is often short
   = foldr add ns_old ns_new
   where
@@ -562,11 +735,11 @@ combine_globals ns_old ns_new     -- ns_new is often short
     choose n m | n `beats` m = n
               | otherwise   = m
 
-    (n,pn) `beats` (m,pm) = n==m && pn `hasBetterProv` pm
+    (GRE n pn _) `beats` (GRE m pm _) = n==m && pn `hasBetterProv` pm
 
-    is_duplicate :: (Name,Provenance) -> (Name,Provenance) -> Bool
-    is_duplicate (n1,LocalDef) (n2,LocalDef) = False
-    is_duplicate (n1,_)        (n2,_)       = n1 == n2
+    is_duplicate :: GlobalRdrElt -> GlobalRdrElt -> Bool
+    is_duplicate (GRE n1 LocalDef _) (GRE n2 LocalDef _) = False
+    is_duplicate (GRE n1 _        _) (GRE n2 _       _) = n1 == n2
 \end{code}
 
 We treat two bindings of a locally-defined name as a duplicate,
@@ -589,13 +762,15 @@ in error messages.
 
 \begin{code}
 unQualInScope :: GlobalRdrEnv -> Name -> Bool
+-- True if 'f' is in scope, and has only one binding
+-- (i.e. false if A.f and B.f are both in scope as unqualified 'f')
 unQualInScope env
   = (`elemNameSet` unqual_names)
   where
     unqual_names :: NameSet
     unqual_names = foldRdrEnv add emptyNameSet env
-    add rdr_name [(name,_)] unquals | isUnqual rdr_name = addOneToNameSet unquals name
-    add _        _          unquals                    = unquals
+    add rdr_name [GRE name _ _] unquals | isUnqual rdr_name = addOneToNameSet unquals name
+    add _        _              unquals                            = unquals
 \end{code}
 
 
@@ -810,12 +985,7 @@ addNameClashErrRn rdr_name (np1:nps)
   where
     msg1 = ptext  SLIT("either") <+> mk_ref np1
     msgs = [ptext SLIT("    or") <+> mk_ref np | np <- nps]
-    mk_ref (name,prov) = quotes (ppr name) <> comma <+> pprNameProvenance name prov
-
-fixityClashErr (rdr_name, ((_,how_in_scope1), (_, how_in_scope2)))
-  = hang (hsep [ptext SLIT("Conflicting fixities for"), quotes (ppr rdr_name)])
-       4 (vcat [ppr how_in_scope1,
-                ppr how_in_scope2])
+    mk_ref (GRE name prov _) = quotes (ppr name) <> comma <+> pprNameProvenance name prov
 
 shadowedNameWarn shadow
   = hsep [ptext SLIT("This binding for"), 
@@ -839,4 +1009,12 @@ dupNamesErr descriptor ((name,loc) : dup_things)
     addErrRn ((ptext SLIT("Conflicting definitions for") <+> quotes (ppr name))
              $$ 
              (ptext SLIT("in") <+> descriptor))
+
+warnDeprec :: Name -> DeprecTxt -> RnM d ()
+warnDeprec name txt
+  = doptRn Opt_WarnDeprecations                                `thenRn` \ warn_drs ->
+    if not warn_drs then returnRn () else
+    addWarnRn (sep [ text (occNameFlavour (nameOccName name)) <+> 
+                    quotes (ppr name) <+> text "is deprecated:", 
+                    nest 4 (ppr txt) ])
 \end{code}