[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