[PATCH] D129833: Use @llvm.threadlocal.address intrinsic to access TLS

[Nikita Popov via Phabricator via cfe-commits]

nikic added a comment.

In D129833#3727881 <https://reviews.llvm.org/D129833#3727881>, @ChuanqiXu wrote:

> And I am working on adding Align properties. But I meet problems since the alignment of threadlocal_address intrinsic depends on its argument so we can't set the alignment for its declaration and we probably need to set the alignment for its call, which means we need to set the alignment when in IRBuilder. Do you think this is good?

I think that would be fine. Alternatively, it could be inferred in InstCombine.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129833