[PATCH] D153356: [Align] Add isAligned taking an APInt

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 22 04:46:37 PDT 2023


arsenm added a comment.

In D153356#4440752 <https://reviews.llvm.org/D153356#4440752>, @gchatelet wrote:

> In D153356#4439433 <https://reviews.llvm.org/D153356#4439433>, @arsenm wrote:
>
>> Could also add isAligned to APInt?
>
> I believe @nikic is opposed to this as well as it would also add a dependency.

There's no dependency if you just use the raw value instead of the Align wrapper. Also could forward declare Align?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D153356/new/

https://reviews.llvm.org/D153356



More information about the llvm-commits mailing list