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

Andrew Brown via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 22 13:27:47 PDT 2022


abrown updated this revision to Diff 446963.
abrown added a comment.

Update to fix tests, clang-format

This change used `git clang-format origin/main -- llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp` to properly format that code and adds some `sed` substitution in `tls-local-exec.ll` to check both the "specify `localexec` explicitly" and "emit `localexec` even under the `generaldynamic` default" cases. I believe this should also cover the cases removed from `tls-general-dynamic.ll`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130053

Files:
  llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp
  llvm/test/CodeGen/WebAssembly/tls-general-dynamic.ll
  llvm/test/CodeGen/WebAssembly/tls-local-exec.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D130053.446963.patch
Type: text/x-patch
Size: 7001 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220722/d647c68e/attachment.bin>


More information about the llvm-commits mailing list