-
Notifications
You must be signed in to change notification settings - Fork 274
Remove deprecated exprt::make_true #3490
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚫
This PR failed Diffblue compatibility checks (cbmc commit: f56ea05).
Build URL: https://travis-ci.com/diffblue/test-gen/builds/93252762
Status will be re-evaluated on next push.
Please contact @peterschrammel, @thk123, or @allredj for support.
Common spurious failures:
- the cbmc commit has disappeared in the mean time (e.g. in a force-push)
- the author is not in the list of contributors (e.g. first-time contributors).
The incompatibility may have been introduced by an earlier PR. In that case merging this
PR should be avoided unless it fixes the current incompatibility.
@peterschrammel @thk123 @allredj This might require genuine changes in TG. Maybe those can just be done as |
All users should use true_exprt() instead.
f56ea05
to
d33be42
Compare
Yeah this needs a real bump bump - will create later today |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚫
This PR failed Diffblue compatibility checks (cbmc commit: d33be42).
Build URL: https://travis-ci.com/diffblue/test-gen/builds/93346369
Status will be re-evaluated on next push.
Please contact @peterschrammel, @thk123, or @allredj for support.
Common spurious failures:
- the cbmc commit has disappeared in the mean time (e.g. in a force-push)
- the author is not in the list of contributors (e.g. first-time contributors).
The incompatibility may have been introduced by an earlier PR. In that case merging this
PR should be avoided unless it fixes the current incompatibility.
Passing Test-gen update PR: https://github.com/diffblue/test-gen/pull/2526 |
Do you want to merge changes in TG before this one is merged, or is this one ok to merge? |
(it's ok to merge cbmc first) |
All users should use true_exprt() instead.