diff options
| author | Andrew Reynolds <andrew.j.reynolds@gmail.com> | 2021-03-10 13:07:39 -0600 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-03-10 13:07:39 -0600 |
| commit | 99acb6adc4858e9228a75283c0d4a640ce7cc812 (patch) | |
| tree | 4427782f5823505831ba11bd1915a17f10caeed1 /src/theory/bv/theory_bv.h | |
| parent | 132504c9f255fdb2c31b9a43bb3b9513db41afc1 (diff) | |
(proof-new) Update ppRewrite to use skolem lemmas (#6102)
This is required for proofs. The internal calculus no longer uses witness forms for reasoning, and hence we cannot return witness terms in ppRewrite. This is required to fix a debug failure, as well as making life easier on external proof conversions.
As a result of this PR, for example, given (P a) as input to ppRewrite, previous we returned:
(P (witness ((x T)) (A x)))
now we return:
(P k), with skolem lemma ( (A k), k )
Followup PRs will remove the use of WITNESS in ppRewrite (e.g. in sets and strings); this PR modifies arithmetic to not return WITNESS in response to ppRewrite.
Diffstat (limited to 'src/theory/bv/theory_bv.h')
| -rw-r--r-- | src/theory/bv/theory_bv.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/theory/bv/theory_bv.h b/src/theory/bv/theory_bv.h index 4e7cfdd2a..93e03e5ca 100644 --- a/src/theory/bv/theory_bv.h +++ b/src/theory/bv/theory_bv.h @@ -90,7 +90,7 @@ class TheoryBV : public Theory PPAssertStatus ppAssert(TrustNode in, TrustSubstitutionMap& outSubstitutions) override; - TrustNode ppRewrite(TNode t) override; + TrustNode ppRewrite(TNode t, std::vector<SkolemLemma>& lems) override; void ppStaticLearn(TNode in, NodeBuilder<>& learned) override; |