diff options
author | Andrew Reynolds <andrew.j.reynolds@gmail.com> | 2020-04-08 15:24:55 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-04-08 15:24:55 -0500 |
commit | 24357fea07bf1eb6b1156a8e455c58faee96b604 (patch) | |
tree | e89418fb98887c62d6d4f4e8d81d39829d61c1c7 /src | |
parent | 98a78889a4c4168f9c6632a0e19e1b8f422c4a9d (diff) |
Fix dump models and dump proofs (#4230)
A recent commit (45e489e) made it so that dump-models did not automatically enable produce-models in the global options object, but instead the SmtEngine enabled produce-models internally. The code for dump-models and dump-proofs was (perhaps out of paranoia) checking produce-models and produce-proofs. This removes this check, which is the correct thing to do since SmtEngine internally ensures produce-models is set.
Diffstat (limited to 'src')
-rw-r--r-- | src/main/command_executor.cpp | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/src/main/command_executor.cpp b/src/main/command_executor.cpp index 323f24492..c5ed54925 100644 --- a/src/main/command_executor.cpp +++ b/src/main/command_executor.cpp @@ -148,13 +148,15 @@ bool CommandExecutor::doCommandSingleton(Command* cmd) // dump the model/proof/unsat core if option is set if (status) { std::vector<std::unique_ptr<Command> > getterCommands; - if (d_options.getProduceModels() && d_options.getDumpModels() && - (res.asSatisfiabilityResult() == Result::SAT || - (res.isUnknown() && res.whyUnknown() == Result::INCOMPLETE))) { + if (d_options.getDumpModels() + && (res.asSatisfiabilityResult() == Result::SAT + || (res.isUnknown() && res.whyUnknown() == Result::INCOMPLETE))) + { getterCommands.emplace_back(new GetModelCommand()); } - if (d_options.getProof() && d_options.getDumpProofs() && - res.asSatisfiabilityResult() == Result::UNSAT) { + if (d_options.getDumpProofs() + && res.asSatisfiabilityResult() == Result::UNSAT) + { getterCommands.emplace_back(new GetProofCommand()); } |