Skip to content
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

Add enumerator attributes #172

Merged
merged 5 commits into from
Aug 16, 2024
Merged

Add enumerator attributes #172

merged 5 commits into from
Aug 16, 2024

Conversation

sim642
Copy link
Member

@sim642 sim642 commented Aug 2, 2024

In opam-repository CI goblint.2.4.0 saw MacOS failures with

#   - /Library/Developer/CommandLineTools/SDKs/MacOSX14.sdk/usr/include/sys/qos.h[130:0-0] : syntax error
#   - Parsing errorFatal error: exception Goblint_lib__Maingoblint.FrontendError("Frontc.ParseError: Parse error")

This seems to be https://github.com/alexey-lysiuk/macos-sdk/blob/69ddec41e6224afebd5f19bb24764305ac3e828a/MacOSX14.5.sdk/usr/include/sys/qos.h#L130-L143, where attributes are after individual enum items (enumerators).

It is valid syntax for GCC attributes: https://gcc.gnu.org/onlinedocs/gcc/Enumerator-Attributes.html.

@sim642 sim642 added the bug label Aug 2, 2024
@sim642 sim642 self-assigned this Aug 2, 2024
@sim642 sim642 added this to the 2.0.5 milestone Aug 2, 2024
@sim642 sim642 marked this pull request as ready for review August 2, 2024 14:28
@sim642
Copy link
Member Author

sim642 commented Aug 13, 2024

Looks like GitHub Actions macos-13 machines may have also updated some SDKs because there are now the same CI failures there.

@sim642
Copy link
Member Author

sim642 commented Aug 16, 2024

Merging now since it fixes Goblint MacOS CI which otherwise fails for everything.

@sim642 sim642 merged commit 135fc3c into develop Aug 16, 2024
56 checks passed
@sim642 sim642 deleted the attr-enumerator branch August 16, 2024 12:45
@sim642
Copy link
Member Author

sim642 commented Sep 6, 2024

This also appears on Linux in gspawn.h: goblint/analyzer#1564 (comment).

sim642 added a commit to sim642/opam-repository that referenced this pull request Nov 25, 2024
CHANGES:

* Add enumerator attributes (goblint/cil#172).
* Generate 32bit and 64bit `Machdep` if possible (goblint/cil#173).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant