-
Notifications
You must be signed in to change notification settings - Fork 22
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
Translate quantifier weights to Boogie #443
Conversation
The following test fails in CI, even after restarting the job two times. It seems deterministic.
I failed to reprodue this error. Even when using the exact same Docker image
Strangely, running Carbon with
The |
b95fa46
to
2b1a1d1
Compare
The failing test is in the My current best guess is that there is a deterministic memory leak in Carbon or in the test suite. When the CI runs out of memory, it might be that Z3 or Boogie is killed, which would explain the "broken pipe" error message. Then, Carbon or Boogie restarts the killed process and the remaining tests work fine. I'm trying to identify the memory leak. |
a515490
to
52c8642
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
4c2548f
to
f73cb5b
Compare
This comment was marked as resolved.
This comment was marked as resolved.
f73cb5b
to
868669c
Compare
As discussed in the meeting, we will look into merging this PR after the January 2022 release. |
868669c
to
c463698
Compare
With this PR Carbon translates quantifier weights (added in viperproject/silver#633, used in viperproject/silicon#674) to Boogie. I added a test to show how to set the weight in the AST.