[PATCH] D17098: [AArch64] analyse dependences of ldp/stp

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 10 14:16:49 PST 2016


qcolombet added a subscriber: qcolombet.

================
Comment at: llvm/lib/Target/AArch64/AArch64InstrInfo.cpp:1446
@@ +1445,3 @@
+    Offset = LdSt->getOperand(2).getImm() * Scale;
+  } else if (LdSt->getNumOperands() == 4) {
+    BaseReg = LdSt->getOperand(2).getReg();
----------------
s/else if/else/
And move the condition of the if in an assert.

================
Comment at: llvm/test/CodeGen/AArch64/ldst-pairing.ll:3
@@ +2,3 @@
+
+; CHECK: stp
+; CHECK: stp
----------------
CHECK-LABEL:

+ Give a description of what is tested.

================
Comment at: llvm/test/CodeGen/AArch64/ldst-pairing.ll:20
@@ +19,3 @@
+
+; CHECK: stp
+; CHECK: stp
----------------
ditto

================
Comment at: llvm/test/CodeGen/AArch64/ldst-pairing.ll:48
@@ +47,3 @@
+
+; CHECK: stp
+; CHECK: stp
----------------
ditto


http://reviews.llvm.org/D17098





More information about the llvm-commits mailing list