-
Notifications
You must be signed in to change notification settings - Fork 273
Move Travis jobs using clang 6.0 to clang 7 #3245
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
More recent versions of Clang provide additional warnings that we may benefit from when run in CI.
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.
✔️
Passed Diffblue compatibility checks (cbmc commit: f51a3d4).
Build URL: https://travis-ci.com/diffblue/test-gen/builds/89532192
Will make a bump moving TG to the same version of clang (and a reminder that I never got my move to clang-6 merge!) |
@peterschrammel @thk123 @forejtv Could you please take a look at this one? #3301 should really not be happening. |
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 haven't yet got my TG clang-6 to work yet, but that isn't really a reason not to move to 7 here anyway.
Was saying 3.8, but CBMC moved to version 7 in #3245
Was saying 3.8, but CBMC moved to version 7 in #3245 Fixup previous commit fixup previous commit add explanation about --binary
More recent versions of Clang provide additional warnings that we may benefit
from when run in CI.