[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