[PATCH] D109963: [AArch64] Split bitmask immediate of bitwise AND operation

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 15 13:46:05 PDT 2021


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64MIPeepholeOpt.cpp:129
+  // Check whether AND's operand is MOV with immediate.
+  MachineInstr *MovMI = MRI->getUniqueVRegDef(MI.getOperand(2).getReg());
+  MachineInstr *SubregToRegMI = nullptr;
----------------
danielkiss wrote:
> With the chromium codebase we found getUniqueVRegDef returns null as it could normally.
> I'd add a check for that.
Since this is running on SSA form, that should be getVRegDef. Is returning null because it is a physical register or for some other reason?


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

https://reviews.llvm.org/D109963



More information about the llvm-commits mailing list