[PATCH] D135817: [AArch64] Use encodeLogicalImmediate for forming the immediate to an AND. NFC.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 12 15:11:24 PDT 2022


efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/lib/Target/AArch64/AArch64FrameLowering.cpp:1817
     if (NeedsRealignment) {
-      const unsigned NrBitsToZero = Log2(MFI.getMaxAlign());
-      assert(NrBitsToZero > 1);
+      assert(Log2(MFI.getMaxAlign()) > 1);
       assert(scratchSPReg != AArch64::SP);
----------------
`assert(MFI.getMaxAlign() > Align(1))`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135817



More information about the llvm-commits mailing list