[PATCH] D19985: [Kryo] Use immediate #0 to zero a register

[Matthias Braun via llvm-commits]

MatzeB added inline comments.

================
Comment at: lib/Target/AArch64/AArch64InstrInfo.cpp:629-632
@@ -623,1 +628,6 @@
+
+  case AArch64::COPY:
+    return (Subtarget.getProcFamily() == AArch64Subtarget::Kryo &&
+            (MI.getOperand(1).getReg() == AArch64::WZR ||
+             MI.getOperand(1).getReg() == AArch64::XZR));
   }
----------------
Shouldn't this rather be "case AArch64::COPY: return true"?

The whole purpose of this function is that some passes can decide whether to copy a value around with COPY or just recompute the value instead.


Repository:
  rL LLVM

http://reviews.llvm.org/D19985