X-Git-Url: http://git.megacz.com/?a=blobdiff_plain;f=ghc%2Fcompiler%2Frename%2FRnIfaces.lhs;h=2715924203d05650410987166ca6dac472cc42e9;hb=1e5271f1d0fbe2f4391152bfedf9d4cbbae6b2bd;hp=c5018a4c81771e261e1d958417fb817ba6809e95;hpb=9d38678ea60ff32f756390a30c659daa22c98c93;p=ghc-hetmet.git diff --git a/ghc/compiler/rename/RnIfaces.lhs b/ghc/compiler/rename/RnIfaces.lhs index c5018a4..2715924 100644 --- a/ghc/compiler/rename/RnIfaces.lhs +++ b/ghc/compiler/rename/RnIfaces.lhs @@ -22,24 +22,25 @@ module RnIfaces ( import CmdLineOpts ( opt_NoPruneDecls, opt_IgnoreIfacePragmas ) import HsSyn ( HsDecl(..), TyClDecl(..), InstDecl(..), IfaceSig(..), HsType(..), ConDecl(..), IE(..), ConDetails(..), Sig(..), + ForeignDecl(..), ForKind(..), isDynamic, FixitySig(..), RuleDecl(..), - isClassOpSig + isClassOpSig, Deprecation(..) ) import BasicTypes ( Version, NewOrData(..), defaultFixity ) import RdrHsSyn ( RdrNameHsDecl, RdrNameInstDecl, RdrNameTyClDecl, RdrNameRuleDecl, - extractHsTyRdrNames + extractHsTyRdrNames, RdrNameDeprecation ) import RnEnv ( mkImportedGlobalName, newImportedBinder, mkImportedGlobalFromRdrName, - lookupOccRn, + lookupOccRn, lookupImplicitOccRn, pprAvail, availName, availNames, addAvailToNameSet, FreeVars, emptyFVs ) import RnMonad -import RnHsSyn ( RenamedHsDecl ) +import RnHsSyn ( RenamedHsDecl, RenamedDeprecation ) import ParseIface ( parseIface, IfaceStuff(..) ) -import FiniteMap ( FiniteMap, sizeFM, emptyFM, delFromFM, +import FiniteMap ( FiniteMap, sizeFM, emptyFM, delFromFM, listToFM, lookupFM, addToFM, addToFM_C, addListToFM, fmToList, elemFM, foldFM ) @@ -57,7 +58,7 @@ import NameSet import Var ( Id ) import SrcLoc ( mkSrcLoc, SrcLoc ) import PrelMods ( pREL_GHC ) -import PrelInfo ( cCallishTyKeys, thinAirModules ) +import PrelInfo ( cCallishTyKeys ) import Bag import Maybes ( MaybeErr(..), maybeToBool, orElse ) import ListSetOps ( unionLists ) @@ -65,6 +66,7 @@ import Outputable import Unique ( Unique ) import StringBuffer ( StringBuffer, hGetStringBuffer ) import FastString ( mkFastString ) +import ErrUtils ( Message ) import Lex import Outputable @@ -100,33 +102,35 @@ loadInterface doc_str mod_name from let mod_map = iImpModInfo ifaces mod_info = lookupFM mod_map mod_name - in_map = maybeToBool mod_info + below_me = case mod_info of + Nothing -> False + Just (_, _, is_boot, _) -> not is_boot in -- Issue a warning for a redundant {- SOURCE -} import -- It's redundant if the moduld is in the iImpModInfo at all, -- because we arrange to read all the ordinary imports before -- any of the {- SOURCE -} imports - warnCheckRn (not (in_map && case from of {ImportByUserSource -> True; other -> False})) + warnCheckRn (not (below_me && case from of {ImportByUserSource -> True; other -> False})) (warnRedundantSourceImport mod_name) `thenRn_` -- CHECK WHETHER WE HAVE IT ALREADY case mod_info of { - Just (_, _, Just (load_mod, _, _)) + Just (_, _, _, Just (load_mod, _)) -> -- We're read it already so don't re-read it returnRn (load_mod, ifaces) ; mod_map_result -> -- READ THE MODULE IN - findAndReadIface doc_str mod_name from in_map + findAndReadIface doc_str mod_name from below_me `thenRn` \ (hi_boot_read, read_result) -> case read_result of { Nothing -> -- Not found, so add an empty export env to the Ifaces map -- so that we don't look again let mod = mkVanillaModule mod_name - new_mod_map = addToFM mod_map mod_name (0, False, Just (mod, False, [])) + new_mod_map = addToFM mod_map mod_name (0, False, False, Just (mod, [])) new_ifaces = ifaces { iImpModInfo = new_mod_map } in setIfacesRn new_ifaces `thenRn_` @@ -146,11 +150,16 @@ loadInterface doc_str mod_name from let rd_decls = pi_decls iface in - foldlRn (loadDecl mod) (iDecls ifaces) rd_decls `thenRn` \ new_decls -> - foldlRn (loadInstDecl mod) (iInsts ifaces) (pi_insts iface) `thenRn` \ new_insts -> - foldlRn (loadRule mod) (iRules ifaces) (pi_rules iface) `thenRn` \ new_rules -> - foldlRn (loadFixDecl mod_name) (iFixes ifaces) rd_decls `thenRn` \ new_fixities -> - mapRn (loadExport this_mod_nm) (pi_exports iface) `thenRn` \ avails_s -> + foldlRn (loadDecl mod) (iDecls ifaces) rd_decls `thenRn` \ new_decls -> + foldlRn (loadInstDecl mod) (iInsts ifaces) (pi_insts iface) `thenRn` \ new_insts -> + (if opt_IgnoreIfacePragmas + then returnRn emptyBag + else foldlRn (loadRule mod) (iRules ifaces) (pi_rules iface)) `thenRn` \ new_rules -> + (if opt_IgnoreIfacePragmas + then returnRn emptyNameEnv + else foldlRn (loadDeprec mod) (iDeprecs ifaces) (pi_deprecs iface)) `thenRn` \ new_deprecs -> + foldlRn (loadFixDecl mod_name) (iFixes ifaces) rd_decls `thenRn` \ new_fixities -> + mapRn (loadExport this_mod_nm) (pi_exports iface) `thenRn` \ avails_s -> let -- For an explicit user import, add to mod_map info about -- the things the imported module depends on, extracted @@ -161,13 +170,14 @@ loadInterface doc_str mod_name from -- Now add info about this module mod_map2 = addToFM mod_map1 mod_name mod_details - mod_details = (pi_mod iface, pi_orphan iface, Just (mod, hi_boot_read, concat avails_s)) + mod_details = (pi_mod iface, pi_orphan iface, hi_boot_read, Just (mod, concat avails_s)) new_ifaces = ifaces { iImpModInfo = mod_map2, iDecls = new_decls, iFixes = new_fixities, + iInsts = new_insts, iRules = new_rules, - iInsts = new_insts } + iDeprecs = new_deprecs } in setIfacesRn new_ifaces `thenRn_` returnRn (mod, new_ifaces) @@ -179,16 +189,19 @@ addModDeps mod mod_deps new_deps = foldr add mod_deps new_deps where is_lib = isLibModule mod -- Don't record dependencies when importing a library module - add (imp_mod, version, has_orphans, _) deps + add (imp_mod, version, has_orphans, is_boot, _) deps | is_lib && not has_orphans = deps - | otherwise = addToFM_C combine deps imp_mod (version, has_orphans, Nothing) + | otherwise = addToFM_C combine deps imp_mod (version, has_orphans, is_boot, Nothing) -- Record dependencies for modules that are -- either are dependent via a non-library module -- or contain orphan rules or instance decls - -- Don't ditch a module that's already loaded!! - combine old@(_, _, Just _) new = old - combine old@(_, _, Nothing) new = new + -- Don't ditch a module that's already loaded + -- If it isn't loaded, and together the is_boot-ness + combine old@(_, _, _, Just _) new = old + combine old@(_, _, old_is_boot, Nothing) + new@(version, has_orphans, new_is_boot, _) + = (version, has_orphans, old_is_boot && new_is_boot, Nothing) loadExport :: ModuleName -> ExportItem -> RnM d [AvailInfo] loadExport this_mod (mod, entities) @@ -332,6 +345,24 @@ loadRule mod rules decl@(IfaceRuleDecl var body src_loc) = setModuleRn (moduleName mod) $ mkImportedGlobalFromRdrName var `thenRn` \ var_name -> returnRn ((unitNameSet var_name, (mod, RuleD decl)) `consBag` rules) + +-- SUP: TEMPORARY HACK, ignoring module deprecations for now +loadDeprec :: Module -> DeprecationEnv -> RdrNameDeprecation -> RnM d DeprecationEnv +loadDeprec mod deprec_env (Deprecation (IEModuleContents _) txt) + = traceRn (text "module deprecation not yet implemented:" <+> ppr mod <> colon <+> ppr txt) `thenRn_` + returnRn deprec_env +loadDeprec mod deprec_env (Deprecation ie txt) + = setModuleRn (moduleName mod) $ + mapRn mkImportedGlobalFromRdrName (namesFromIE ie) `thenRn` \ names -> + traceRn (text "loaded deprecation(s) for" <+> hcat (punctuate comma (map ppr names)) <> colon <+> ppr txt) `thenRn_` + returnRn (extendNameEnv deprec_env (zip names (repeat txt))) + +namesFromIE :: IE a -> [a] +namesFromIE (IEVar n ) = [n] +namesFromIE (IEThingAbs n ) = [n] +namesFromIE (IEThingAll n ) = [n] +namesFromIE (IEThingWith n ns) = n:ns +namesFromIE (IEModuleContents _ ) = [] \end{code} @@ -365,11 +396,20 @@ checkUpToDate mod_name checkModUsage [] = returnRn True -- Yes! Everything is up to date! -checkModUsage ((mod_name, old_mod_vers, _, whats_imported) : rest) +checkModUsage ((mod_name, old_mod_vers, _, _, Specifically []) : rest) + -- If CurrentModule.hi contains + -- import Foo :: ; + -- then that simply records that Foo lies below CurrentModule in the + -- hierarchy, but CurrentModule doesn't depend in any way on Foo. + -- In this case we don't even want to open Foo's interface. + = traceRn (ptext SLIT("Nothing used from:") <+> ppr mod_name) `thenRn_` + checkModUsage rest -- This one's ok, so check the rest + +checkModUsage ((mod_name, old_mod_vers, _, _, whats_imported) : rest) = loadInterface doc_str mod_name ImportBySystem `thenRn` \ (mod, ifaces) -> let maybe_mod_vers = case lookupFM (iImpModInfo ifaces) mod_name of - Just (version, _, Just (_, _, _)) -> Just version + Just (version, _, _, Just (_, _)) -> Just version other -> Nothing in case maybe_mod_vers of { @@ -419,7 +459,7 @@ checkEntityUsage mod decls ((occ_name,old_vers) : rest) case lookupNameEnv decls name of Nothing -> -- We used it before, but it ain't there now - putDocRn (sep [ptext SLIT("No longer exported:"), ppr name]) + traceRn (sep [ptext SLIT("No longer exported:"), ppr name]) `thenRn_` returnRn False Just (new_vers,_,_,_) -- It's there, but is it up to date? @@ -429,7 +469,7 @@ checkEntityUsage mod decls ((occ_name,old_vers) : rest) | otherwise -- Out of date, so bale out - -> putDocRn (sep [ptext SLIT("Out of date:"), ppr name]) `thenRn_` + -> traceRn (sep [ptext SLIT("Out of date:"), ppr name]) `thenRn_` returnRn False \end{code} @@ -522,7 +562,7 @@ getInterfaceExports mod_name from -- anyway, but this does no harm.) returnRn (mod, []) - Just (_, _, Just (mod, _, avails)) -> returnRn (mod, avails) + Just (_, _, _, Just (mod, avails)) -> returnRn (mod, avails) where doc_str = sep [pprModuleName mod_name, ptext SLIT("is directly imported")] \end{code} @@ -542,7 +582,7 @@ getImportedInstDecls gates getIfacesRn `thenRn` \ ifaces -> let orphan_mods = - [mod | (mod, (_, True, Nothing)) <- fmToList (iImpModInfo ifaces)] + [mod | (mod, (_, True, _, Nothing)) <- fmToList (iImpModInfo ifaces)] in loadOrphanModules orphan_mods `thenRn_` @@ -575,7 +615,9 @@ ppr_brief_inst_decl (mod, InstD (InstDecl inst_ty _ _ _ _)) other -> ppr inst_ty getImportedRules :: RnMG [(Module,RdrNameHsDecl)] -getImportedRules +getImportedRules + | opt_IgnoreIfacePragmas = returnRn [] + | otherwise = getIfacesRn `thenRn` \ ifaces -> let gates = iSlurp ifaces -- Anything at all that's been slurped @@ -609,6 +651,13 @@ lookupFixity name Nothing -> returnRn defaultFixity | otherwise -- Imported + -- For imported names, we have to get their fixities by doing a loadHomeInterface, + -- and consulting the Ifaces that comes back from that, because the interface + -- file for the Name might not have been loaded yet. Why not? Suppose you import module A, + -- which exports a function 'f', which is defined in module B. Then B isn't loaded + -- right away (after all, it's possible that nothing from B will be used). + -- When we come across a use of 'f', we need to know its fixity, and it's then, + -- and only then, that we load B.hi. That is what's happening here. = loadHomeInterface doc name `thenRn` \ ifaces -> case lookupNameEnv (iFixes ifaces) name of Just (FixitySig _ fix _) -> returnRn fix @@ -624,13 +673,13 @@ lookupFixity name %* * %********************************************************* -getImportVersions figures out -what the ``usage information'' for this moudule is; -that is, what it must record in its interface file as the things it uses. -It records: +getImportVersions figures out what the ``usage information'' for this +moudule is; that is, what it must record in its interface file as the +things it uses. It records: + \begin{itemize} -\item anything reachable from its body code -\item any module exported with a @module Foo@. +\item anything reachable from its body code +\item any module exported with a @module Foo@. \end{itemize} % Why the latter? Because if @Foo@ changes then this module's export list @@ -644,81 +693,118 @@ What about this? import B( f ) | f = h 3 g = ... | h = ... \end{verbatim} -Should we record @B.f@ in @A@'s usages? In fact we don't. Certainly, if -anything about @B.f@ changes than anyone who imports @A@ should be recompiled; -they'll get an early exit if they don't use @B.f@. However, even if @B.f@ -doesn't change at all, @B.h@ may do so, and this change may not be reflected -in @f@'s version number. So there are two things going on when compiling module @A@: + +Should we record @B.f@ in @A@'s usages? In fact we don't. Certainly, +if anything about @B.f@ changes than anyone who imports @A@ should be +recompiled; they'll get an early exit if they don't use @B.f@. +However, even if @B.f@ doesn't change at all, @B.h@ may do so, and +this change may not be reflected in @f@'s version number. So there +are two things going on when compiling module @A@: + \begin{enumerate} -\item Are @A.o@ and @A.hi@ correct? Then we can bale out early. -\item Should modules that import @A@ be recompiled? +\item Are @A.o@ and @A.hi@ correct? Then we can bale out early. +\item Should modules that import @A@ be recompiled? \end{enumerate} + For (1) it is slightly harmful to record @B.f@ in @A@'s usages, -because a change in @B.f@'s version will provoke full recompilation of @A@, -producing an identical @A.o@, -and @A.hi@ differing only in its usage-version of @B.f@ -(which isn't used by any importer). - -For (2), because of the tricky @B.h@ question above, -we ensure that @A.hi@ is touched -(even if identical to its previous version) -if A's recompilation was triggered by an imported @.hi@ file date change. +because a change in @B.f@'s version will provoke full recompilation of +@A@, producing an identical @A.o@, and @A.hi@ differing only in its +usage-version of @B.f@ (and this usage-version info isn't used by any +importer). + +For (2), because of the tricky @B.h@ question above, we ensure that +@A.hi@ is touched (even if identical to its previous version) if A's +recompilation was triggered by an imported @.hi@ file date change. Given that, there's no need to record @B.f@ in @A@'s usages. -On the other hand, if @A@ exports @module B@, -then we {\em do} count @module B@ among @A@'s usages, -because we must recompile @A@ to ensure that @A.hi@ changes appropriately. +On the other hand, if @A@ exports @module B@, then we {\em do} count +@module B@ among @A@'s usages, because we must recompile @A@ to ensure +that @A.hi@ changes appropriately. + +HOWEVER, we *do* record the usage + import B :: ; +in A.hi, to record the fact that A does import B. This is used to decide +to look to look for B.hi rather than B.hi-boot when compiling a module that +imports A. This line says that A imports B, but uses nothing in it. +So we'll get an early bale-out when compiling A if B's version changes. \begin{code} getImportVersions :: ModuleName -- Name of this module - -> Maybe [IE any] -- Export list for this module + -> ExportEnv -- Info about exports -> RnMG (VersionInfo Name) -- Version info for these names -getImportVersions this_mod exports +getImportVersions this_mod (ExportEnv export_avails _ export_all_mods) = getIfacesRn `thenRn` \ ifaces -> let mod_map = iImpModInfo ifaces imp_names = iVSlurp ifaces -- mv_map groups together all the things imported from a particular module. - mv_map1, mv_map2 :: FiniteMap ModuleName (WhatsImported Name) - - -- mv_map1 records all the modules that have a "module M" - -- in this module's export list with an "Everything" - mv_map1 = foldr add_mod emptyFM export_mods - - -- mv_map2 adds the version numbers of things exported individually - mv_map2 = foldr add_mv mv_map1 imp_names - - -- Build the result list by adding info for each module, - -- *omitting* (a) library modules - -- (b) source-imported modules - mk_version_info mod_name (version, has_orphans, cts) so_far - | omit cts = so_far -- Don't record usage info for this module - | otherwise = (mod_name, version, has_orphans, whats_imported) : so_far - where - whats_imported = case lookupFM mv_map2 mod_name of - Just wi -> wi - Nothing -> Specifically [] - - omit (Just (mod, boot_import, _)) = isLibModule mod || boot_import - omit Nothing = False + mv_map :: FiniteMap ModuleName [(Name,Version)] + mv_map = foldr add_mv emptyFM imp_names + + -- Build the result list by adding info for each module. + -- For (a) a library module, we don't record it at all unless it contains orphans + -- (We must never lose track of orphans.) + -- + -- (b) a source-imported module, don't record the dependency at all + -- + -- (b) may seem a bit strange. The idea is that the usages in a .hi file records + -- *all* the module's dependencies other than the loop-breakers. We use + -- this info in findAndReadInterface to decide whether to look for a .hi file or + -- a .hi-boot file. + -- + -- This means we won't track version changes, or orphans, from .hi-boot files. + -- The former is potentially rather bad news. It could be fixed by recording + -- whether something is a boot file along with the usage info for it, but + -- I can't be bothered just now. + + mk_version_info mod_name (version, has_orphans, is_boot, contents) so_far + = let + go_for_it exports = (mod_name, version, has_orphans, is_boot, exports) + : so_far + in + case contents of + Nothing -> -- We didn't even open the interface + -- This happens when a module, Foo, that we explicitly imported has + -- 'import Baz' in its interface file, recording that Baz is below + -- Foo in the module dependency hierarchy. We want to propagate this + -- information. The Nothing says that we didn't even open the interface + -- file but we must still propagate the dependeny info. + go_for_it (Specifically []) + + Just (mod, _) -- We did open the interface + | is_lib_module && not has_orphans + -> so_far + + | is_lib_module -- Record the module but not detailed + || mod_name `elem` export_all_mods -- version information for the imports + -> go_for_it Everything + + | otherwise + -> case lookupFM mv_map mod_name of + Just whats_imported -> go_for_it (Specifically whats_imported) + Nothing -> go_for_it (Specifically []) + -- This happens if you have + -- import Foo + -- but don't actually *use* anything from Foo + -- In which case record an empty dependency list + where + is_lib_module = isLibModule mod + in + -- A module shouldn't load its own interface + -- This seems like a convenient place to check + WARN( maybeToBool (lookupFM mod_map this_mod), + ptext SLIT("Wierd:") <+> ppr this_mod <+> ptext SLIT("loads its own interface") ) + returnRn (foldFM mk_version_info [] mod_map) where - export_mods = case exports of - Nothing -> [] - Just es -> [mod | IEModuleContents mod <- es, mod /= this_mod] - add_mv v@(name, version) mv_map - = addToFM_C add_item mv_map mod (Specifically [v]) - where + = addToFM_C add_item mv_map mod [v] + where mod = moduleName (nameModule name) - - add_item Everything _ = Everything - add_item (Specifically xs) _ = Specifically (v:xs) - - add_mod mod mv_map = addToFM mv_map mod Everything + add_item vs _ = (v:vs) \end{code} \begin{code} @@ -770,7 +856,7 @@ getDeclBinders new_name (TyClD (TySynonym tycon _ _ src_loc)) = new_name tycon src_loc `thenRn` \ tycon_name -> returnRn (Just (AvailTC tycon_name [tycon_name])) -getDeclBinders new_name (TyClD (ClassDecl _ cname _ sigs _ _ _ _ _ src_loc)) +getDeclBinders new_name (TyClD (ClassDecl _ cname _ _ sigs _ _ _ _ _ src_loc)) = new_name cname src_loc `thenRn` \ class_name -> -- Record the names for the class ops @@ -787,11 +873,25 @@ getDeclBinders new_name (SigD (IfaceSig var ty prags src_loc)) returnRn (Just (Avail var_name)) getDeclBinders new_name (FixD _) = returnRn Nothing -getDeclBinders new_name (ForD _) = returnRn Nothing + + -- foreign declarations +getDeclBinders new_name (ForD (ForeignDecl nm kind _ dyn _ loc)) + | binds_haskell_name kind dyn + = new_name nm loc `thenRn` \ name -> + returnRn (Just (Avail name)) + + | otherwise -- a foreign export + = lookupImplicitOccRn nm `thenRn_` + returnRn Nothing + getDeclBinders new_name (DefD _) = returnRn Nothing getDeclBinders new_name (InstD _) = returnRn Nothing getDeclBinders new_name (RuleD _) = returnRn Nothing +binds_haskell_name (FoImport _) _ = True +binds_haskell_name FoLabel _ = True +binds_haskell_name FoExport ext_nm = isDynamic ext_nm + ---------------- getConFieldNames new_name (ConDecl con _ _ (RecCon fielddecls) src_loc : rest) = mapRn (\n -> new_name n src_loc) (con:fields) `thenRn` \ cfs -> @@ -812,7 +912,7 @@ getConFieldNames new_name (ConDecl con _ _ condecl src_loc : rest) getConFieldNames new_name [] = returnRn [] -getClassOpNames new_name (ClassOpSig op _ _ src_loc) = new_name op src_loc +getClassOpNames new_name (ClassOpSig op _ _ _ src_loc) = new_name op src_loc \end{code} @getDeclSysBinders@ gets the implicit binders introduced by a decl. @@ -825,7 +925,7 @@ and the dict fun of an instance decl, because both of these have bindings of their own elsewhere. \begin{code} -getDeclSysBinders new_name (TyClD (ClassDecl _ cname _ sigs _ _ tname dname snames src_loc)) +getDeclSysBinders new_name (TyClD (ClassDecl _ cname _ _ sigs _ _ tname dname snames src_loc)) = new_name dname src_loc `thenRn` \ datacon_name -> new_name tname src_loc `thenRn` \ tycon_name -> sequenceRn [new_name n src_loc | n <- snames] `thenRn` \ scsel_names -> @@ -907,15 +1007,17 @@ readIface the_mod file_path context = [], glasgow_exts = 1#, loc = mkSrcLoc (mkFastString file_path) 1 } of - PFailed err -> failWithRn Nothing err POk _ (PIface mod_nm iface) -> warnCheckRn (mod_nm == moduleName the_mod) - (hsep [ ptext SLIT("Something is amiss; requested module name") - , pprModule the_mod - , ptext SLIT("differs from name found in the interface file ") - , pprModuleName mod_nm - ]) - `thenRn_` returnRn (Just (the_mod, iface)) + (hiModuleNameMismatchWarn the_mod mod_nm) `thenRn_` + returnRn (Just (the_mod, iface)) + + PFailed err -> failWithRn Nothing err + other -> failWithRn Nothing (ptext SLIT("Unrecognisable interface file")) + -- This last case can happen if the interface file is (say) empty + -- in which case the parser thinks it looks like an IdInfo or + -- something like that. Just an artefact of the fact that the + -- parser is used for several purposes at once. Left err | isDoesNotExistError err -> returnRn Nothing @@ -960,4 +1062,13 @@ importDeclWarn name warnRedundantSourceImport mod_name = ptext SLIT("Unnecessary {- SOURCE -} in the import of module") <+> quotes (pprModuleName mod_name) + +hiModuleNameMismatchWarn :: Module -> ModuleName -> Message +hiModuleNameMismatchWarn requested_mod mod_nm = + hsep [ ptext SLIT("Something is amiss; requested module name") + , pprModule requested_mod + , ptext SLIT("differs from name found in the interface file ") + , pprModuleName mod_nm + ] + \end{code}