diff options
author | Andrew Reynolds <andrew.j.reynolds@gmail.com> | 2020-11-19 11:44:35 -0600 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-11-19 11:44:35 -0600 |
commit | 3654512dd8b341c5725e550d438b23508493b991 (patch) | |
tree | 367684e5521e35a452ef669a4e58bb1b5005509c /src/theory/quantifiers | |
parent | b0130a1e032c201fab3c4b19f25871428b761967 (diff) |
Fix issues related to eliminating extended arithmetic operators (#5475)
This fixes 2 issues related to eliminating arithmetic operators:
(1) counterexample lemmas in CEGQI were not being preprocessed
(2) ensureLiteral was not doing term formula removal.
This corrects these two issues. Now ensureLiteral does full theory preprocessing on the term we are ensuring literal for, meaning this may trigger lemmas if the term contains extended arithmetic operators like div.
Fixes #5469, fixes #5470, fixes #5471.
This adds --sygus-inst to 2 of these benchmarks which moreover makes them solvable.
This also improves our assertions and trace messages.
Diffstat (limited to 'src/theory/quantifiers')
-rw-r--r-- | src/theory/quantifiers/cegqi/inst_strategy_cegqi.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/theory/quantifiers/cegqi/inst_strategy_cegqi.cpp b/src/theory/quantifiers/cegqi/inst_strategy_cegqi.cpp index 3bf3cc425..561817aa8 100644 --- a/src/theory/quantifiers/cegqi/inst_strategy_cegqi.cpp +++ b/src/theory/quantifiers/cegqi/inst_strategy_cegqi.cpp @@ -603,7 +603,8 @@ void InstStrategyCegqi::registerCounterexampleLemma(Node q, Node lem) ce_vars.push_back(tutil->getInstantiationConstant(q, i)); } CegInstantiator* cinst = getInstantiator(q); - LemmaStatus status = d_quantEngine->getOutputChannel().lemma(lem); + LemmaStatus status = + d_quantEngine->getOutputChannel().lemma(lem, LemmaProperty::PREPROCESS); Node ppLem = status.getRewrittenLemma(); Trace("cegqi-debug") << "Counterexample lemma (post-preprocess): " << ppLem << std::endl; |