[PATCH] D140123: [TLS] Clamp the alignment of TLS global variables if required by the target

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 10 08:14:24 PST 2023


probinson added inline comments.


================
Comment at: clang/test/CodeGen/tls-maxalign-modflag.c:11-12
+
+// CHECK: llvm.module.flags
+// CHECK: !{i32 1, !"MaxTLSAlign", i32 256}
----------------
I'd feel better if these two lines were tied together. Maybe no practical difference but I think it makes the intent clearer.


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

https://reviews.llvm.org/D140123



More information about the llvm-commits mailing list