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/uf/theory_uf.cpp | |
| 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/uf/theory_uf.cpp')
| -rw-r--r-- | src/theory/uf/theory_uf.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/theory/uf/theory_uf.cpp b/src/theory/uf/theory_uf.cpp index 273e81740..32524b562 100644 --- a/src/theory/uf/theory_uf.cpp +++ b/src/theory/uf/theory_uf.cpp @@ -206,7 +206,7 @@ void TheoryUF::notifyFact(TNode atom, bool pol, TNode fact, bool isInternal) } //--------------------------------- end standard check -TrustNode TheoryUF::ppRewrite(TNode node) +TrustNode TheoryUF::ppRewrite(TNode node, std::vector<SkolemLemma>& lems) { Trace("uf-exp-def") << "TheoryUF::ppRewrite: expanding definition : " << node << std::endl; |