[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