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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D129833: Use @llvm.th... Nikita Popov via Phabricator via cfe-commits

Reply via email to