[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
Fri Dec 16 13:35:12 PST 2022


probinson added a comment.

In D140123#4000196 <https://reviews.llvm.org/D140123#4000196>, @arsenm wrote:

> 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

Our platform is a little bit unusual in that the maximum TLS alignment is not the same as the maximum alignment for globals. We'd rather not constrain alignments any more than we have to.


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

https://reviews.llvm.org/D140123



More information about the llvm-commits mailing list