-
Notifications
You must be signed in to change notification settings - Fork 14
add {pre,post}_prepare_hook
for LLVM to solve A64FX build issue by changing optarch
#97
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
{pre,post}prepare_hook
for LLVM to solve A64FX build issue by changing optarch
{pre,post}_prepare_hook
for LLVM to solve A64FX build issue by changing optarch
Tested interactively with
So, should be good to go. |
Since the same workaround should also solve the Rust 1.65.0 issue, I've applied it for that application as well, and removed the hook added by #86. We should be able to do a rebuild of Rust 1.65.0 now. |
Added easystacks for both LLVM 15 and (a rebuild of) Rust 1.65.0, so we can try the hook here. If it works, we could even deploy them from here, remove the easystacks from the PR, and open a PR in software-layer with the same easystacks (CI should pass right away). |
bot: build repo:eessi.io-2023.06-software instance:eessi-bot-deucalion for:arch=aarch64/a64fx |
New job on instance
|
bot: build repo:eessi.io-2023.06-software instance:eessi-bot-deucalion for:arch=aarch64/a64fx |
New job on instance
|
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.
LGTM, we need to build the hooks for both EESSI versions,so will need to open a PR in the software-layer
repo for the easystack files
bot: build repo:eessi.io-2023.06-software instance:eessi-bot-deucalion for:arch=aarch64/a64fx |
New job on instance
|
New job on instance
|
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.
LGTM
Still doing an interactive test build with this file to test it, please don't deploy/merge it yet.