-
Notifications
You must be signed in to change notification settings - Fork 277
switch codebuild from clang 7 to clang 8 #4746
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.
✔️
Passed Diffblue compatibility checks (cbmc commit: 4a83428).
Build URL: https://travis-ci.com/diffblue/test-gen/builds/114238943
Codecov Report
@@ Coverage Diff @@
## develop #4746 +/- ##
==========================================
Coverage ? 68.44%
==========================================
Files ? 1270
Lines ? 104828
Branches ? 0
==========================================
Hits ? 71753
Misses ? 33075
Partials ? 0 Continue to review full report at Codecov.
|
Yes, confirmed, cbmc-linux-clang builds with clang 8! |
d8cb021
to
c3f2646
Compare
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: 720e783).
Build URL: https://travis-ci.com/diffblue/test-gen/builds/114384475
Status will be re-evaluated on next push.
Common spurious failures include: 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); compatibility was already broken by an earlier merge.
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: d8cb021).
Build URL: https://travis-ci.com/diffblue/test-gen/builds/114386110
Status will be re-evaluated on next push.
Common spurious failures include: 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); compatibility was already broken by an earlier merge.
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: c3f2646).
Build URL: https://travis-ci.com/diffblue/test-gen/builds/114388107
This is to catch problems as in #4744.
The default constructor cannot exist; marking it as 'delete' re-enables compilation with clang.
c3f2646
to
f4451b5
Compare
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: f4451b5).
Build URL: https://travis-ci.com/diffblue/test-gen/builds/114459416
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.
CI passed
This is to catch problems as in #4744.