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/arith/theory_arith.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/arith/theory_arith.h')
| -rw-r--r-- | src/theory/arith/theory_arith.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/theory/arith/theory_arith.h b/src/theory/arith/theory_arith.h index a208c9b10..41cc9591d 100644 --- a/src/theory/arith/theory_arith.h +++ b/src/theory/arith/theory_arith.h @@ -119,7 +119,7 @@ class TheoryArith : public Theory { * This calls the operator elimination utility to eliminate extended * symbols. */ - TrustNode ppRewrite(TNode atom) override; + TrustNode ppRewrite(TNode atom, std::vector<SkolemLemma>& lems) override; void ppStaticLearn(TNode in, NodeBuilder<>& learned) override; std::string identify() const override { return std::string("TheoryArith"); } |