[PATCH] D116300: [OMPIRBuilder] Fix store inst alignment for ordered depend directive

Peixin Qiao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 5 05:47:00 PST 2022


peixin added a comment.

In D116300#3221883 <https://reviews.llvm.org/D116300#3221883>, @shraiysh wrote:

> Thanks Kiran. The i64 vec change LGTM (I still didn't understand why the alignment has to be 8).

Actually, the 4-byte alignment can also give correct running result. But the value is always stored with the type of integer 64, and the 8-byte alignment is more reasonable.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116300



More information about the llvm-commits mailing list