[PATCH] D109426: [WebAssembly] Add explict TLS symbol flag
Sam Clegg via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 9 01:10:15 PDT 2021
sbc100 added a comment.
In D109426#2990483 <https://reviews.llvm.org/D109426#2990483>, @tlively wrote:
> I have to say that I don't quite understand all the MCStreamer stuff, especially what fixSymbolsInTLSFixups is for, but the code looks good to me. Do we have a plan for when we will be able to remove the name-based implicit TLS?
I should have mentioned that this code is lifted fairly directly from MCELFStreamer.cpp, at least the pattern:
https://github.com/llvm/llvm-project/blob/9d4896f50e441ea5b9e8ae78ebe328e006cb6b67/llvm/lib/MC/MCELFStreamer.cpp#L395
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109426/new/
https://reviews.llvm.org/D109426
More information about the llvm-commits
mailing list