-
Notifications
You must be signed in to change notification settings - Fork 21
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
asserting in
#455
base: master
Are you sure you want to change the base?
asserting in
#455
Conversation
…nfig # Conflicts: # src/main/scala/viper/carbon/verifier/BoogieInterface.scala
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.
I think the pull request needs splitting into a silver one and a carbon one.
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.
I'm not sure how pull requests work across submodules; could this be a separate pull request for silver?
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.
In any case, github doesn't seem to understand how to present the diffs for these files.
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.
I did make two pull requests: see viperproject/silver#660, this pull request references the silver commit in the silver pull request
case sil.Unfolding(_, exp) => | ||
translateExp(exp) | ||
case sil.Applying(_, exp) => translateExp(exp) | ||
case h: sil.HintExp => translateExp(h.body) |
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 is nice to have. Is HintExp a sealed trait?
Requires viperproject/silver#660
This is my attempt at adding supporting the
asserting in
feature in Carbon.The "tests/all/asserting-in/wand.vpr" test from silver fails. Although, when I was initially working on this, this was also an issue when encoding
asserting in
using function preconditions, now the error seems specific toasserting in
, so it may be worth looking into.