-
Notifications
You must be signed in to change notification settings - Fork 274
Proof fails on OSX 10.15 that worked earlier versions #5388
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
Comments
Note: the relevant section of queue.h is
|
generated with
|
@danielsn Would GCC/Clang on Catalina actually accept that input? Doing |
@danielsn Can you check whether your GCC/Clang accepts
? Even https://clang.llvm.org/docs/LanguageExtensions.html#enumerations-with-a-fixed-underlying-type suggests this is for Objective C, but perhaps this slipped into a C source nonetheless. |
@tautschnig It runs with no errors
|
|
Yes this appears to be a clang extension. We’ll have to check if/how we can support it in the frontend (not sure off the top of my head if our enum representation has an easy way to change the underlying type, but it shouldn’t be too hard). |
We are currently investigating how to implement this in the C front end for CBMC. |
We now have a PR for supporting the syntax, but not yet the semantics (i.e. you’ll no longer get a parse error, but the underlying type won’t be correct). Also interesting to note that the
works, sort of – the underlying spec of Fully supporting this will take a bit longer, but in the meantime the syntax changes might be enough to run certain proofs on OSX again already. |
Also cbmc warns about the fact that in e.g. |
@danielsn We believe this is fixed on latest develop, therefore we’re closing this for now. |
CBMC version:
5.12 (cbmc-5.12.1)
Operating system:
Exact command line resulting in the issue:
clone https://github.com/awslabs/aws-c-common/tree/master/.cbmc-batch/jobs/aws_array_list_get_at_ptr
What behaviour did you expect:
CBMC to compile the file
What happened instead:
It failed with a
PARSING ERROR
Note that we can get this to run using
gcc
but fails withgoto-cc
. I had to remove a couple flags gcc didn't know about, but this seems to be a minimal test caseThe text was updated successfully, but these errors were encountered: