[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