Skip to content

Commit

Permalink
wip: remove qualifyTop
Browse files Browse the repository at this point in the history
  • Loading branch information
facundominguez committed Dec 12, 2024
1 parent 4cf2465 commit c8f9a2a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 8 deletions.
5 changes: 2 additions & 3 deletions liquidhaskell-boot/src/Language/Haskell/Liquid/Bare/Axiom.hs
Original file line number Diff line number Diff line change
Expand Up @@ -310,11 +310,10 @@ makeAxiom :: Bare.Env -> Bare.TycEnv -> ModName -> LogicMap
-> (LocSymbol, Maybe SpecType, Ghc.Var, Ghc.CoreExpr)
-> (Ghc.Var, LocSpecType, F.Equation)
--------------------------------------------------------------------------------
makeAxiom env tycEnv name lmap (x, mbT, v, def)
makeAxiom env tycEnv _name lmap (x, mbT, v, def)
= (v, t, e)
where
t = Bare.qualifyTop env name (F.loc t0) t0
(t0, e) = makeAssumeType allowTC embs lmap dm x mbT v def
(t, e) = makeAssumeType allowTC embs lmap dm x mbT v def
embs = Bare.tcEmbs tycEnv
dm = Bare.tcDataConMap tycEnv
allowTC = typeclass (getConfig env)
Expand Down
5 changes: 0 additions & 5 deletions liquidhaskell-boot/src/Language/Haskell/Liquid/Bare/Expand.hs
Original file line number Diff line number Diff line change
Expand Up @@ -491,10 +491,6 @@ cookSpecTypeE env sigEnv name@(ModName _ _) x bt
. fmap txExpToBind -- What does this function DO
. (specExpandType rtEnv . fmap (generalizeWith x))
. (if doplug || not allowTC then maybePlug allowTC sigEnv name x else id)
-- we do not qualify/resolve Expr/Pred when typeclass is enabled
-- since ghci will not be able to recognize fully qualified names
-- instead, we leave qualification to ghc elaboration
. Bare.qualifyTop env name l

allowTC = typeclass (getConfig env)
-- modT = mname `S.member` wiredInMods
Expand All @@ -509,7 +505,6 @@ cookSpecTypeE env sigEnv name@(ModName _ _) x bt
rtEnv = Bare.sigRTEnv sigEnv
embs = Bare.sigEmbs sigEnv
tyi = Bare.sigTyRTyMap sigEnv
l = F.loc bt

-- | We don't want to generalize type variables that maybe bound in the
-- outer scope, e.g. see tests/basic/pos/LocalPlug00.hs
Expand Down

0 comments on commit c8f9a2a

Please sign in to comment.