[PATCH] D153356: [Align] Add isAligned taking an APInt
Guillaume Chatelet via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 26 06:44:07 PDT 2023
gchatelet added a comment.
In D153356#4444104 <https://reviews.llvm.org/D153356#4444104>, @nikic wrote:
> Does isAligned() need to be defined in the header? If it's in the source file then we just need the forward declaration.
I never thought it would work to forward declare and pass by value even if the implementation is in the source file. Thx for the suggestion!
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