[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