[PATCH] D143708: [RISCV] Add emulated TLS supported
Jessica Clarke via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 16 13:46:09 PST 2023
jrtc27 added a comment.
In D143708#4133064 <https://reviews.llvm.org/D143708#4133064>, @MaskRay wrote:
> In D143708#4132222 <https://reviews.llvm.org/D143708#4132222>, @asb wrote:
>
>> The consensus from the RISC-V LLVM sync-up call was that it makes sense to go ahead with this on the basis their are users, the infrastructure in LLVM already exists to use it etc, and it's a tiny patch to enable etc.
>>
>> In which case, assuming no-one who wasn't able to make it to the call wanted to raise strong objections, we can hopefully focus on code review of this patch and move it forwards.
>
> I have a mild objection but I am fine if my opinion is overthrown. I think OpenBSD developers should take on the work to migrate to ELF TLS, not using this definitely.
>
> `-femulated-tls` can be used for the current `*-linux-android*` and `*-openbsd*` target triples, and OpenBSD RISC-V, but not others. The test should be changed to use openbsd, not linux-gnu.
Or just bare metal.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D143708/new/
https://reviews.llvm.org/D143708
More information about the llvm-commits
mailing list