Commit a06629b4 authored by John Ericson's avatar John Ericson Committed by Marge Bot

Do not throw away backpack instantiations for module lookup cache

Currently, there is only one home package so this probably doesn't
matter. But if we support multiple home packages, they could differ only
in arguments (same indef component being applied).

It looks like it used to be this way before
4e8a0607, but that commit doesn't seem
to comment on this change in the particular. (It's main purpose is
creating the InstalledUnitId and recategorizing the UnitId expressions
accordingly.)

Trying this as a separate commit for testing purposes. I leave it to
others to decide whether this is a good change on its own.
parent 98b0d6ee
Pipeline #10088 passed with stages
in 313 minutes and 57 seconds
......@@ -245,7 +245,7 @@ modLocationCache hsc_env mod do_this = do
mkHomeInstalledModule :: DynFlags -> ModuleName -> InstalledModule
mkHomeInstalledModule dflags mod_name =
let iuid = fst (splitUnitIdInsts (thisPackage dflags))
let iuid = thisInstalledUnitId dflags
in InstalledModule iuid mod_name
-- This returns a module because it's more convenient for users
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment