[PATCH] D118414: [lld][WebAssembly] Handle TLS symbols in older object file

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 27 16:31:52 PST 2022


sbc100 added a comment.

In D118414#3277878 <https://reviews.llvm.org/D118414#3277878>, @dschuff wrote:

> I guess the existing tls.s test case still covers the case where neither the symbol nor the segment is marked as TLS but the name is `.tdata`?

Yes I believe that is covered here:

https://github.com/llvm/llvm-project/blob/bddc814b442ae9f30d62e2f881274d6255411225/lld/test/wasm/tls.s#L54-L61


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D118414/new/

https://reviews.llvm.org/D118414



More information about the llvm-commits mailing list