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

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/Local.cpp:1425-1426
 
+    if (GO->isThreadLocal() && DL.exceedsMaxTLSAlign(PrefAlign))
+      PrefAlign = Align(DL.getMaxTLSAlign());
+
----------------
Why wouldn't this be in  GlobalObject::canIncreaseAlignment?


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

https://reviews.llvm.org/D140123