Conversation
This adds support for typecast expressions that cast to the type of the operand to the SMT2 converter.
aaed2bb to
a6997dd
Compare
tautschnig
left a comment
There was a problem hiding this comment.
Seems fine and should be supported for sure, but why are we running into this at all? The simplifier should have gotten rid of this, so I suppose there is code that never sees the simplifier.
Yes, we never made an effort to ensure that only simplified expressions are given to solvers. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #8828 +/- ##
========================================
Coverage 80.00% 80.00%
========================================
Files 1700 1700
Lines 188332 188335 +3
Branches 73 73
========================================
+ Hits 150679 150685 +6
+ Misses 37653 37650 -3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
I don't think that should be a requirement either, but it may point to a missed opportunity to use the simplifier. |
This adds support for typecast expressions that cast to the type of the operand to the SMT2 converter.