diff options
author | Andres Noetzli <noetzli@stanford.edu> | 2019-05-15 17:38:04 +0000 |
---|---|---|
committer | Aina Niemetz <aina.niemetz@gmail.com> | 2019-05-15 10:38:04 -0700 |
commit | f2d113cf3cbb0f4966a7c909b9cd2c14aa753eb1 (patch) | |
tree | df20d8af54d7d7d33d5ab41e7f2401c51a10d2dc /src/options/options_handler.cpp | |
parent | bc550fa115401256616042ccb7a559ec252e319b (diff) |
Fix model of Boolean vars with eager bit-blaster (#2998)
When bit-blasting eagerly, we were not assigning values to the Boolean
variables in the `TheoryModel`. With eager bit-blasting, the BV SAT
solver gets all (converted) terms, including the Boolean ones, so
`EagerBitblaster::collectModelInfo()` is responsible for assigning
values to Boolean variables. However, it has only been assigning values
to bit-vector variables, which lead to wrong models. This commit fixes
the issue by asking the `CnfStream` for the Boolean variables, querying
the SAT solver for their value, and assigning them in the `TheoryModel`.
Diffstat (limited to 'src/options/options_handler.cpp')
0 files changed, 0 insertions, 0 deletions