[PATCH] D66295: [ARM] Sink add/mul(shufflevector(insertelement(...), ...), ...) for MVE instruction selection

[Sam Tebbs via Phabricator via llvm-commits]

samtebbs marked 2 inline comments as done.
samtebbs added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMISelLowering.cpp:14263
+      Instruction *Shuffle = dyn_cast<Instruction>(I->getOperand(Op));
+      if (Shuffle) {
+        if (match(I->getOperand(Op),
----------------
dmgreen wrote:
> Do we need to check that this is an instruction, or will the match handle that for us?
We don't in fact, will remove.


================
Comment at: llvm/lib/Target/ARM/ARMISelLowering.cpp:14268
+                      m_Undef(), m_Zero()))) {
+          Instruction *ShuffleOp1 =
+              dyn_cast<Instruction>(Shuffle->getOperand(0));
----------------
dmgreen wrote:
> ShuffleOp1 isn't needed?
It isn't :)


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

https://reviews.llvm.org/D66295