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

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

I feel this is too specific and doesn't belong in the datalayout. I think a general maximum global alignment would make a bit more sense? I know very little about object formats



================
Comment at: llvm/test/CodeGen/X86/tls-align.ll:60
+  ret void
+}
----------------
You should be able to get a simpler patch out of any pass that calls getOrEnforceKnownAlignment (e.g. instcombine)


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

https://reviews.llvm.org/D140123