Revert 'Eliminate top level existentials in side-condtions'#3620
Revert 'Eliminate top level existentials in side-condtions'#3620goodlyrottenapple wants to merge 6 commits intomasterfrom
Conversation
| rhsBottom <- | ||
| fmap isBottom . liftSimplifier $ | ||
| SMT.Evaluator.filterMultiOr $srcLoc | ||
| =<< Pattern.simplify right |
There was a problem hiding this comment.
this didn't end up being the issue, but it's not correct and should be changed nonetheless.
|
As it turns out, one of the RPC server |
Does the test make sense in the first place? |
Yes, that's what happens. Just not sure why it would be such a bad thing to have an existential quantification over both term and predicate... |
|
Unfortunately, the "error somewhere else" is triggered by foundry proofs (I tested |
This PR fixes #3605
The bug was introduced by #3202, which eliminated top level existentials in side-conditions. However, this was incorrect because when translated to Z3, these variables would get a universal quantification, which is what was happening in #3605