-
Notifications
You must be signed in to change notification settings - Fork 76
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
Bumping isa-l to v2.31.0 #7967
Bumping isa-l to v2.31.0 #7967
Conversation
15ef8b3
to
53db327
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.
should update versions.md
@liranmauda does this build successfully on mac arm64 now? |
I have managed to build it on my mac (M1) |
53db327
to
1a5379e
Compare
1a5379e
to
215c505
Compare
@liranmauda It fails on mac with node for arm64. We need to add a condition to the mac section to handle node_arch=x64 vs arm64 like we do for linux. In the isa-l readme and makefile it shows that on arm the toolchain to use is gcc/gas. If you want we can merge this anyway to start running regression with the updated code, and fix the mac arm build in next PR. |
@guymguym I agree, probably we need to do the same as in the Linux. I will rebase, and when it is approved I will merge it. |
215c505
to
e6a917e
Compare
Bumping isa-l to v2.31.0 Signed-off-by: liranmauda <liran.mauda@gmail.com>
e6a917e
to
8a472c6
Compare
Explain the changes
Bumping isa-l to v2.31.0
Issues: Fixed #xxx / Gap #xxx