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.

Sign up for GitHub

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

Jump to bottom

M1 LLVM toolchain #1544

Closed
Tracked by #1299
DanAlbert opened this issue Jul 26, 2021 · 2 comments
Closed
Tracked by #1299

M1 LLVM toolchain #1544

DanAlbert opened this issue Jul 26, 2021 · 2 comments
Assignees
@DanAlbert
Labels
enhancement

Comments

@DanAlbert
Copy link
Member

Subtask for #1299

clang-r416183c already has this, so we just need to update from clang-r416183b.
@DanAlbert DanAlbert mentioned this issue Jul 26, 2021
Closed
9 tasks
@DanAlbert DanAlbert self-assigned this Aug 31, 2021
@DanAlbert
Copy link
Member Author

https://android-review.googlesource.com/c/platform/ndk/+/1815336/ does this for r24. Will cherry-pick for r23b as well

chenguoyin-alibaba pushed a commit to riscv-android-src/platform-ndk that referenced this issue Oct 21, 2021
@DanAlbert
Update LLVM to r416183c for M1 support.
f949a6c 
There's an even newer LLVM that we should be using for r24, but for
now I'm staying on the current revision so we can keep r24 and r23
from diverging too much before we need to so testing is easier.

Test: ./checkbuild.py && ./run_tests.py
Bug: android/ndk#1544
Change-Id: I4d0cfb4dc4df6e7a3f42e7487de87ca43e56f379
(cherry picked from commit e3f5e9f)
@rprichard
Copy link
Collaborator

This is fixed in r23b and in r24-beta1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DanAlbert DanAlbert

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@DanAlbert @rprichard