[PATCH] D97280: [AArch64] Extend vecreduce -> udot handling to mla reductions

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 4 01:02:36 PST 2021


SjoerdMeijer accepted this revision.
SjoerdMeijer added a comment.
This revision is now accepted and ready to land.

Looks okay to me.



================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:11756
+  if (ExtOpcode == ISD::MUL) {
+    A = Op0.getOperand(0);
+    B = Op0.getOperand(1);
----------------
Nit: can we not do:

  Op0 = Op0.getOperand(0);

to avoid creating As and Bs?


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

https://reviews.llvm.org/D97280



More information about the llvm-commits mailing list