[PATCH] D121898: [Verifier] Verify parameter alignment.
Phoebe Wang via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 17 23:12:45 PDT 2022
pengfei accepted this revision.
pengfei added a comment.
This revision is now accepted and ready to land.
In D121898#3391242 <https://reviews.llvm.org/D121898#3391242>, @skan wrote:
> Is it possible that we define a common max alignment across all layers?
I think arbitrary/maximum alignment doesn't make sense in reality and usually implies something wrong. So adding a verifier seems a good choice. LGTM but wait one or two days for other reviewers.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121898/new/
https://reviews.llvm.org/D121898
More information about the llvm-commits
mailing list