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

-mtp=cp15 generates __aeabi_read_tp in Thumb2 mode #51671

Closed
nathanchance opened this issue Oct 27, 2021 · 5 comments
Closed

-mtp=cp15 generates __aeabi_read_tp in Thumb2 mode #51671

nathanchance opened this issue Oct 27, 2021 · 5 comments
Assignees
Labels
backend:ARM bugzilla Issues migrated from bugzilla

Comments

@nathanchance
Copy link
Member

Bugzilla Link 52329
Resolution FIXED
Resolved on Nov 08, 2021 08:41
Version trunk
OS Windows NT
Blocks #4440 #51489
CC @arndb,@nickdesaulniers,@smithp35,@tstellar
Fixed by commit(s) d7e089f ed38280

Extended Description

This impacts the Linux kernel: ClangBuiltLinux/linux#1485

Reproducer: https://godbolt.org/z/aejoTqo4n

Fix: https://reviews.llvm.org/D112600

Opening this to track for LLVM 13.0.1.

@nathanchance
Copy link
Member Author

assigned to @smithp35

@nathanchance
Copy link
Member Author

Fixed: d7e089f

Leaving this open for LLVM 13.0.1 (as the kernel is expecting it to be picked there: https://lore.kernel.org/r/20211028083527.1057158-1-ardb@kernel.org/

@tstellar
Copy link
Collaborator

tstellar commented Nov 2, 2021

Hi Peter,

What is your opinion on backporting this?

https://reviews.llvm.org/rGd7e089f2d6a5cd5f283a90ab29241d20d4fc3ed1

@smithp35
Copy link
Collaborator

smithp35 commented Nov 2, 2021

I think it should be fine to backport. The fix is straightfoward and the functionality is only enabled when the option is used so the risk is low.

@tstellar
Copy link
Collaborator

tstellar commented Nov 8, 2021

Merged: ed38280

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 11, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend:ARM bugzilla Issues migrated from bugzilla
Projects
None yet
Development

No branches or pull requests

3 participants