[project @ 1996-06-11 13:18:54 by partain]
[ghc-hetmet.git] / ghc / compiler / rename / RnIfaces.lhs
index 76fe13c..3db7db8 100644 (file)
@@ -7,17 +7,14 @@
 #include "HsVersions.h"
 
 module RnIfaces (
---     findHiFiles,
        cachedIface,
        cachedDecl,
-       readIface,
        rnIfaces,
        IfaceCache(..)
     ) where
 
-import Ubiq
+IMP_Ubiq()
 
-import LibDirectory
 import PreludeGlaST    ( thenPrimIO, seqPrimIO, readVar, writeVar, MutableVar(..) )
 
 import HsSyn
@@ -38,10 +35,11 @@ import Bag          ( emptyBag, unitBag, consBag, snocBag,
 import ErrUtils                ( Error(..), Warning(..) )
 import FiniteMap       ( emptyFM, lookupFM, addToFM, addToFM_C, plusFM, eltsFM,
                          fmToList, delListFromFM, sizeFM, foldFM, unitFM,
-                         plusFM_C, keysFM{-ToDo:rm-}
+                         plusFM_C, addListToFM, keysFM{-ToDo:rm-}
                        )
 import Maybes          ( maybeToBool )
-import Name            ( moduleNamePair, origName, RdrName(..) )
+import Name            ( origName, moduleOf, nameOf, qualToOrigName, OrigName(..),
+                         isLexCon, RdrName(..), Name{-instance NamedThing-} )
 import PprStyle                -- ToDo:rm
 import Outputable      -- ToDo:rm
 import PrelInfo                ( builtinNameInfo )
@@ -67,80 +65,6 @@ type IfaceCache
 
 *********************************************************
 *                                                      *
-\subsection{Looking for interface files}
-*                                                      *
-*********************************************************
-
-Return a mapping from module-name to
-absolute-filename-for-that-interface.
-\begin{code}
-{- OLD:
-findHiFiles :: [FilePath] -> [FilePath] -> IO (FiniteMap Module FilePath)
-
-findHiFiles dirs sysdirs
-  = --hPutStr stderr "  findHiFiles "  >>
-    do_dirs emptyFM (dirs ++ sysdirs)  >>= \ result ->
-    --hPutStr stderr " done\n"         >>
-    return result
-  where
-    do_dirs env [] = return env
-    do_dirs env (dir:dirs)
-      = do_dir  env     dir    >>= \ new_env ->
-       do_dirs new_env dirs
-    -------
-    do_dir env dir
-      = --hPutStr stderr "D" >>
-       getDirectoryContents dir    >>= \ entries ->
-       do_entries env entries
-      where
-       do_entries env [] = return env
-       do_entries env (e:es)
-         = do_entry   env     e    >>= \ new_env ->
-           do_entries new_env es
-       -------
-       do_entry env e
-         = case (acceptable_hi (reverse e)) of
-             Nothing  -> --trace ("Deemed uncool:"++e) $
-                         --hPutStr stderr "." >>
-                         return env
-             Just mod ->
-               let
-                     pmod = _PK_ mod
-               in
-               case (lookupFM env pmod) of
-                 Nothing -> --trace ("Adding "++mod++" -> "++e) $
-                            --hPutStr stderr "!" >>
-                            return (addToFM env pmod (dir ++ '/':e))
-                            -- ToDo: use DIR_SEP, not /
-
-                 Just xx -> ( if de_dot xx /= e then trace ("Already mapped!! "++mod++" -> "++xx++"; ignoring:"++e) else id) $
-                            --hPutStr stderr "." >>
-                            return env
-    -------
-    acceptable_hi rev_e -- looking at pathname *backwards*
-      = case (startsWith (reverse opt_HiSuffix) rev_e) of
-         Nothing -> Nothing
-         Just xs -> plausible_modname xs{-reversed-}
-
-    -------
-    de_dot ('.' : '/' : xs) = xs
-    de_dot xs              = xs
-
-    -------
-    plausible_modname rev_e
-      = let
-           cand = reverse (takeWhile is_modname_char rev_e)
-       in
-       if null cand || not (isUpper (head cand))
-       then Nothing
-       else Just cand
-      where
-       is_modname_char c = isAlphanum c || c == '_'
--}
-\end{code}
-
-*********************************************************
-*                                                      *
 \subsection{Reading interface files}
 *                                                      *
 *********************************************************
@@ -173,22 +97,22 @@ cachedIface :: Bool                -- True  => want merged interface for original name
            -> Module
            -> IO (MaybeErr ParsedIface Error)
 
-cachedIface want_orig_iface iface_cache mod
+cachedIface want_orig_iface iface_cache modname
   = readVar iface_cache `thenPrimIO` \ (iface_fm, orig_fm, file_fm) ->
 
-    case (lookupFM iface_fm mod) of
+    case (lookupFM iface_fm modname) of
       Just iface -> return (want_iface iface orig_fm)
       Nothing    ->
-       case (lookupFM file_fm mod) of
-         Nothing   -> return (Failed (noIfaceErr mod))
+       case (lookupFM file_fm modname) of
+         Nothing   -> return (Failed (noIfaceErr modname))
          Just file ->
-           readIface file mod >>= \ read_iface ->
+           readIface file modname >>= \ read_iface ->
            case read_iface of
              Failed err      -> -- pprTrace "module-file map:\n" (ppAboves [ppCat [ppPStr m, ppStr f] | (m,f) <- fmToList file_fm]) $
                                 return (Failed err)
              Succeeded iface ->
                let
-                   iface_fm' = addToFM iface_fm mod iface
+                   iface_fm' = addToFM iface_fm modname iface
                    orig_fm'  = addToFM_C mergeIfaces orig_fm (iface_mod iface) iface
                in
                writeVar iface_cache (iface_fm', orig_fm', file_fm) `seqPrimIO`
@@ -196,8 +120,8 @@ cachedIface want_orig_iface iface_cache mod
   where
     want_iface iface orig_fm 
       | want_orig_iface
-      = case lookupFM orig_fm mod of
-         Nothing         -> Failed (noOrigIfaceErr mod)
+      = case lookupFM orig_fm modname of
+         Nothing         -> Failed (noOrigIfaceErr modname)
           Just orig_iface -> Succeeded orig_iface
       | otherwise
       = Succeeded iface
@@ -239,19 +163,19 @@ mergeIfaces (ParsedIface mod1 (_, files1) _ _ _ _ _ _ fixes1 tdefs1 vdefs1 idefs
 ----------
 cachedDecl :: IfaceCache
           -> Bool      -- True <=> tycon or class name
-          -> RdrName
+          -> OrigName
           -> IO (MaybeErr RdrIfaceDecl Error)
 
-cachedDecl iface_cache class_or_tycon orig 
-  = cachedIface True iface_cache mod   >>= \ maybe_iface ->
+cachedDecl iface_cache class_or_tycon name@(OrigName mod str)
+  = -- pprTrace "cachedDecl:" (ppr PprDebug name) $
+    cachedIface True iface_cache mod   >>= \ maybe_iface ->
     case maybe_iface of
-      Failed err -> return (Failed err)
+      Failed err -> --pprTrace "cachedDecl:fail:" (ppr PprDebug orig) $
+                   return (Failed err)
       Succeeded (ParsedIface _ _ _ _ _ _ exps _ _ tdefs vdefs _ _) -> 
        case (lookupFM (if class_or_tycon then tdefs else vdefs) str) of
          Just decl -> return (Succeeded decl)
          Nothing   -> return (Failed (noDeclInIfaceErr mod str))
-  where
-    (mod, str) = moduleNamePair orig
 
 ----------
 cachedDeclByType :: IfaceCache
@@ -262,13 +186,13 @@ cachedDeclByType iface_cache rn
     -- the idea is: check that, e.g., if we're given an
     -- RnClass, then we really get back a ClassDecl from
     -- the cache (not an RnData, or something silly)
-  = cachedDecl iface_cache (isRnTyConOrClass rn) (origName rn)  >>= \ maybe_decl ->
+  = cachedDecl iface_cache (isRnTyConOrClass rn) (origName "cachedDeclByType" rn)  >>= \ maybe_decl ->
     let
        return_maybe_decl = return maybe_decl
        return_failed msg = return (Failed msg)
     in
     case maybe_decl of
-      Failed _ -> return_maybe_decl
+      Failed io_msg -> return_failed (ifaceIoErr io_msg rn)
       Succeeded if_decl ->
        case rn of
          WiredInId _       -> return_failed (ifaceLookupWiredErr "value" rn)
@@ -310,21 +234,20 @@ cachedDeclByType iface_cache rn
 \end{code}
 
 \begin{code}
-readIface :: FilePath -> Module
-             -> IO (MaybeErr ParsedIface Error)
+readIface :: FilePath -> Module -> IO (MaybeErr ParsedIface Error)
 
-readIface file mod
-  = --hPutStr stderr ("  reading "++file)      >>
+readIface file modname
+  = hPutStr stderr ("  reading "++file)        >>
     readFile file              `thenPrimIO` \ read_result ->
     case read_result of
       Left  err      -> return (Failed (cannaeReadErr file err))
-      Right contents -> --hPutStr stderr " parsing"   >>
+      Right contents -> hPutStr stderr ".."   >>
                        let parsed = parseIface contents in
-                       --hPutStr stderr " done\n"    >>
+                       hPutStr stderr "..\n" >>
                        return (
                        case parsed of
                          Failed _    -> parsed
-                         Succeeded p -> Succeeded (init_merge mod p)
+                         Succeeded p -> Succeeded (init_merge modname p)
                        )
   where
     init_merge this (ParsedIface mod _ v sv us vs exps insts fixes tdefs vdefs idefs prags)
@@ -358,7 +281,6 @@ rnIfaces iface_cache imp_mods us
         todo
   = {-
     pprTrace "rnIfaces:going after:" (ppCat (map (ppr PprDebug) todo)) $
-
     pprTrace "rnIfaces:qual:"      (ppCat [ppBesides[ppPStr m,ppChar '.',ppPStr n] | (n,m) <- keysFM qual]) $
     pprTrace "rnIfaces:unqual:"    (ppCat (map ppPStr (keysFM unqual))) $
     pprTrace "rnIfaces:tc_qual:"   (ppCat [ppBesides[ppPStr m,ppChar '.',ppPStr n] | (n,m) <- keysFM tc_qual]) $
@@ -397,7 +319,10 @@ rnIfaces iface_cache imp_mods us
            if_errs_warns)
   where
     decls_and_insts todo def_env occ_env to_return us
-      =        do_decls todo                    -- initial batch of names to process
+      =        let
+           (us1,us2) = splitUniqSupply us
+       in
+       do_decls todo                    -- initial batch of names to process
                 (def_env, occ_env, us1) -- init stuff down
                 to_return               -- acc results
           >>= \ (decls_return,
@@ -408,9 +333,8 @@ rnIfaces iface_cache imp_mods us
 
        do_insts decls_def_env decls_occ_env emptyRnEnv emptyFM
                 (add_errs errs decls_return) us2
-      where
-       (us1,us2) = splitUniqSupply us
 
+    --------
     do_insts def_env occ_env prev_env done_insts to_return us
       | size_tc_env occ_env == size_tc_env prev_env
       = return (to_return, occ_env)
@@ -458,10 +382,10 @@ rnIfaces iface_cache imp_mods us
                     do_decls ns down to_return
 
          Nothing
-          | fst (moduleNamePair n) == modname ->
+          | moduleOf (origName "do_decls" n) == modname ->
                     -- avoid looking in interface for the module being compiled
-                    -- pprTrace "do_decls:this module error:" (ppr PprDebug n) $
-                    do_decls ns down (add_err (thisModImplicitErr modname n) to_return)
+                    --pprTrace "do_decls:this module error:" (ppr PprDebug n) $
+                    do_decls ns down (add_warn (thisModImplicitWarn modname n) to_return)
 
           | otherwise ->
                     -- OK, see what the cache has for us...
@@ -469,7 +393,7 @@ rnIfaces iface_cache imp_mods us
             cachedDeclByType iface_cache n >>= \ maybe_ans ->
             case maybe_ans of
               Failed err -> -- add the error, but keep going:
-                            -- pprTrace "do_decls:cache error:" (ppr PprDebug n) $
+                            --pprTrace "do_decls:cache error:" (ppr PprDebug n) $
                             do_decls ns down (add_err err to_return)
 
               Succeeded iface_decl -> -- something needing renaming!
@@ -514,10 +438,9 @@ type Go_Down   = (RnEnv,   -- stuff we already have defns for;
                 )
 
 lookup_defd (def_env, _, _) n
-  | isRnTyConOrClass n 
-  = lookupTcRnEnv def_env (origName n)
-  | otherwise 
-  = lookupRnEnv def_env (origName n)
+  = (if isRnTyConOrClass n then lookupTcRnEnv else lookupRnEnv) def_env
+       (case (origName "lookup_defd" n) of { OrigName m s -> Qual m s })
+       -- this is hack because we are reusing the RnEnv technology
 
 defenv    (def_env, _, _) = def_env
 occenv    (_, occ_env, _) = occ_env
@@ -527,10 +450,14 @@ new_uniqsupply us (def_env, occ_env, _) = (def_env, occ_env, us)
 
 add_occs (val_defds, tc_defds) (val_imps, tc_imps) (def_env, occ_env, us)
   = case (extendGlobalRnEnv def_env val_defds tc_defds) of { (new_def_env, def_dups) ->
-    ASSERT(isEmptyBag def_dups)
+    (if isEmptyBag def_dups then \x->x else pprTrace "add_occs:" (ppCat [ppr PprDebug n | (n,_,_) <- bagToList def_dups])) $
+--  ASSERT(isEmptyBag def_dups)
     let
-       val_occs = val_defds ++ fmToList val_imps
-       tc_occs  = tc_defds  ++ fmToList tc_imps
+       de_orig imps = [ (Qual m n, v) | (OrigName m n, v) <- fmToList imps ]
+       -- again, this hackery because we are reusing the RnEnv technology
+
+       val_occs = val_defds ++ de_orig val_imps
+       tc_occs  = tc_defds  ++ de_orig tc_imps
     in
     case (extendGlobalRnEnv occ_env val_occs tc_occs)   of { (new_occ_env, occ_dups) ->
 
@@ -558,10 +485,11 @@ add_insts is ((tydecls, classdecls, instdecls, sigs), implicit, msgs)
   = ((tydecls, classdecls, is ++ instdecls, sigs), implicit, msgs)
 
 add_implicits (val_imps, tc_imps) (decls, (val_fm, tc_fm), msgs)
-  = (decls, (val_fm `plusFM` val_imps, tc_fm `plusFM`  tc_imps), msgs)
+  = (decls, (val_fm `plusFM` val_imps, tc_fm `plusFM` tc_imps), msgs)
 
 add_err  err (decls,implicit,(errs,warns)) = (decls,implicit,(errs `snocBag`   err,warns))
 add_errs ers (decls,implicit,(errs,warns)) = (decls,implicit,(errs `unionBags` ers,warns))
+add_warn wrn (decls,implicit,(errs,warns)) = (decls,implicit,(errs, warns `snocBag` wrn))
 add_warns ws (decls,implicit,(errs,warns)) = (decls,implicit,(errs, warns `unionBags` ws))
 \end{code}
 
@@ -641,8 +569,8 @@ rnIfaceDecl (ValSig f src_loc ty)
 sub :: ImplicitEnv -> ([(RdrName,RnName)], [(RdrName,RnName)]) -> ImplicitEnv
 
 sub (val_ment, tc_ment) (val_defds, tc_defds)
-  = (delListFromFM val_ment (map fst val_defds),
-     delListFromFM tc_ment  (map fst tc_defds))
+  = (delListFromFM val_ment (map (qualToOrigName . fst) val_defds),
+     delListFromFM tc_ment  (map (qualToOrigName . fst) tc_defds))
 \end{code}
 
 % ------------------------------
@@ -658,6 +586,7 @@ cacheInstModules iface_cache imp_mods
        (imp_imods, _)  = removeDups cmpPString (bagToList (unionManyBags (map get_ims imp_ifaces)))
         get_ims (ParsedIface _ _ _ _ _ _ _ ims _ _ _ _ _) = ims
     in
+    --pprTrace "cacheInstModules:" (ppCat (map ppPStr imp_imods)) $
     accumulate (map (cachedIface False iface_cache) imp_imods) >>= \ err_or_ifaces ->
 
     -- Sanity Check:
@@ -682,7 +611,7 @@ cacheInstModules iface_cache imp_mods
 @rnIfaceInstStuff@: Deal with instance declarations from interface files.
 
 \begin{code}
-type InstanceEnv = FiniteMap (RdrName, RdrName) Int
+type InstanceEnv = FiniteMap (OrigName, OrigName) Int
 
 rnIfaceInstStuff
        :: IfaceCache           -- all about ifaces we've read
@@ -722,8 +651,8 @@ rnIfaceInstStuff iface_cache modname us occ_env done_inst_env to_return
 
     case (initRn False{-iface-} modname occ_env us (
            setExtraRn emptyUFM{-no fixities-}  $
-           mapRn rnIfaceInst interesting_insts `thenRn` \ insts ->
-           getImplicitUpRn                     `thenRn` \ implicits ->
+           mapRn (rnIfaceInst modname) interesting_insts `thenRn` \ insts ->
+           getImplicitUpRn                               `thenRn` \ implicits ->
            returnRn (insts, implicits))) of {
       ((if_insts, if_implicits), if_errs, if_warns) ->
 
@@ -738,51 +667,58 @@ rnIfaceInstStuff iface_cache modname us occ_env done_inst_env to_return
   where
     get_insts (ParsedIface _ _ _ _ _ _ _ _ _ _ _ insts _) = insts
 
+    tycon_class clas tycon = (qualToOrigName clas, qualToOrigName tycon)
+
     add_done_inst (InstSig clas tycon _ _) inst_env
-      = addToFM_C (+) inst_env (tycon,clas) 1
+      = addToFM_C (+) inst_env (tycon_class clas tycon) 1
 
     is_done_inst (InstSig clas tycon _ _)
-      = maybeToBool (lookupFM done_inst_env (tycon,clas))
+      = maybeToBool (lookupFM done_inst_env (tycon_class clas tycon))
 
     add_imp_occs (val_imps, tc_imps) occ_env
-      = case extendGlobalRnEnv occ_env (fmToList val_imps) (fmToList tc_imps) of
+      = case (extendGlobalRnEnv occ_env (de_orig val_imps) (de_orig tc_imps)) of
          (ext_occ_env, occ_dups) -> ASSERT(isEmptyBag occ_dups)
                                     ext_occ_env
+      where
+       de_orig imps = [ (Qual m n, v) | (OrigName m n, v) <- fmToList imps ]
+       -- again, this hackery because we are reusing the RnEnv technology
 
     want_inst i@(InstSig clas tycon _ _)
       = -- it's a "good instance" (one to hang onto) if we have a
        -- chance of referring to *both* the class and tycon later on ...
-
+       --pprTrace "want_inst:" (ppCat [ppr PprDebug clas, ppr PprDebug tycon, ppr PprDebug (mentionable tycon), ppr PprDebug (mentionable clas), ppr PprDebug(is_done_inst i)]) $
        mentionable tycon && mentionable clas && not (is_done_inst i)
       where
        mentionable nm
          = case lookupTcRnEnv occ_env nm of
              Just  _ -> True
              Nothing -> -- maybe it's builtin
-               case nm of
-                 Qual _ _ -> False
-                 Unqual n ->
-                   case (lookupFM b_tc_names n) of
-                     Just  _ -> True
-                     Nothing -> maybeToBool (lookupFM b_keys n)
+               let orig = qualToOrigName nm in
+               case (lookupFM b_tc_names orig) of
+                 Just  _ -> True
+                 Nothing -> maybeToBool (lookupFM b_keys orig)
 
     (b_tc_names, b_keys) -- pretty UGLY ...
       = case builtinNameInfo of ((_,builtin_tcs),b_keys,_) -> (builtin_tcs,b_keys)
-
+{-
     ppr_insts insts
       = ppAboves (map ppr_inst insts)
       where
        ppr_inst (InstSig c t _ inst_decl)
          = ppCat [ppr PprDebug c, ppr PprDebug t, ppr PprDebug inst_decl]
+-}
 \end{code}
 
 \begin{code}
-rnIfaceInst :: RdrIfaceInst -> RnM_Fixes _RealWorld RenamedInstDecl
+rnIfaceInst :: Module -> RdrIfaceInst -> RnM_Fixes _RealWorld RenamedInstDecl
 
-rnIfaceInst (InstSig _ _ _ inst_decl) = rnInstDecl inst_decl
+rnIfaceInst mod (InstSig _ _ _ inst_decl) = rnInstDecl (inst_decl mod)
 \end{code}
 
 \begin{code}
+type BigMaps = (FiniteMap Module Version, -- module-version map
+               FiniteMap (FAST_STRING,Module) Version) -- ordinary version map
+
 finalIfaceInfo ::
           IfaceCache                   -- iface cache
        -> Module                       -- this module's name
@@ -800,47 +736,76 @@ finalIfaceInfo iface_cache modname if_final_env@((qual, unqual, tc_qual, tc_unqu
 --  pprTrace "usageIf:unqual:"    (ppCat (map ppPStr (keysFM unqual))) $
 --  pprTrace "usageIf:tc_qual:"   (ppCat [ppBesides[ppPStr m,ppChar '.',ppPStr n] | (n,m) <- keysFM tc_qual]) $
 --  pprTrace "usageIf:tc_unqual:" (ppCat (map ppPStr (keysFM tc_unqual))) $
+    readVar iface_cache        `thenPrimIO` \ (_, orig_iface_fm, _) ->
     let
+       all_ifaces = eltsFM orig_iface_fm
+       -- all the interfaces we have looked at
+
+       big_maps
+         -- combine all the version maps we have seen into maps to
+         -- (a) lookup a module-version number, lookup an entity's
+         -- individual version number
+         = foldr mk_map (emptyFM,emptyFM) all_ifaces
+
        val_stuff@(val_usages, val_versions)
-         = foldFM process_item (emptyFM, emptyFM){-init-} qual
+         = foldFM (process_item big_maps) (emptyFM, emptyFM){-init-} qual
 
        (all_usages, all_versions)
-         = foldFM process_item val_stuff{-keep going-} tc_qual
+         = foldFM (process_item big_maps) val_stuff{-keep going-} tc_qual
     in
     return (all_usages, all_versions, [])
   where
-    process_item :: (FAST_STRING,Module) -> RnName -- RnEnv (QualNames) components
+    mk_map (ParsedIface m _ mv _ _ vers _ _ _ _ _ _ _) (mv_map, ver_map)
+      = (addToFM     mv_map  m mv, -- add this module
+        addListToFM ver_map [ ((n,m), v) | (n,v) <- fmToList vers ])
+
+    -----------------------
+    process_item :: BigMaps
+                -> (FAST_STRING,Module) -> RnName -- RnEnv (QualNames) components
                 -> (UsagesMap, VersionsMap)       -- input
                 -> (UsagesMap, VersionsMap)       -- output
 
-    process_item (n,m) rn as_before@(usages, versions)
+    process_item (big_mv_map, big_version_map) key@(n,m) rn as_before@(usages, versions)
       | irrelevant rn
       = as_before
       | m == modname -- this module => add to "versions"
       =        (usages, addToFM versions n 1{-stub-})
       | otherwise  -- from another module => add to "usages"
-      = (add_to_usages usages m n 1{-stub-}, versions)
+      = (add_to_usages usages key, versions)
+      where
+       add_to_usages usages key@(n,m)
+         = let
+               mod_v = case (lookupFM big_mv_map m) of
+                         Nothing -> pprTrace "big_mv_map:miss? " (ppPStr m) $
+                                    1
+                         Just nv -> nv
+               key_v = case (lookupFM big_version_map key) of
+                         Nothing -> pprTrace "big_version_map:miss? " (ppCat [ppPStr n, ppPStr m]) $
+                                    1
+                         Just nv -> nv
+           in
+           addToFM usages m (
+               case (lookupFM usages m) of
+                 Nothing -> -- nothing for this module yet...
+                   (mod_v, unitFM n key_v)
+
+                 Just (mversion, mstuff) -> -- the "new" stuff will shadow the old
+                   ASSERT(mversion == mod_v)
+                   (mversion, addToFM mstuff n key_v)
+           )
 
     irrelevant (RnConstr  _ _) = True  -- We don't report these in their
     irrelevant (RnField   _ _) = True  -- own right in usages/etc.
     irrelevant (RnClassOp _ _) = True
+    irrelevant (RnImplicit  n) = isLexCon (nameOf (origName "irrelevant" n)) -- really a RnConstr
     irrelevant _              = False
 
-    add_to_usages usages m n version
-      = addToFM usages m (
-           case (lookupFM usages m) of
-             Nothing -> -- nothing for this module yet...
-               (1{-stub-}, unitFM n version)
-
-             Just (mversion, mstuff) -> -- the "new" stuff will shadow the old
-               (mversion, addToFM mstuff n version)
-       )
 \end{code}
 
 
 \begin{code}
-thisModImplicitErr mod n sty
-  = ppCat [ppPStr SLIT("Implicit import of"), ppr sty n, ppPStr SLIT("when compiling"), ppPStr mod]
+thisModImplicitWarn mod n sty
+  = ppBesides [ppPStr SLIT("An interface has an implicit need of "), ppr sty n, ppPStr SLIT("; assuming this module will provide it.")]
 
 noIfaceErr mod sty
   = ppCat [ppPStr SLIT("Could not find interface for:"), ppPStr mod]
@@ -860,4 +825,7 @@ ifaceLookupWiredErr msg n sty
 
 badIfaceLookupErr msg name decl sty
   = ppBesides [ppPStr SLIT("Expected a "), ppStr msg, ppPStr SLIT(" declaration, but got this: ???")]
+
+ifaceIoErr io_msg rn sty
+  = ppBesides [io_msg sty, ppStr "; looking for: ", ppr sty rn]
 \end{code}