X-Git-Url: http://git.megacz.com/?a=blobdiff_plain;f=ghc%2Fcompiler%2Frename%2FRnIfaces.lhs;h=e62b780ea49a0c2ce8321a45b073a639fc6d0c65;hb=8fbe28ca67a6c07575aee35bfade43e967ad792e;hp=ad02e6dce3a53603a7431ad5f728811cd5b80773;hpb=2b4d4dcc7f2bb1e90a7c06f111c8460b6348920a;p=ghc-hetmet.git diff --git a/ghc/compiler/rename/RnIfaces.lhs b/ghc/compiler/rename/RnIfaces.lhs index ad02e6d..e62b780 100644 --- a/ghc/compiler/rename/RnIfaces.lhs +++ b/ghc/compiler/rename/RnIfaces.lhs @@ -4,728 +4,516 @@ \section[RnIfaces]{Cacheing and Renaming of Interfaces} \begin{code} -module RnIfaces ( - findAndReadIface, +module RnIfaces + ( + getInterfaceExports, + recordLocalSlurps, + mkImportInfo, - getInterfaceExports, getDeferredDecls, - getImportedInstDecls, getImportedRules, - lookupFixityRn, loadHomeInterface, - importDecl, ImportDeclResult(..), recordLocalSlurps, loadBuiltinRules, - mkImportExportInfo, getSlurped, + slurpImpDecls, closeDecls, - checkModUsage, outOfDate, upToDate, - - getDeclBinders, getDeclSysBinders, - removeContext -- removeContext probably belongs somewhere else - ) where + RecompileRequired, outOfDate, upToDate, recompileRequired + ) +where #include "HsVersions.h" -import CmdLineOpts ( opt_NoPruneDecls, opt_NoPruneTyDecls, opt_IgnoreIfacePragmas ) -import HsSyn ( HsDecl(..), TyClDecl(..), InstDecl(..), IfaceSig(..), - HsType(..), ConDecl(..), IE(..), ConDetails(..), Sig(..), - ForeignDecl(..), ForKind(..), isDynamicExtName, - FixitySig(..), RuleDecl(..), - isClassOpSig, DeprecDecl(..) +import CmdLineOpts ( opt_IgnoreIfacePragmas, opt_NoPruneDecls ) +import HscTypes +import HsSyn ( HsDecl(..), Sig(..), TyClDecl(..), ConDecl(..), ConDetails(..), + InstDecl(..), HsType(..), hsTyVarNames, getBangType + ) +import HsImpExp ( ImportDecl(..) ) +import RdrHsSyn ( RdrNameTyClDecl, RdrNameInstDecl, RdrNameRuleDecl ) +import RnHsSyn ( RenamedHsDecl, RenamedTyClDecl, + extractHsTyNames, extractHsCtxtTyNames, + tyClDeclFVs, ruleDeclFVs, instDeclFVs ) -import HsImpExp ( ieNames ) -import CoreSyn ( CoreRule ) -import BasicTypes ( Version, NewOrData(..) ) -import RdrHsSyn ( RdrNameHsDecl, RdrNameInstDecl, RdrNameRuleDecl, - RdrNameDeprecation, RdrNameIE, - extractHsTyRdrNames +import RnHiFiles ( tryLoadInterface, loadHomeInterface, loadInterface, + loadOrphanModules ) +import RnSource ( rnTyClDecl, rnInstDecl, rnIfaceRuleDecl ) import RnEnv import RnMonad -import ParseIface ( parseIface, IfaceStuff(..) ) - +import Id ( idType ) +import Type ( namesOfType ) +import TyCon ( isSynTyCon, getSynTyConDefn ) import Name ( Name {-instance NamedThing-}, nameOccName, - nameModule, isLocallyDefined, - isWiredInName, NamedThing(..), - elemNameEnv, extendNameEnv + nameModule, isLocalName, nameUnique, + NamedThing(..) ) -import Module ( Module, mkVanillaModule, pprModuleName, - moduleName, isLocalModule, +import Name ( elemNameEnv, delFromNameEnv ) +import Module ( Module, ModuleEnv, + moduleName, isHomeModule, ModuleName, WhereFrom(..), + emptyModuleEnv, + extendModuleEnv_C, foldModuleEnv, lookupModuleEnv, + elemModuleSet, extendModuleSet ) -import RdrName ( RdrName, rdrNameOcc ) import NameSet -import SrcLoc ( mkSrcLoc, SrcLoc ) -import PrelInfo ( cCallishTyKeys ) -import Maybes ( maybeToBool ) -import Unique ( Uniquable(..) ) -import StringBuffer ( hGetStringBuffer ) -import FastString ( mkFastString ) -import ErrUtils ( Message ) -import Util ( sortLt ) -import Lex +import PrelInfo ( wiredInThingEnv, fractionalClassKeys ) +import TysWiredIn ( doubleTyCon ) +import Maybes ( orElse ) import FiniteMap import Outputable import Bag - -import List ( nub ) +import Util ( sortLt ) \end{code} %********************************************************* %* * -\subsection{Loading a new interface file} +\subsection{Getting what a module exports} %* * %********************************************************* +@getInterfaceExports@ is called only for directly-imported modules. + \begin{code} -loadHomeInterface :: SDoc -> Name -> RnM d Ifaces -loadHomeInterface doc_str name - = loadInterface doc_str (moduleName (nameModule name)) ImportBySystem - -loadOrphanModules :: [ModuleName] -> RnM d () -loadOrphanModules mods - | null mods = returnRn () - | otherwise = traceRn (text "Loading orphan modules:" <+> - fsep (map pprModuleName mods)) `thenRn_` - mapRn_ load mods `thenRn_` - returnRn () +getInterfaceExports :: ModuleName -> WhereFrom -> RnMG (Module, [(ModuleName,Avails)]) +getInterfaceExports mod_name from + = loadInterface doc_str mod_name from `thenRn` \ iface -> + returnRn (mi_module iface, mi_exports iface) where - load mod = loadInterface (mk_doc mod) mod ImportBySystem - mk_doc mod = pprModuleName mod <+> ptext SLIT("is a orphan-instance module") - - -loadInterface :: SDoc -> ModuleName -> WhereFrom -> RnM d Ifaces -loadInterface doc mod from - = tryLoadInterface doc mod from `thenRn` \ (ifaces, maybe_err) -> - case maybe_err of - Nothing -> returnRn ifaces - Just err -> failWithRn ifaces err - -tryLoadInterface :: SDoc -> ModuleName -> WhereFrom -> RnM d (Ifaces, Maybe Message) - -- Returns (Just err) if an error happened - -- Guarantees to return with iImpModInfo m --> (... Just cts) - -- (If the load fails, we plug in a vanilla placeholder -tryLoadInterface doc_str mod_name from - = getIfacesRn `thenRn` \ ifaces -> - let - mod_map = iImpModInfo ifaces - mod_info = lookupFM mod_map mod_name - - hi_boot_file - = case (from, mod_info) of - (ImportByUser, _) -> False -- Not hi-boot - (ImportByUserSource, _) -> True -- hi-boot - (ImportBySystem, Just (_, is_boot, _)) -> is_boot -- - (ImportBySystem, Nothing) -> False - -- We're importing a module we know absolutely - -- nothing about, so we assume it's from - -- another package, where we aren't doing - -- dependency tracking. So it won't be a hi-boot file. - - redundant_source_import - = case (from, mod_info) of - (ImportByUserSource, Just (_,False,_)) -> True - other -> False - in - -- CHECK WHETHER WE HAVE IT ALREADY - case mod_info of { - Just (_, _, True) - -> -- We're read it already so don't re-read it - returnRn (ifaces, Nothing) ; - - _ -> - - -- Issue a warning for a redundant {- SOURCE -} import - -- NB that we arrange to read all the ordinary imports before - -- any of the {- SOURCE -} imports - warnCheckRn (not redundant_source_import) - (warnRedundantSourceImport mod_name) `thenRn_` - - -- READ THE MODULE IN - findAndReadIface doc_str mod_name hi_boot_file `thenRn` \ read_resultb -> - case read_result of { - Left err -> -- Not found, so add an empty export env to the Ifaces map - -- so that we don't look again - let - new_mod_map = addToFM mod_map mod_name (False, False, True) - new_ifaces = ifaces { iImpModInfo = new_mod_map } - in - setIfacesRn new_ifaces `thenRn_` - returnRn (new_ifaces, Just err) ; - - -- Found and parsed! - Right (mod, iface) -> - - -- LOAD IT INTO Ifaces - - -- NB: *first* we do loadDecl, so that the provenance of all the locally-defined - --- names is done correctly (notably, whether this is an .hi file or .hi-boot file). - -- If we do loadExport first the wrong info gets into the cache (unless we - -- explicitly tag each export which seems a bit of a bore) - - - -- Sanity check. If we're system-importing a module we know nothing at all - -- about, it should be from a different package to this one - WARN( not (maybeToBool mod_info) && - case from of { ImportBySystem -> True; other -> False } && - isLocalModule mod, - ppr mod ) - - loadDecls mod (iDecls ifaces) (pi_decls iface) `thenRn` \ (decls_vers, new_decls) -> - loadRules mod (iRules ifaces) (pi_rules iface) `thenRn` \ (rule_vers, new_rules) -> - loadFixDecls mod_name (pi_fixity iface) `thenRn` \ (fix_vers, fix_env) -> - foldlRn (loadDeprec mod) emptyDeprecEnv (pi_deprecs iface) `thenRn` \ deprec_env -> - foldlRn (loadInstDecl mod) (iInsts ifaces) (pi_insts iface) `thenRn` \ new_insts -> - loadExports (pi_exports iface) `thenRn` \ avails -> - let - version = VersionInfo { modVers = pi_vers iface, - fixVers = fix_vers, - ruleVers = rule_vers, - declVers = decl_vers } - - -- For an explicit user import, add to mod_map info about - -- the things the imported module depends on, extracted - -- from its usage info. - mod_map1 = case from of - ImportByUser -> addModDeps mod (pi_usages iface) mod_map - other -> mod_map - mod_map2 = addToFM mod_map1 mod_name (pi_orphan iface, hi_boot_file, True) - - -- Now add info about this module to the PST - new_pst = extendModuleEnv pst mod mod_detils - mod_details = ModDetails { mdModule = mod, mvVersion = version, - mdExports = avails, - mdFixEnv = fix_env, mdDeprecEnv = deprec_env } - - new_ifaces = ifaces { iPST = new_pst, - iDecls = new_decls, - iInsts = new_insts, - iRules = new_rules, - iImpModInfo = mod_map2 } - in - setIfacesRn new_ifaces `thenRn_` - returnRn (new_ifaces, Nothing) - }} + doc_str = sep [ppr mod_name, ptext SLIT("is directly imported")] +\end{code} ------------------------------------------------------ --- Adding module dependencies from the --- import decls in the interface file ------------------------------------------------------ - -addModDeps :: Module -> PackageSymbolTable -> [ImportVersion a] - -> ImportedModuleInfo -> ImportedModuleInfo --- (addModDeps M ivs deps) --- We are importing module M, and M.hi contains 'import' decls given by ivs -addModDeps mod new_deps mod_deps - = foldr add mod_deps filtered_new_deps - where - -- Don't record dependencies when importing a module from another package - -- Except for its descendents which contain orphans, - -- and in that case, forget about the boot indicator - filtered_new_deps :: (ModuleName, (WhetherHasOrphans, IsBootInterface)) - filtered_new_deps - | isLocalModule mod = [ (imp_mod, (has_orphans, is_boot, False)) - | (imp_mod, has_orphans, is_boot, _) <- new_deps - ] - | otherwise = [ (imp_mod, (True, False, False)) - | (imp_mod, has_orphans, _, _) <- new_deps, - has_orphans - ] - add (imp_mod, dep) deps = addToFM_C combine deps imp_mod dep - - combine old@(_, old_is_boot, old_is_loaded) new - | old_is_loaded || not old_is_boot = old -- Keep the old info if it's already loaded - -- or if it's a non-boot pending load - | otherwise = new -- Otherwise pick new info - - ------------------------------------------------------ --- Loading the export list ------------------------------------------------------ - -loadExports :: [ExportItem] -> RnM d Avails -loadExports items - = getModuleRn `thenRn` \ this_mod -> - mapRn (loadExport this_mod) items `thenRn` \ avails_s -> - returnRn (concat avails_s) - - -loadExport :: Module -> ExportItem -> RnM d [AvailInfo] -loadExport this_mod (mod, entities) - | mod == moduleName this_mod = returnRn [] - -- If the module exports anything defined in this module, just ignore it. - -- Reason: otherwise it looks as if there are two local definition sites - -- for the thing, and an error gets reported. Easiest thing is just to - -- filter them out up front. This situation only arises if a module - -- imports itself, or another module that imported it. (Necessarily, - -- this invoves a loop.) Consequence: if you say - -- module A where - -- import B( AType ) - -- type AType = ... - -- - -- module B( AType ) where - -- import {-# SOURCE #-} A( AType ) - -- - -- then you'll get a 'B does not export AType' message. A bit bogus - -- but it's a bogus thing to do! - | otherwise - = mapRn (load_entity mod) entities - where - new_name mod occ = newGlobalName mod occ - - load_entity mod (Avail occ) - = new_name mod occ `thenRn` \ name -> - returnRn (Avail name) - load_entity mod (AvailTC occ occs) - = new_name mod occ `thenRn` \ name -> - mapRn (new_name mod) occs `thenRn` \ names -> - returnRn (AvailTC name names) - - ------------------------------------------------------ --- Loading type/class/value decls ------------------------------------------------------ - -loadDecls :: Module - -> DeclsMap - -> [(Version, RdrNameHsDecl)] - -> RnM d (NameEnv Version, DeclsMap) -loadDecls mod decls_map decls - = foldlRn (loadDecl mod) (emptyNameEnv, decls_map) decls - -loadDecl :: Module - -> (NameEnv Version, DeclsMap) - -> (Version, RdrNameHsDecl) - -> RnM d (NameEnv Version, DeclsMap) -loadDecl mod (version_map, decls_map) (version, decl) - = getDeclBinders new_name decl `thenRn` \ maybe_avail -> - case maybe_avail of { - Nothing -> returnRn (version_map, decls_map); -- No bindings - Just avail -> - - getDeclSysBinders new_name decl `thenRn` \ sys_bndrs -> - let - full_avail = addSysAvails avail sys_bndrs - -- Add the sys-binders to avail. When we import the decl, - -- it's full_avail that will get added to the 'already-slurped' set (iSlurp) - -- If we miss out sys-binders, we'll read the decl multiple times! - - main_name = availName avail - new_decls_map = foldl add_decl decls_map - [ (name, (full_avail, name==main_name, (mod, decl'))) - | name <- availNames full_avail] - add_decl decls_map (name, stuff) - = WARN( name `elemNameEnv` decls_map, ppr name ) - extendNameEnv decls_map name stuff - - new_version_map = extendNameEnv version_map main_name version - in - returnRn (new_version_map, new_decls_map) - } - where - -- newTopBinder puts into the cache the binder with the - -- module information set correctly. When the decl is later renamed, - -- the binding site will thereby get the correct module. - -- There maybe occurrences that don't have the correct Module, but - -- by the typechecker will propagate the binding definition to all - -- the occurrences, so that doesn't matter - new_name rdr_name loc = newTopBinder mod rdr_name loc - - {- - If a signature decl is being loaded, and optIgnoreIfacePragmas is on, - we toss away unfolding information. - - Also, if the signature is loaded from a module we're importing from source, - we do the same. This is to avoid situations when compiling a pair of mutually - recursive modules, peering at unfolding info in the interface file of the other, - e.g., you compile A, it looks at B's interface file and may as a result change - its interface file. Hence, B is recompiled, maybe changing its interface file, - which will the unfolding info used in A to become invalid. Simple way out is to - just ignore unfolding info. - - [Jan 99: I junked the second test above. If we're importing from an hi-boot - file there isn't going to *be* any pragma info. Maybe the above comment - dates from a time where we picked up a .hi file first if it existed?] - -} - decl' = case decl of - SigD (IfaceSig name tp ls loc) | opt_IgnoreIfacePragmas - -> SigD (IfaceSig name tp [] loc) - other -> decl - ------------------------------------------------------ --- Loading fixity decls ------------------------------------------------------ - -loadFixDecls mod_name (version, decls) - | null decls = returnRn (version, emptyNameEnv) +%********************************************************* +%* * +\subsection{Keeping track of what we've slurped, and version numbers} +%* * +%********************************************************* - | otherwise - = mapRn (loadFixDecl mod_name) decls `thenRn` \ to_add -> - returnRn (version, mkNameEnv to_add) - -loadFixDecl mod_name sig@(FixitySig rdr_name fixity loc) - = newGlobalName mod_name (rdrNameOcc rdr_name) `thenRn` \ name -> - returnRn (name, FixitySig name fixity loc) - - ------------------------------------------------------ --- Loading instance decls ------------------------------------------------------ - -loadInstDecl :: Module - -> IfaceInsts - -> RdrNameInstDecl - -> RnM d IfaceInsts -loadInstDecl mod insts decl@(InstDecl inst_ty binds uprags dfun_name src_loc) - = - -- Find out what type constructors and classes are "gates" for the - -- instance declaration. If all these "gates" are slurped in then - -- we should slurp the instance decl too. - -- - -- We *don't* want to count names in the context part as gates, though. - -- For example: - -- instance Foo a => Baz (T a) where ... - -- - -- Here the gates are Baz and T, but *not* Foo. - let - munged_inst_ty = removeContext inst_ty - free_names = extractHsTyRdrNames munged_inst_ty - in - setModuleRn mod $ - mapRn lookupOrigName free_names `thenRn` \ gate_names -> - returnRn ((mkNameSet gate_names, (mod, InstD decl)) `consBag` insts) +mkImportInof 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. +We produce a line for every module B below the module, A, currently being +compiled: + import B ; +to record the fact that A does import B indireclty. 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. --- In interface files, the instance decls now look like --- forall a. Foo a -> Baz (T a) --- so we have to strip off function argument types as well --- as the bit before the '=>' (which is always empty in interface files) -removeContext (HsForAllTy tvs cxt ty) = HsForAllTy tvs [] (removeFuns ty) -removeContext ty = removeFuns ty +\begin{code} +mkImportInfo :: ModuleName -- Name of this module + -> [ImportDecl n] -- The import decls + -> RnMG [ImportVersion Name] -removeFuns (HsFunTy _ ty) = removeFuns ty -removeFuns ty = ty +mkImportInfo this_mod imports + = getIfacesRn `thenRn` \ ifaces -> + getHomeIfaceTableRn `thenRn` \ hit -> + let + (imp_pkg_mods, imp_home_names) = iVSlurp ifaces + pit = iPIT ifaces + + import_all_mods :: [ModuleName] + -- Modules where we imported all the names + -- (apart from hiding some, perhaps) + import_all_mods = [ m | ImportDecl m _ _ _ imp_list _ <- imports, + import_all imp_list ] + where + import_all (Just (False, _)) = False -- Imports are specified explicitly + import_all other = True -- Everything is imported + + -- mv_map groups together all the things imported and used + -- from a particular module in this package + -- We use a finite map because we want the domain + mv_map :: ModuleEnv [Name] + mv_map = foldNameSet add_mv emptyModuleEnv imp_home_names + add_mv name mv_map = extendModuleEnv_C add_item mv_map mod [name] + where + mod = nameModule name + add_item names _ = name:names + + -- In our usage list we record + -- a) Specifically: Detailed version info for imports from modules in this package + -- Gotten from iVSlurp plus import_all_mods + -- + -- b) Everything: Just the module version for imports from modules in other packages + -- Gotten from iVSlurp plus import_all_mods + -- + -- c) NothingAtAll: The name only of modules, Baz, in this package that are 'below' us, + -- but which we didn't need at all (this is needed only to decide whether + -- to open Baz.hi or Baz.hi-boot higher up the tree). + -- 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 info. + -- These modules are in a combination of HIT/PIT and iImpModInfo + -- + -- d) NothingAtAll: The name only of all orphan modules we know of (this is needed + -- so that anyone who imports us can find the orphan modules) + -- These modules are in a combination of HIT/PIT and iImpModInfo + + import_info0 = foldModuleEnv mk_imp_info [] pit + import_info1 = foldModuleEnv mk_imp_info import_info0 hit + import_info = [ (mod_name, orphans, is_boot, NothingAtAll) + | (mod_name, (orphans, is_boot)) <- fmToList (iImpModInfo ifaces) ] ++ + import_info1 + + mk_imp_info :: ModIface -> [ImportVersion Name] -> [ImportVersion Name] + mk_imp_info iface so_far + | Just ns <- lookupModuleEnv mv_map mod -- Case (a) + = go_for_it (Specifically mod_vers maybe_export_vers + (mk_import_items ns) rules_vers) ------------------------------------------------------ --- Loading Rules ------------------------------------------------------ + | mod `elemModuleSet` imp_pkg_mods -- Case (b) + = go_for_it (Everything mod_vers) -loadRules :: Module -> IfaceRules - -> (Version, [RdrNameRuleDecl]) - -> RnM d (Version, IfaceRules) -loadRules mod rule_bag (version, rules) - | null rules || opt_IgnoreIfacePragmas - = returnRn (version, rule_bag) - | otherwise - = setModuleRn mod $ - mapRn (loadRule mod) rules `thenRn` \ new_rules -> - returnRn (version, rule_bag `unionBags` listToBag new_rules) - -loadRule :: Module -> RdrNameRuleDecl -> RnM d GatedDecl --- "Gate" the rule simply by whether the rule variable is --- needed. We can refine this later. -loadRule mod decl@(IfaceRule _ _ var _ _ src_loc) - = lookupOrigName var `thenRn` \ var_name -> - returnRn (unitNameSet var_name, (mod, RuleD decl)) - -loadBuiltinRules :: [(RdrName, CoreRule)] -> RnMG () -loadBuiltinRules builtin_rules - = getIfacesRn `thenRn` \ ifaces -> - mapRn loadBuiltinRule builtin_rules `thenRn` \ rule_decls -> - setIfacesRn (ifaces { iRules = iRules ifaces `unionBags` listToBag rule_decls }) - -loadBuiltinRule (var, rule) - = lookupOrigName var `thenRn` \ var_name -> - returnRn (unitNameSet var_name, (nameModule var_name, RuleD (IfaceRuleOut var rule))) - - ------------------------------------------------------ --- Loading Deprecations ------------------------------------------------------ - -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_` - -- SUP: TEMPORARY HACK, ignoring module deprecations for now - returnRn deprec_env - -loadDeprec mod deprec_env (Deprecation ie txt _) - = setModuleRn mod $ - mapRn lookupOrigName (ieNames ie) `thenRn` \ names -> - traceRn (text "loaded deprecation(s) for" <+> hcat (punctuate comma (map ppr names)) <> colon <+> ppr txt) `thenRn_` - returnRn (extendNameEnvList deprec_env (zip names (repeat txt))) + | import_all_mod -- Case (a) and (b); the import-all part + = if is_home_pkg_mod then + go_for_it (Specifically mod_vers (Just export_vers) [] rules_vers) + else + go_for_it (Everything mod_vers) + + | is_home_pkg_mod || has_orphans -- Case (c) or (d) + = go_for_it NothingAtAll + + | otherwise = so_far + where + go_for_it exports = (mod_name, has_orphans, mi_boot iface, exports) : so_far + + mod = mi_module iface + mod_name = moduleName mod + is_home_pkg_mod = isHomeModule mod + version_info = mi_version iface + version_env = vers_decls version_info + mod_vers = vers_module version_info + rules_vers = vers_rules version_info + export_vers = vers_exports version_info + import_all_mod = mod_name `elem` import_all_mods + has_orphans = mi_orphan iface + + -- The sort is to put them into canonical order + mk_import_items ns = [(n,v) | n <- sortLt lt_occ ns, + let v = lookupNameEnv version_env n `orElse` + pprPanic "mk_whats_imported" (ppr n) + ] + where + lt_occ n1 n2 = nameOccName n1 < nameOccName n2 + + maybe_export_vers | import_all_mod = Just (vers_exports version_info) + | otherwise = Nothing + in + returnRn import_info \end{code} - -%******************************************************** -%* * -\subsection{Checking usage information} -%* * -%******************************************************** +%********************************************************* +%* * +\subsection{Slurping declarations} +%* * +%********************************************************* \begin{code} -upToDate = True -outOfDate = False +------------------------------------------------------- +slurpImpDecls source_fvs + = traceRn (text "slurpImp" <+> fsep (map ppr (nameSetToList source_fvs))) `thenRn_` -checkModUsage :: [ImportVersion OccName] -> RnMG Bool --- Given the usage information extracted from the old --- M.hi file for the module being compiled, figure out --- whether M needs to be recompiled. + -- The current slurped-set records all local things + slurpSourceRefs source_fvs `thenRn` \ (decls, needed) -> -checkModUsage [] = returnRn upToDate -- Yes! Everything is up to date! + -- Then get everything else + closeDecls decls needed -checkModUsage ((mod_name, _, _, NothingAtAll) : 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, _, _, whats_imported) : rest) - = tryLoadInterface doc_str mod_name ImportBySystem `thenRn` \ (ifaces, maybe_err) -> - case maybe_err of { - Just err -> out_of_date (sep [ptext SLIT("Can't find version number for module"), - pprModuleName mod_name]) ; - -- Couldn't find or parse a module mentioned in the - -- old interface file. Don't complain -- it might just be that - -- the current module doesn't need that import and it's been deleted +------------------------------------------------------- +slurpSourceRefs :: FreeVars -- Variables referenced in source + -> RnMG ([RenamedHsDecl], + FreeVars) -- Un-satisfied needs +-- The declaration (and hence home module) of each gate has +-- already been loaded - Nothing -> - let - (_, new_mod_vers, new_fix_vers, new_rule_vers, _, _) - = case lookupFM (iImpModInfo ifaces) mod_name of - Just (_, _, Just stuff) -> stuff - - old_mod_vers = case whats_imported of - Everything v -> v - Specifically v _ _ _ -> v - -- NothingAtAll case dealt with by previous eqn for checkModUsage - in - -- If the module version hasn't changed, just move on - if new_mod_vers == old_mod_vers then - traceRn (sep [ptext SLIT("Module version unchanged:"), pprModuleName mod_name]) - `thenRn_` checkModUsage rest - else - traceRn (sep [ptext SLIT("Module version has changed:"), pprModuleName mod_name]) - `thenRn_` - -- Module version changed, so check entities inside +slurpSourceRefs source_fvs + = go_outer [] -- Accumulating decls + emptyFVs -- Unsatisfied needs + emptyFVs -- Accumulating gates + (nameSetToList source_fvs) -- Things whose defn hasn't been loaded yet + where + -- The outer loop repeatedly slurps the decls for the current gates + -- and the instance decls + + -- The outer loop is needed because consider + + go_outer decls fvs all_gates [] + = returnRn (decls, fvs) + + go_outer decls fvs all_gates refs -- refs are not necessarily slurped yet + = traceRn (text "go_outer" <+> ppr refs) `thenRn_` + foldlRn go_inner (decls, fvs, emptyFVs) refs `thenRn` \ (decls1, fvs1, gates1) -> + getImportedInstDecls (all_gates `plusFV` gates1) `thenRn` \ inst_decls -> + rnIfaceInstDecls decls1 fvs1 gates1 inst_decls `thenRn` \ (decls2, fvs2, gates2) -> + go_outer decls2 fvs2 (all_gates `plusFV` gates2) + (nameSetToList (gates2 `minusNameSet` all_gates)) + -- Knock out the all_gates because even if we don't slurp any new + -- decls we can get some apparently-new gates from wired-in names + + go_inner (decls, fvs, gates) wanted_name + = importDecl wanted_name `thenRn` \ import_result -> + case import_result of + AlreadySlurped -> returnRn (decls, fvs, gates) + InTypeEnv ty_thing -> returnRn (decls, fvs, gates `plusFV` getWiredInGates ty_thing) + + HereItIs decl -> rnIfaceTyClDecl decl `thenRn` \ (new_decl, fvs1) -> + returnRn (TyClD new_decl : decls, + fvs1 `plusFV` fvs, + gates `plusFV` getGates source_fvs new_decl) +\end{code} - -- If the usage info wants to say "I imported everything from this module" - -- it does so by making whats_imported equal to Everything - -- In that case, we must recompile - case whats_imported of { -- NothingAtAll dealt with earlier - - Everything _ - -> out_of_date (ptext SLIT("...and I needed the whole module")) ; - Specifically _ old_fix_vers old_rule_vers old_local_vers -> +\begin{code} +------------------------------------------------------- +-- closeDecls keeps going until the free-var set is empty +closeDecls decls needed + | not (isEmptyFVs needed) + = slurpDecls decls needed `thenRn` \ (decls1, needed1) -> + closeDecls decls1 needed1 - if old_fix_vers /= new_fix_vers then - out_of_date (ptext SLIT("Fixities changed")) - else if old_rule_vers /= new_rule_vers then - out_of_date (ptext SLIT("Rules changed")) - else - -- Non-empty usage list, so check item by item - checkEntityUsage mod_name (iDecls ifaces) old_local_vers `thenRn` \ up_to_date -> - if up_to_date then - traceRn (ptext SLIT("...but the bits I use haven't.")) `thenRn_` - checkModUsage rest -- This one's ok, so check the rest - else - returnRn outOfDate -- This one failed, so just bail out now - }} + | otherwise + = getImportedRules `thenRn` \ rule_decls -> + case rule_decls of + [] -> returnRn decls -- No new rules, so we are done + other -> rnIfaceDecls rnIfaceRuleDecl rule_decls `thenRn` \ rule_decls' -> + let + rule_fvs = plusFVs (map ruleDeclFVs rule_decls') + in + traceRn (text "closeRules" <+> ppr rule_decls' $$ fsep (map ppr (nameSetToList rule_fvs))) `thenRn_` + closeDecls (map RuleD rule_decls' ++ decls) rule_fvs + + + +------------------------------------------------------- +-- Augment decls with any decls needed by needed. +-- Return also free vars of the new decls (only) +slurpDecls decls needed + = go decls emptyFVs (nameSetToList needed) where - doc_str = sep [ptext SLIT("need version info for"), pprModuleName mod_name] + go decls fvs [] = returnRn (decls, fvs) + go decls fvs (ref:refs) = slurpDecl decls fvs ref `thenRn` \ (decls1, fvs1) -> + go decls1 fvs1 refs + +------------------------------------------------------- +slurpDecl decls fvs wanted_name + = importDecl wanted_name `thenRn` \ import_result -> + case import_result of + -- Found a declaration... rename it + HereItIs decl -> rnIfaceTyClDecl decl `thenRn` \ (new_decl, fvs1) -> + returnRn (TyClD new_decl:decls, fvs1 `plusFV` fvs) + + -- No declaration... (wired in thing, or deferred, or already slurped) + other -> returnRn (decls, fvs) + + +------------------------------------------------------- +rnIfaceDecls rn decls = mapRn (rnIfaceDecl rn) decls +rnIfaceDecl rn (mod, decl) = initIfaceRnMS mod (rn decl) + +rnIfaceInstDecls decls fvs gates inst_decls + = rnIfaceDecls rnInstDecl inst_decls `thenRn` \ inst_decls' -> + returnRn (map InstD inst_decls' ++ decls, + fvs `plusFV` plusFVs (map instDeclFVs inst_decls'), + gates `plusFV` plusFVs (map getInstDeclGates inst_decls')) + +rnIfaceTyClDecl (mod, decl) = initIfaceRnMS mod (rnTyClDecl decl) `thenRn` \ decl' -> + returnRn (decl', tyClDeclFVs decl') +\end{code} -checkEntityUsage mod decls [] - = returnRn upToDate -- Yes! All up to date! +\begin{code} +getSlurped + = getIfacesRn `thenRn` \ ifaces -> + returnRn (iSlurp ifaces) + +recordSlurp ifaces@(Ifaces { iDecls = (decls_map, n_slurped), + iSlurp = slurped_names, + iVSlurp = (imp_mods, imp_names) }) + avail + = ASSERT2( not (isLocalName (availName avail)), ppr avail ) + ifaces { iDecls = (decls_map', n_slurped+1), + iSlurp = new_slurped_names, + iVSlurp = new_vslurp } + where + decls_map' = foldl delFromNameEnv decls_map (availNames avail) + main_name = availName avail + mod = nameModule main_name + new_slurped_names = addAvailToNameSet slurped_names avail + new_vslurp | isHomeModule mod = (imp_mods, addOneToNameSet imp_names main_name) + | otherwise = (extendModuleSet imp_mods mod, imp_names) + +recordLocalSlurps new_names + = getIfacesRn `thenRn` \ ifaces -> + setIfacesRn (ifaces { iSlurp = iSlurp ifaces `unionNameSets` new_names }) +\end{code} + -checkEntityUsage mod decls ((occ_name,old_vers) : rest) - = newGlobalName mod occ_name `thenRn` \ name -> - case lookupNameEnv decls name of - Nothing -> -- We used it before, but it ain't there now - out_of_date (sep [ptext SLIT("No longer exported:"), ppr name]) +%********************************************************* +%* * +\subsection{Extracting the `gates'} +%* * +%********************************************************* - Just (new_vers,_,_,_) -- It's there, but is it up to date? - | new_vers == old_vers - -- Up to date, so check the rest - -> checkEntityUsage mod decls rest +The gating story +~~~~~~~~~~~~~~~~~ +We want to avoid sucking in too many instance declarations. +An instance decl is only useful if the types and classes mentioned in +its 'head' are all available in the program being compiled. E.g. - | otherwise - -- Out of date, so bale out - -> out_of_date (sep [ptext SLIT("Out of date:"), ppr name]) + instance (..) => C (T1 a) (T2 b) where ... -out_of_date msg = traceRn msg `thenRn_` returnRn outOfDate -\end{code} +is only useful if C, T1 and T2 are all "available". So we keep +instance decls that have been parsed from .hi files, but not yet +slurped in, in a pool called the 'gated instance pool'. +Each has its set of 'gates': {C, T1, T2} in the above example. +More precisely, the gates of a module are the types and classes +that are mentioned in: -%********************************************************* -%* * -\subsection{Getting in a declaration} -%* * -%********************************************************* + a) the source code + b) the type of an Id that's mentioned in the source code + [includes constructors and selectors] + c) the RHS of a type synonym that is a gate + d) the superclasses of a class that is a gate + e) the context of an instance decl that is slurped in -\begin{code} -importDecl :: Name -> RnMG ImportDeclResult +We slurp in an instance decl from the gated instance pool iff + + all its gates are either in the gates of the module, + or are a previously-loaded class. -data ImportDeclResult - = AlreadySlurped - | WiredIn - | Deferred - | HereItIs (Module, RdrNameHsDecl) +The latter constraint is because there might have been an instance +decl slurped in during an earlier compilation, like this: -importDecl name - = getIfacesRn `thenRn` \ ifaces -> - getHomeSymbolTableRn `thenRn` \ hst -> - if name `elemNameSet` iSlurp ifaces - || inTypeEnv (iPST ifaces) name - || inTypeEnv hst name - then -- Already dealt with - returnRn AlreadySlurped + instance Foo a => Baz (Maybe a) where ... - else if isLocallyDefined name then -- Don't bring in decls from - -- the renamed module's own interface file - addWarnRn (importDeclWarn name) `thenRn_` - returnRn AlreadySlurped +In the module being compiled we might need (Baz (Maybe T)), where T +is defined in this module, and hence we need (Foo T). So @Foo@ becomes +a gate. But there's no way to 'see' that, so we simply treat all +previously-loaded classes as gates. - else if isWiredInName name then - -- When we find a wired-in name we must load its - -- home module so that we find any instance decls therein - loadHomeInterface doc name `thenRn_` - returnRn WiredIn +Consructors and class operations +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ +When we import a declaration like - else getNonWiredInDecl name - where - doc = ptext SLIT("need home module for wired in thing") <+> ppr name - -getNonWiredInDecl :: Name -> RnMG ImportDeclResult -getNonWiredInDecl needed_name - = traceRn doc_str `thenRn_` - loadHomeInterface doc_str needed_name `thenRn` \ ifaces -> - case lookupNameEnv (iDecls ifaces) needed_name of - - Just (version, avail, is_tycon_name, decl@(_, TyClD (TyData DataType _ _ _ _ ncons _ _ _ _ _))) - -- This case deals with deferred import of algebraic data types - - | not opt_NoPruneTyDecls - - && (opt_IgnoreIfacePragmas || ncons > 1) - -- We only defer if imported interface pragmas are ingored - -- or if it's not a product type. - -- Sole reason: The wrapper for a strict function may need to look - -- inside its arg, and hence need to see its arg type's constructors. - - && not (getUnique tycon_name `elem` cCallishTyKeys) - -- Never defer ccall types; we have to unbox them, - -- and importing them does no harm - - -> -- OK, so we're importing a deferrable data type - if needed_name == tycon_name then - -- The needed_name is the TyCon of a data type decl - -- Record that it's slurped, put it in the deferred set - -- and don't return a declaration at all - setIfacesRn (recordSlurp (ifaces {iDeferred = iDeferred ifaces - `addOneToNameSet` tycon_name}) - version (AvailTC needed_name [needed_name])) `thenRn_` - returnRn Deferred - else - -- The needed name is a constructor of a data type decl, - -- getting a constructor, so remove the TyCon from the deferred set - -- (if it's there) and return the full declaration - setIfacesRn (recordSlurp (ifaces {iDeferred = iDeferred ifaces - `delFromNameSet` tycon_name}) - version avail) `thenRn_` - returnRn (HereItIs decl) - where - tycon_name = availName avail - - Just (version,avail,_,decl) - -> setIfacesRn (recordSlurp ifaces version avail) `thenRn_` - returnRn (HereItIs decl) + data T = T1 Wibble | T2 Wobble - Nothing - -> addErrRn (getDeclErr needed_name) `thenRn_` - returnRn AlreadySlurped - where - doc_str = ptext SLIT("need decl for") <+> ppr needed_name +we don't want to treat @Wibble@ and @Wobble@ as gates {\em unless} +@T1@, @T2@ respectively are mentioned by the user program. If only +@T@ is mentioned we want only @T@ to be a gate; that way we don't suck +in useless instance decls for (say) @Eq Wibble@, when they can't +possibly be useful. -getDeferredDecls :: RnMG [(Module, RdrNameHsDecl)] -getDeferredDecls - = getIfacesRn `thenRn` \ ifaces -> - let - decls_map = iDecls ifaces - deferred_names = nameSetToList (iDeferred ifaces) - get_abstract_decl n = case lookupNameEnv decls_map n of - Just (_, _, _, decl) -> decl - in - traceRn (sep [text "getDeferredDecls", nest 4 (fsep (map ppr deferred_names))]) `thenRn_` - returnRn (map get_abstract_decl deferred_names) -\end{code} +And that's just what (b) says: we only treat T1's type as a gate if +T1 is mentioned. getGates, which deals with decls we are slurping in, +has to be a bit careful, because a mention of T1 will slurp in T's whole +declaration. -@getWiredInDecl@ maps a wired-in @Name@ to what it makes available. -It behaves exactly as if the wired in decl were actually in an interface file. -Specifically, -\begin{itemize} -\item if the wired-in name is a data type constructor or a data constructor, - it brings in the type constructor and all the data constructors; and - marks as ``occurrences'' any free vars of the data con. +----------------------------- +@getGates@ takes a newly imported (and renamed) decl, and the free +vars of the source program, and extracts from the decl the gate names. -\item similarly for synonum type constructor +\begin{code} +getGates :: FreeVars -- Things mentioned in the source program + -> RenamedTyClDecl + -> FreeVars + +getGates source_fvs decl + = get_gates (\n -> n `elemNameSet` source_fvs) decl -\item if the wired-in name is another wired-in Id, it marks as ``occurrences'' - the free vars of the Id's type. +get_gates is_used (IfaceSig _ ty _ _) + = extractHsTyNames ty -\item it loads the interface file for the wired-in thing for the - sole purpose of making sure that its instance declarations are available -\end{itemize} -All this is necessary so that we know all types that are ``in play'', so -that we know just what instances to bring into scope. +get_gates is_used (ClassDecl ctxt cls tvs _ sigs _ _ _ ) + = (delListFromNameSet (foldr (plusFV . get) (extractHsCtxtTyNames ctxt) sigs) + (hsTyVarNames tvs) + `addOneToNameSet` cls) + `plusFV` maybe_double + where + get (ClassOpSig n _ ty _) + | is_used n = extractHsTyNames ty + | otherwise = emptyFVs + + -- If we load any numeric class that doesn't have + -- Int as an instance, add Double to the gates. + -- This takes account of the fact that Double might be needed for + -- defaulting, but we don't want to load Double (and all its baggage) + -- if the more exotic classes aren't used at all. + maybe_double | nameUnique cls `elem` fractionalClassKeys + = unitFV (getName doubleTyCon) + | otherwise + = emptyFVs + +get_gates is_used (TySynonym tycon tvs ty _) + = delListFromNameSet (extractHsTyNames ty) (hsTyVarNames tvs) + -- A type synonym type constructor isn't a "gate" for instance decls + +get_gates is_used (TyData _ ctxt tycon tvs cons _ _ _ _ _) + = delListFromNameSet (foldr (plusFV . get) (extractHsCtxtTyNames ctxt) cons) + (hsTyVarNames tvs) + `addOneToNameSet` tycon + where + get (ConDecl n _ tvs ctxt details _) + | is_used n + -- If the constructor is method, get fvs from all its fields + = delListFromNameSet (get_details details `plusFV` + extractHsCtxtTyNames ctxt) + (hsTyVarNames tvs) + get (ConDecl n _ tvs ctxt (RecCon fields) _) + -- Even if the constructor isn't mentioned, the fields + -- might be, as selectors. They can't mention existentially + -- bound tyvars (typechecker checks for that) so no need for + -- the deleteListFromNameSet part + = foldr (plusFV . get_field) emptyFVs fields + get other_con = emptyFVs + get_details (VanillaCon tys) = plusFVs (map get_bang tys) + get_details (InfixCon t1 t2) = get_bang t1 `plusFV` get_bang t2 + get_details (RecCon fields) = plusFVs [get_bang t | (_, t) <- fields] - -%********************************************************* -%* * -\subsection{Getting what a module exports} -%* * -%********************************************************* + get_field (fs,t) | any is_used fs = get_bang t + | otherwise = emptyFVs -@getInterfaceExports@ is called only for directly-imported modules. + get_bang bty = extractHsTyNames (getBangType bty) +\end{code} + +@getWiredInGates@ is just like @getGates@, but it sees a previously-loaded +thing rather than a declaration. \begin{code} -getInterfaceExports :: ModuleName -> WhereFrom -> RnMG (Module, Avails) -getInterfaceExports mod_name from - = getHomeSymbolTableRn `thenRn` \ hst -> - case lookupModuleEnvByName hst mod_name of { - Just mds -> returnRn (mdModule mds, mdExports mds) ; - - - loadInterface doc_str mod_name from `thenRn` \ ifaces -> - case lookupModuleEnv (iPST ifaces) mod_name of - Just mds -> returnRn (mdModule mod, mdExports mds) - -- loadInterface always puts something in the map - -- even if it's a fake +getWiredInGates :: TyThing -> FreeVars +-- The TyThing is one that we already have in our type environment, either +-- a) because the TyCon or Id is wired in, or +-- b) from a previous compile +-- Either way, we might have instance decls in the (persistent) collection +-- of parsed-but-not-slurped instance decls that should be slurped in. +-- This might be the first module that mentions both the type and the class +-- for that instance decl, even though both the type and the class were +-- mentioned in other modules, and hence are in the type environment + +getWiredInGates (AnId the_id) = namesOfType (idType the_id) +getWiredInGates (AClass cl) = emptyFVs -- The superclasses must also be previously + -- loaded, and hence are automatically gates +getWiredInGates (ATyCon tc) + | isSynTyCon tc = delListFromNameSet (namesOfType ty) (map getName tyvars) + | otherwise = unitFV (getName tc) where - doc_str = sep [pprModuleName mod_name, ptext SLIT("is directly imported")] -\end{code} - + (tyvars,ty) = getSynTyConDefn tc -%********************************************************* -%* * -\subsection{Instance declarations are handled specially} -%* * -%********************************************************* +getInstDeclGates (InstDecl inst_ty _ _ _ _) = extractHsTyNames inst_ty +\end{code} \begin{code} -getImportedInstDecls :: NameSet -> RnMG [(Module,RdrNameHsDecl)] +getImportedInstDecls :: NameSet -> RnMG [(Module,RdrNameInstDecl)] getImportedInstDecls gates = -- First, load any orphan-instance modules that aren't aready loaded -- Orphan-instance modules are recorded in the module dependecnies getIfacesRn `thenRn` \ ifaces -> let orphan_mods = - [mod | (mod, (True, _, Nothing)) <- fmToList (iImpModInfo ifaces)] + [mod | (mod, (True, _)) <- fmToList (iImpModInfo ifaces)] in loadOrphanModules orphan_mods `thenRn_` @@ -733,8 +521,9 @@ getImportedInstDecls gates -- Find the un-gated ones and return them, -- removing them from the bag kept in Ifaces getIfacesRn `thenRn` \ ifaces -> + getTypeEnvRn `thenRn` \ lookup -> let - (decls, new_insts) = selectGated gates (iInsts ifaces) + (decls, new_insts) = selectGated gates lookup (iInsts ifaces) in setIfacesRn (ifaces { iInsts = new_insts }) `thenRn_` @@ -746,20 +535,21 @@ getImportedInstDecls gates where gate_list = nameSetToList gates -ppr_brief_inst_decl (mod, InstD (InstDecl inst_ty _ _ _ _)) +ppr_brief_inst_decl (mod, InstDecl inst_ty _ _ _ _) = case inst_ty of HsForAllTy _ _ tau -> ppr tau other -> ppr inst_ty -getImportedRules :: RnMG [(Module,RdrNameHsDecl)] +getImportedRules :: RnMG [(Module,RdrNameRuleDecl)] getImportedRules | opt_IgnoreIfacePragmas = returnRn [] | otherwise = getIfacesRn `thenRn` \ ifaces -> + getTypeEnvRn `thenRn` \ lookup -> let gates = iSlurp ifaces -- Anything at all that's been slurped rules = iRules ifaces - (decls, new_rules) = selectGated gates rules + (decls, new_rules) = selectGated gates lookup rules in if null decls then returnRn [] @@ -769,427 +559,235 @@ getImportedRules text "Slurped" <+> int (length decls) <+> text "rules"]) `thenRn_` returnRn decls -selectGated gates decl_bag +selectGated gates lookup (decl_bag, n_slurped) -- Select only those decls whose gates are *all* in 'gates' + -- or are a class in 'lookup' #ifdef DEBUG | opt_NoPruneDecls -- Just to try the effect of not gating at all - = (foldrBag (\ (_,d) ds -> d:ds) [] decl_bag, emptyBag) -- Grab them all + = let + decls = foldrBag (\ (_,d) ds -> d:ds) [] decl_bag -- Grab them all + in + (decls, (emptyBag, n_slurped + length decls)) | otherwise #endif - = foldrBag select ([], emptyBag) decl_bag + = case foldrBag select ([], emptyBag) decl_bag of + (decls, new_bag) -> (decls, (new_bag, n_slurped + length decls)) where + available n = n `elemNameSet` gates + || case lookup n of { Just (AClass c) -> True; other -> False } + select (reqd, decl) (yes, no) - | isEmptyNameSet (reqd `minusNameSet` gates) = (decl:yes, no) - | otherwise = (yes, (reqd,decl) `consBag` no) - -lookupFixityRn :: Name -> RnMS Fixity -lookupFixityRn name - | isLocallyDefined name - = getFixityEnv `thenRn` \ local_fix_env -> - returnRn (lookupLocalFixity local_fix_env name) - - | 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. - = getHomeSymbolTableRn `thenRn` \ hst -> - case lookupFixityEnv hst name of { - Just fixity -> returnRn fixity ; - Nothing -> - - loadHomeInterface doc name `thenRn` \ ifaces -> - returnRn (lookupFixityEnv (iPST ifaces) name `orElse` defaultFixity) - } - where - doc = ptext SLIT("Checking fixity for") <+> ppr name + | all available reqd = (decl:yes, no) + | otherwise = (yes, (reqd,decl) `consBag` no) \end{code} %********************************************************* %* * -\subsection{Keeping track of what we've slurped, and version numbers} +\subsection{Getting in a declaration} %* * %********************************************************* -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 (a) anything reachable from its body code -\item (b) any module exported with a @module Foo@ -\item (c) anything reachable from an exported item -\end{itemize} - -Why (b)? Because if @Foo@ changes then this module's export list -will change, so we must recompile this module at least as far as -making a new interface file --- but in practice that means complete -recompilation. - -Why (c)? Consider this: -\begin{verbatim} - module A( f, g ) where | module B( f ) where - import B( f ) | f = h 3 - g = ... | h = ... -\end{verbatim} - -Here, @B.f@ isn't used in A. Should we nevertheless record @B.f@ in -@A@'s usages? Our idea is that we aren't going to touch A.hi if it is -*identical* to what it was before. If anything about @B.f@ changes -than anyone who imports @A@ should be recompiled in case they use -@B.f@ (they'll get an early exit if they don't). So, if anything -about @B.f@ changes we'd better make sure that something in A.hi -changes, and the convenient way to do that is to record the version -number @B.f@ in A.hi in the usage list. If B.f changes that'll force a -complete recompiation of A, which is overkill but it's the only way to -write a new, slightly different, A.hi. - -But the example is tricker. 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. But with -O, a module that imports A must be recompiled if -@B.h@ changes! So A must record a dependency on @B.h@. So we treat -the occurrence of @B.f@ in the export list *just as if* it were in the -code of A, and thereby haul in all the stuff reachable from it. - -[NB: If B was compiled with -O, but A isn't, we should really *still* -haul in all the unfoldings for B, in case the module that imports A *is* -compiled with -O. I think this is the case.] - -Even if B is used at all we get a usage line for B - 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} -mkImportExportInfo :: ModuleName -- Name of this module - -> Avails -- Info about exports - -> Maybe [RdrNameIE] -- The export header - -> RnMG ([ExportItem], -- Export info for iface file; sorted - [ImportVersion OccName]) -- Import info for iface file; sorted - -- Both results are sorted into canonical order to - -- reduce needless wobbling of interface files - -mkImportExportInfo this_mod export_avails exports - = getIfacesRn `thenRn` \ ifaces -> - let - export_all_mods = case exports of - Nothing -> [] - Just es -> [mod | IEModuleContents mod <- es, - mod /= this_mod] - - mod_map = iImpModInfo ifaces - imp_names = iVSlurp ifaces - - -- mv_map groups together all the things imported from a particular module. - mv_map :: FiniteMap ModuleName [(OccName,Version)] - mv_map = foldr add_mv emptyFM imp_names - - add_mv (name, version) mv_map = addItem mv_map (moduleName (nameModule name)) - (nameOccName name, version) - - -- 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_imp_info mod_name (has_orphans, is_boot, contents) so_far - | mod_name == this_mod -- Check if M appears in the set of modules 'below' M - -- This seems like a convenient place to check - = WARN( not is_boot, ptext SLIT("Wierd:") <+> ppr this_mod <+> - ptext SLIT("imports itself (perhaps indirectly)") ) - so_far - - | otherwise - = let - go_for_it exports = (mod_name, 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. - -- The module in question must be a local module (in the same package) - go_for_it NothingAtAll - - Just (mod, mod_vers, fix_vers, rule_vers, how_imported, _) - | is_sys_import && 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 mod_vers) - - | otherwise - -> case lookupFM mv_map mod_name of - Just whats_imported -> go_for_it (Specifically mod_vers fix_vers rule_vers - (sortImport whats_imported)) - Nothing -> go_for_it NothingAtAll - -- 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 = not (isLocalModule mod) - is_sys_import = case how_imported of - ImportBySystem -> True - other -> False - - - import_info = foldFM mk_imp_info [] mod_map - - -- Sort exports into groups by module - export_fm :: FiniteMap ModuleName [RdrAvailInfo] - export_fm = foldr insert emptyFM export_avails - - insert avail efm = addItem efm (moduleName (nameModule (availName avail))) - (rdrAvailInfo avail) - - export_info = [(m, sortExport as) | (m,as) <- fmToList export_fm] - in - traceRn (text "Modules in Ifaces: " <+> fsep (map ppr (keysFM mod_map))) `thenRn_` - returnRn (export_info, import_info) +importDecl :: Name -> RnMG ImportDeclResult +data ImportDeclResult + = AlreadySlurped + | InTypeEnv TyThing + | HereItIs (Module, RdrNameTyClDecl) -addItem :: FiniteMap ModuleName [a] -> ModuleName -> a -> FiniteMap ModuleName [a] -addItem fm mod x = addToFM_C add_item fm mod [x] - where - add_item xs _ = x:xs +importDecl name + = -- STEP 1: Check if we've slurped it in while compiling this module + getIfacesRn `thenRn` \ ifaces -> + if name `elemNameSet` iSlurp ifaces then + returnRn AlreadySlurped + else -sortImport :: [(OccName,Version)] -> [(OccName,Version)] - -- Make the usage lists appear in canonical order -sortImport vs = sortLt lt vs - where - lt (n1,v1) (n2,v2) = n1 < n2 + -- STEP 2: Check if it's already in the type environment + getTypeEnvRn `thenRn` \ lookup -> + case lookup name of { + Just ty_thing | name `elemNameEnv` wiredInThingEnv + -> -- When we find a wired-in name we must load its home + -- module so that we find any instance decls lurking therein + loadHomeInterface wi_doc name `thenRn_` + returnRn (InTypeEnv ty_thing) -sortExport :: [RdrAvailInfo] -> [RdrAvailInfo] -sortExport as = sortLt lt as - where - lt a1 a2 = availName a1 < availName a2 -\end{code} + | otherwise + -> returnRn (InTypeEnv ty_thing) ; -\begin{code} -getSlurped - = getIfacesRn `thenRn` \ ifaces -> - returnRn (iSlurp ifaces) + Nothing -> -recordSlurp ifaces@(Ifaces { iSlurp = slurped_names, iVSlurp = imp_names }) - version avail - = let - new_slurped_names = addAvailToNameSet slurped_names avail - new_imp_names = (availName avail, version) : imp_names - in - ifaces { iSlurp = new_slurped_names, iVSlurp = new_imp_names } + -- STEP 3: OK, we have to slurp it in from an interface file + -- First load the interface file + traceRn nd_doc `thenRn_` + loadHomeInterface nd_doc name `thenRn_` + getIfacesRn `thenRn` \ ifaces -> -recordLocalSlurps local_avails - = getIfacesRn `thenRn` \ ifaces -> + -- STEP 4: Get the declaration out let - new_slurped_names = foldl addAvailToNameSet (iSlurp ifaces) local_avails + (decls_map, _) = iDecls ifaces in - setIfacesRn (ifaces { iSlurp = new_slurped_names }) + case lookupNameEnv decls_map name of + Just (avail,_,decl) + -> setIfacesRn (recordSlurp ifaces avail) `thenRn_` + returnRn (HereItIs decl) + + Nothing + -> addErrRn (getDeclErr name) `thenRn_` + returnRn AlreadySlurped + } + where + wi_doc = ptext SLIT("need home module for wired in thing") <+> ppr name + nd_doc = ptext SLIT("need decl for") <+> ppr name + \end{code} -%********************************************************* +%******************************************************** %* * -\subsection{Getting binders out of a declaration} +\subsection{Checking usage information} %* * -%********************************************************* - -@getDeclBinders@ returns the names for a @RdrNameHsDecl@. -It's used for both source code (from @availsFromDecl@) and interface files -(from @loadDecl@). +%******************************************************** -It doesn't deal with source-code specific things: @ValD@, @DefD@. They -are handled by the sourc-code specific stuff in @RnNames@. +@recompileRequired@ is called from the HscMain. It checks whether +a recompilation is required. It needs access to the persistent state, +finder, etc, because it may have to load lots of interface files to +check their versions. \begin{code} -getDeclBinders :: (RdrName -> SrcLoc -> RnM d Name) -- New-name function - -> RdrNameHsDecl - -> RnM d (Maybe AvailInfo) - -getDeclBinders new_name (TyClD (TyData _ _ tycon _ condecls _ _ _ src_loc _ _)) - = new_name tycon src_loc `thenRn` \ tycon_name -> - getConFieldNames new_name condecls `thenRn` \ sub_names -> - returnRn (Just (AvailTC tycon_name (tycon_name : nub sub_names))) - -- The "nub" is because getConFieldNames can legitimately return duplicates, - -- when a record declaration has the same field in multiple constructors +type RecompileRequired = Bool +upToDate = False -- Recompile not required +outOfDate = True -- Recompile required + +recompileRequired :: FilePath -- Only needed for debug msgs + -> Bool -- Source unchanged + -> ModIface -- Old interface + -> RnMG RecompileRequired +recompileRequired iface_path source_unchanged iface + = traceHiDiffsRn (text "Considering whether compilation is required for" <+> text iface_path <> colon) `thenRn_` + + -- CHECK WHETHER THE SOURCE HAS CHANGED + if not source_unchanged then + traceHiDiffsRn (nest 4 (text "Source file changed or recompilation check turned off")) `thenRn_` + returnRn outOfDate + else -getDeclBinders new_name (TyClD (TySynonym tycon _ _ src_loc)) - = new_name tycon src_loc `thenRn` \ tycon_name -> - returnRn (Just (AvailTC tycon_name [tycon_name])) + -- Source code unchanged and no errors yet... carry on + checkList [checkModUsage u | u <- mi_usages iface] -getDeclBinders new_name (TyClD (ClassDecl _ cname _ _ sigs _ _ _ src_loc)) - = new_name cname src_loc `thenRn` \ class_name -> +checkList :: [RnMG RecompileRequired] -> RnMG RecompileRequired +checkList [] = returnRn upToDate +checkList (check:checks) = check `thenRn` \ recompile -> + if recompile then + returnRn outOfDate + else + checkList checks +\end{code} + +\begin{code} +checkModUsage :: ImportVersion Name -> RnMG RecompileRequired +-- Given the usage information extracted from the old +-- M.hi file for the module being compiled, figure out +-- whether M needs to be recompiled. - -- Record the names for the class ops +checkModUsage (mod_name, _, _, NothingAtAll) + -- 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. + = up_to_date (ptext SLIT("Nothing used from:") <+> ppr mod_name) + +checkModUsage (mod_name, _, is_boot, whats_imported) + = -- Load the imported interface is possible + -- We use tryLoadInterface, because failure is not an error + -- (might just be that the old .hi file for this module is out of date) + -- We use ImportByUser/ImportByUserSource as the 'from' flag, + -- a) because we need to know whether to load the .hi-boot file + -- b) because loadInterface things matters are amiss if we + -- ImportBySystem an interface it knows nothing about let - -- just want class-op sigs - op_sigs = filter isClassOpSig sigs + doc_str = sep [ptext SLIT("need version info for"), ppr mod_name] + from | is_boot = ImportByUserSource + | otherwise = ImportByUser in - mapRn (getClassOpNames new_name) op_sigs `thenRn` \ sub_names -> - - returnRn (Just (AvailTC class_name (class_name : sub_names))) - -getDeclBinders new_name (SigD (IfaceSig var ty prags src_loc)) - = new_name var src_loc `thenRn` \ var_name -> - returnRn (Just (Avail var_name)) + tryLoadInterface doc_str mod_name from `thenRn` \ (iface, maybe_err) -> -getDeclBinders new_name (FixD _) = returnRn Nothing -getDeclBinders new_name (DeprecD _) = 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 - = lookupOrigName nm `thenRn_` - returnRn Nothing + case maybe_err of { + Just err -> out_of_date (sep [ptext SLIT("Can't find version number for module"), + ppr mod_name]) ; + -- Couldn't find or parse a module mentioned in the + -- old interface file. Don't complain -- it might just be that + -- the current module doesn't need that import and it's been deleted -getDeclBinders new_name (DefD _) = returnRn Nothing -getDeclBinders new_name (InstD _) = returnRn Nothing -getDeclBinders new_name (RuleD _) = returnRn Nothing + Nothing -> + let + new_vers = mi_version iface + new_decl_vers = vers_decls new_vers + in + case whats_imported of { -- NothingAtAll dealt with earlier -binds_haskell_name (FoImport _) _ = True -binds_haskell_name FoLabel _ = True -binds_haskell_name FoExport ext_nm = isDynamicExtName ext_nm + Everything old_mod_vers -> checkModuleVersion old_mod_vers new_vers `thenRn` \ recompile -> + if recompile then + out_of_date (ptext SLIT("...and I needed the whole module")) + else + returnRn upToDate ; ----------------- -getConFieldNames new_name (ConDecl con _ _ _ (RecCon fielddecls) src_loc : rest) - = mapRn (\n -> new_name n src_loc) (con:fields) `thenRn` \ cfs -> - getConFieldNames new_name rest `thenRn` \ ns -> - returnRn (cfs ++ ns) - where - fields = concat (map fst fielddecls) + Specifically old_mod_vers maybe_old_export_vers old_decl_vers old_rule_vers -> -getConFieldNames new_name (ConDecl con _ _ _ condecl src_loc : rest) - = new_name con src_loc `thenRn` \ n -> - getConFieldNames new_name rest `thenRn` \ ns -> - returnRn (n : ns) + -- CHECK MODULE + checkModuleVersion old_mod_vers new_vers `thenRn` \ recompile -> + if not recompile then + returnRn upToDate + else + + -- CHECK EXPORT LIST + if checkExportList maybe_old_export_vers new_vers then + out_of_date (ptext SLIT("Export list changed")) + else -getConFieldNames new_name [] = returnRn [] + -- CHECK RULES + if old_rule_vers /= vers_rules new_vers then + out_of_date (ptext SLIT("Rules changed")) + else -getClassOpNames new_name (ClassOpSig op _ _ src_loc) = new_name op src_loc -\end{code} + -- CHECK ITEMS ONE BY ONE + checkList [checkEntityUsage new_decl_vers u | u <- old_decl_vers] `thenRn` \ recompile -> + if recompile then + returnRn outOfDate -- This one failed, so just bail out now + else + up_to_date (ptext SLIT("...but the bits I use haven't.")) -@getDeclSysBinders@ gets the implicit binders introduced by a decl. -A the moment that's just the tycon and datacon that come with a class decl. -They aren't returned by @getDeclBinders@ because they aren't in scope; -but they {\em should} be put into the @DeclsMap@ of this module. + }} -Note that this excludes the default-method names of a class decl, -and the dict fun of an instance decl, because both of these have -bindings of their own elsewhere. +------------------------ +checkModuleVersion old_mod_vers new_vers + | vers_module new_vers == old_mod_vers + = up_to_date (ptext SLIT("Module version unchanged")) -\begin{code} -getDeclSysBinders new_name (TyClD (ClassDecl _ cname _ _ sigs _ _ names - src_loc)) - = sequenceRn [new_name n src_loc | n <- names] + | otherwise + = out_of_date (ptext SLIT("Module version has changed")) -getDeclSysBinders new_name (TyClD (TyData _ _ _ _ cons _ _ _ _ _ _)) - = sequenceRn [new_name wkr_name src_loc | ConDecl _ wkr_name _ _ _ src_loc <- cons] +------------------------ +checkExportList Nothing new_vers = upToDate +checkExportList (Just v) new_vers = v /= vers_exports new_vers -getDeclSysBinders new_name other_decl - = returnRn [] -\end{code} +------------------------ +checkEntityUsage new_vers (name,old_vers) + = case lookupNameEnv new_vers name of -%********************************************************* -%* * -\subsection{Reading an interface file} -%* * -%********************************************************* + Nothing -> -- We used it before, but it ain't there now + out_of_date (sep [ptext SLIT("No longer exported:"), ppr name]) -\begin{code} -findAndReadIface :: SDoc -> ModuleName - -> IsBootInterface -- True <=> Look for a .hi-boot file - -- False <=> Look for .hi file - -> RnM d (Either Message (Module, ParsedIface)) - -- Nothing <=> file not found, or unreadable, or illegible - -- Just x <=> successfully found and parsed - -findAndReadIface doc_str mod_name hi_boot_file - = traceRn trace_msg `thenRn_` - -- we keep two maps for interface files, - -- one for 'normal' ones, the other for .hi-boot files, - -- hence the need to signal which kind we're interested. - - getFinderRn `thenRn` \ finder -> - ioToRn (findModule finder mod_name) `thenRn` \ maybe_module -> - - case maybe_module of - Just mod | hi_boot_file, Just fpath <- moduleHiBootFile mod - -> readIface mod fpath - | not hi_boot_file, Just fpath <- moduleHiFile mod - -> readIface mod fpath - - -- Can't find it - other -> traceRn (ptext SLIT("...not found")) `thenRn_` - returnRn (Left (noIfaceErr finder mod_name hi_boot_file)) + Just new_vers -- It's there, but is it up to date? + | new_vers == old_vers -> returnRn upToDate + | otherwise -> out_of_date (sep [ptext SLIT("Out of date:"), ppr name]) - where - trace_msg = sep [hsep [ptext SLIT("Reading"), - if hi_boot_file then ptext SLIT("[boot]") else empty, - ptext SLIT("interface for"), - pprModuleName mod_name <> semi], - nest 4 (ptext SLIT("reason:") <+> doc_str)] +up_to_date msg = traceHiDiffsRn msg `thenRn_` returnRn upToDate +out_of_date msg = traceHiDiffsRn msg `thenRn_` returnRn outOfDate \end{code} -@readIface@ tries just the one file. - -\begin{code} -readIface :: Module -> String -> RnM d (Either Message (Module, ParsedIface)) - -- Nothing <=> file not found, or unreadable, or illegible - -- Just x <=> successfully found and parsed -readIface wanted_mod file_path - = traceRn (ptext SLIT("...reading from") <+> text file_path) `thenRn_` - ioToRnM (hGetStringBuffer False file_path) `thenRn` \ read_result -> - case read_result of - Right contents -> - case parseIface contents - PState{ bol = 0#, atbol = 1#, - context = [], - glasgow_exts = 1#, - loc = mkSrcLoc (mkFastString file_path) 1 } of - POk _ (PIface iface) -> - warnCheckRn (moduleName wanted_mod == read_mod) - (hiModuleNameMismatchWarn wanted_mod read_mod) `thenRn_` - returnRn (Right (mod, iface)) - where - read_mod = moduleName (pi_mod iface) - - PFailed err -> bale_out err - parse_result -> bale_out empty - -- 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 io_err -> bale_out (text (show io_err)) - where - bale_out err = returnRn (Left (badIfaceFile file_path err)) -\end{code} %********************************************************* %* * @@ -1198,45 +796,8 @@ readIface wanted_mod file_path %********************************************************* \begin{code} -noIfaceErr mod_name boot_file search_path - = vcat [ptext SLIT("Could not find interface file for") <+> quotes (pprModuleName mod_name), - ptext SLIT("in the directories") <+> - -- \& to avoid cpp interpreting this string as a - -- comment starter with a pre-4.06 mkdependHS --SDM - vcat [ text dir <> text "/\&*" <> pp_suffix suffix - | (dir,suffix) <- search_path] - ] - where - pp_suffix suffix | boot_file = ptext SLIT(".hi-boot") - | otherwise = text suffix - -badIfaceFile file err - = vcat [ptext SLIT("Bad interface file:") <+> text file, - nest 4 err] - getDeclErr name = vcat [ptext SLIT("Failed to find interface decl for") <+> quotes (ppr name), ptext SLIT("from module") <+> quotes (ppr (nameModule name)) ] - -importDeclWarn name - = sep [ptext SLIT( - "Compiler tried to import decl from interface file with same name as module."), - ptext SLIT( - "(possible cause: module name clashes with interface file already in scope.)") - ] $$ - hsep [ptext SLIT("name:"), quotes (ppr name)] - -warnRedundantSourceImport mod_name - = ptext SLIT("Unnecessary {- SOURCE -} in the import of module") - <+> quotes (pprModuleName mod_name) - -hiModuleNameMismatchWarn :: Module -> ModuleName -> Message -hiModuleNameMismatchWarn requested_mod read_mod = - hsep [ ptext SLIT("Something is amiss; requested module name") - , ppr requested_mod - , ptext SLIT("differs from name found in the interface file") - , pprModuleName read_mod - ] - \end{code}