[PATCH] D120013: [WebAssembly] Make EH/SjLj vars unconditionally thread local

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 17 06:46:13 PST 2022


sbc100 added a comment.

Can you verify that the `core2` and `other` emscripten test suites both pass with this change



================
Comment at: llvm/lib/CodeGen/WasmEHPrepare.cpp:221
   LPadContextGV = cast<GlobalVariable>(
       M.getOrInsertGlobal("__wasm_lpad_context", LPadContextTy));
+  LPadContextGV->setThreadLocalMode(GlobalValue::GeneralDynamicTLSModel);
----------------
I was expecting to see `LPadContextGV->setThreadLocalMode(GlobalValue::GeneralDynamicTLSModel);` here... 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120013



More information about the llvm-commits mailing list