[PATCH] D59317: [ARM] Search backwards for CMP when combining into CBZ

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 14 03:49:43 PDT 2019


samparker added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMConstantIslandPass.cpp:1931
+    if (Pred != ARMCC::AL || CmpMI->getOperand(1).getImm() != 0 ||
+        !isARMLowRegister(Reg))
+      continue;
----------------
Do we need to still check this? I'm assuming we can't have a tCMPi8 with a high reg?


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

https://reviews.llvm.org/D59317





More information about the llvm-commits mailing list