[PATCH] D154943: [ARM] Restructure MOVi32imm expansion to not do pointless instructions

John Brawn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 12 08:55:27 PDT 2023


john.brawn added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMExpandPseudoInsts.cpp:1049
+              .setMIFlags(MIFlags);
+      LLVM_DEBUG(dbgs() << "And:       "; Lsl->dump(););
+      PendingShift = 0;
----------------
chapuni wrote:
> `Lsl` is used only here in -Asserts.
Fixed in https://reviews.llvm.org/rG25fccfd41f47


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154943



More information about the llvm-commits mailing list