[PATCH] D106051: [ARM] Extend more reductions during lowering

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 16 06:44:07 PDT 2021


dmgreen added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMISelLowering.cpp:16064
+    EVT AVT = A.getValueType();
+    for (MVT Ty : ExtTypes)
+      if (AVT.getVectorNumElements() == Ty.getVectorNumElements() &&
----------------
SjoerdMeijer wrote:
> Was curious if we can still us `llvm::any_of` for this as a bit of a shorter notation.
Yeah.. let me try that. I removed it as I thought the nested lambdas were getting ugly, but you might be alright that it's OK really.


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

https://reviews.llvm.org/D106051



More information about the llvm-commits mailing list