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

Kan Shengchen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 22 19:18:27 PDT 2022


skan added a comment.

In D121898#3401216 <https://reviews.llvm.org/D121898#3401216>, @LuoYuanke wrote:

> In D121898#3399677 <https://reviews.llvm.org/D121898#3399677>, @nikic wrote:
>
>> @LuoYuanke Not sure if this is documented anywhere, but there is this check in the DataLayout implementation: https://github.com/llvm/llvm-project/blob/fea20cb99087208d589d81bb7bbbc84198c7ffa4/llvm/lib/IR/DataLayout.cpp#L250-L256
>
> Thanks, @nikic. I'll come up with a patch to enlarge the alignment bit field.

I would argue that maybe we should shrink the alignment bit field...


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