[PATCH] D65177: [WebAssembly] Set __tls_align to 1 when there is no TLS

[Thomas Lively via Phabricator via llvm-commits]

tlively accepted this revision.
tlively added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lld/test/wasm/no-tls.test:1
+RUN: llc -mattr=+bulk-memory -filetype=obj %p/Inputs/start.ll -o %t.o
+
----------------
Why not have the input in this file?


================
Comment at: lld/wasm/Driver.cpp:532
     WasmSym::tlsBase = createGlobalVariable("__tls_base", true);
     WasmSym::tlsSize = createGlobalVariable("__tls_size", false);
+    WasmSym::tlsAlign = createGlobalVariable("__tls_align", false, 1);
----------------
May as well update these calls to explicitly pass zero as well, and make it nonoptional. I assume there are no other calls to this function elsewhere?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65177