diff options
author | Morgan Deters <mdeters@cs.nyu.edu> | 2013-02-07 18:13:01 -0500 |
---|---|---|
committer | Morgan Deters <mdeters@cs.nyu.edu> | 2013-04-01 15:38:14 -0400 |
commit | 994e6eb72e3475967a9a40a0566744ce1794f20a (patch) | |
tree | fec05368ea52fbb00204ee3f3a168c2434610eae /src/smt | |
parent | bf7f728381bb27163f3e056d698ba4da6316b9c8 (diff) |
Fix bug 491 and related issues with checkModel() and quantifiers. Enabling previously-failing testcase.
Diffstat (limited to 'src/smt')
-rw-r--r-- | src/smt/smt_engine.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/smt/smt_engine.cpp b/src/smt/smt_engine.cpp index 21a37a43d..2b909a9a9 100644 --- a/src/smt/smt_engine.cpp +++ b/src/smt/smt_engine.cpp @@ -3369,7 +3369,7 @@ void SmtEngine::checkModel(bool hardFailure) { // We have a "fake context" for the substitution map (we don't need it // to be context-dependent) context::Context fakeContext; - SubstitutionMap substitutions(&fakeContext); + SubstitutionMap substitutions(&fakeContext, /* substituteUnderQuantifiers = */ false); for(size_t k = 0; k < m->getNumCommands(); ++k) { const DeclareFunctionCommand* c = dynamic_cast<const DeclareFunctionCommand*>(m->getCommand(k)); @@ -3457,13 +3457,13 @@ void SmtEngine::checkModel(bool hardFailure) { Notice() << "SmtEngine::checkModel(): -- simplifies to " << n << endl; TheoryId thy = Theory::theoryOf(n); - if(thy == THEORY_QUANTIFIERS || thy == THEORY_REWRITERULES) { + if(thy == THEORY_REWRITERULES) { // Note this "skip" is done here, rather than above. This is // because (1) the quantifier could in principle simplify to false, // which should be reported, and (2) checking for the quantifier // above, before simplification, doesn't catch buried quantifiers // anyway (those not at the top-level). - Notice() << "SmtEngine::checkModel(): -- skipping quantified assertion" + Notice() << "SmtEngine::checkModel(): -- skipping rewrite-rules assertion" << endl; continue; } |