[PATCH] D30044: [ARM] Enable SMLAL[B|T] instruction selection

Tim Northover via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 17 14:32:07 PST 2017


t.p.northover added inline comments.


================
Comment at: lib/Target/ARM/ARMISelLowering.cpp:9476
+  }
+  if (SRA.getOperand(0) != Mul)
+    return SDValue();
----------------
samparker wrote:
> t.p.northover wrote:
> > What if it's not the same mul as before?
> Sorry I don't get your point, what I am missing?
Just knowing that the input to the SRA is **some** multiply operation isn't sufficient. You need to make sure it's the same one that produced the low bits.


https://reviews.llvm.org/D30044





More information about the llvm-commits mailing list