[PATCH] D121898: [Verifier] Verify parameter alignment.

LuoYuanke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 21 05:54:02 PDT 2022


LuoYuanke added a comment.

In D121898#3392260 <https://reviews.llvm.org/D121898#3392260>, @pengfei wrote:

> Can we check if the pointer has `byval/byref` attribute? I think the `MemAlign` should only be meaningful for them. And if we simply passing a pointer with `OrigAlign`, it's safe to truncate to the maximum of `MemAlign` in backend.

LGTM, thanks Phoebe. I create anther patch at https://reviews.llvm.org/D122130.


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