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

More MacOS fixes #9

Merged
merged 4 commits into from
Dec 21, 2024
Merged

Conversation

alastairreid
Copy link

These commits fix more of the issues using ASLi on MacOS

  • C++ backends not working because they need to specify a C++ standard version
  • Installing cmake because it is needed by google test
  • Upgrading google test submodule because cmake was warning about using old make files

This does not fix all the issues yet

  • Having trouble using lit/filecheck to check an error message (because it is on stderr)
  • Hardcoded clang version in the demo
  • The c23 backend does not work on MacOS because the maximum size value is 128 bits. (Oddly, this is not an issue on other platforms)
  • Issue with the %decolor substitution used with lit

These affect our ability to run tests

This is required to make googletest build.
The particular standard was chosen to be old enough that
most systems will have no problem - but it may be possible
to go back further.

Signed-off-by: Alastair Reid <alastair.d.reid@gmail.com>
This resolves a warning about cmake versions.
I chose not to update to 1.15.x because those seem to need
recent C++ features and we don't want or need to be
on the bleeding edge here.

Signed-off-by: Alastair Reid <alastair.d.reid@gmail.com>
Signed-off-by: Alastair Reid <alastair.d.reid@gmail.com>
Signed-off-by: Alastair Reid <alastair.d.reid@gmail.com>
@alastairreid alastairreid merged commit 7d8877a into IntelLabs:master Dec 21, 2024
1 check passed
@alastairreid alastairreid deleted the areid-macos-fixes2 branch December 21, 2024 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant