[PATCH] D130053: [WebAssembly] Use `localexec` as default TLS model for non-Emscripten targets

[Sam Clegg via Phabricator via llvm-commits]

sbc100 added a comment.

In D130053#3677260 <https://reviews.llvm.org/D130053#3677260>, @penzn wrote:

>> Because TLS data is stored a "passive" segment which has to be loaded using the `memory.init` instruction.  (Both passive segments and this instruction are part of the bulk-memory proposal).
>
> Thanks! Should something about this added to tool conventions? I understand why may be it wasn't - this is internal to how compiler can implement TLS, not "handshake" between tools.

I think this should be tooling-conventions somewhere yes.. I don't think we have anything there yet about TLS or threading.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130053