[PATCH] D130492: [AArch64][SVE] Add patterns to select mla/mls

[Cullen Rhodes via Phabricator via llvm-commits]

c-rhodes marked an inline comment as done.
c-rhodes added inline comments.


================
Comment at: llvm/lib/Target/AArch64/SVEInstrFormats.td:2982
+
+  // e.g. add(a, select(mask, mul(b, b), splat(0))) -> mla(a, mask, b, b)
+  def : Pat<(outerop nxv16i8:$Op1, (vselect nxv16i1:$pred, (mulop (nxv16i1 (AArch64ptrue 31)), nxv16i8:$Op2, nxv16i8:$Op2), (SVEDup0))),
----------------
paulwalker-arm wrote:
> What's the connection to `b*b`? What breaks if you allow `b*c`?
> What's the connection to `b*b`? What breaks if you allow `b*c`?

Nothing, I'm just a donut. Updated patch to match `b*c`.


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

https://reviews.llvm.org/D130492