[PATCH] D105680: [ARM] Lower v16i8 -> i64 VMLA reductions.
Sjoerd Meijer via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 9 02:21:24 PDT 2021
SjoerdMeijer added inline comments.
================
Comment at: llvm/lib/Target/ARM/ARMISelLowering.cpp:16051
auto Create64bitNode = [&](unsigned Opcode, ArrayRef<SDValue> Ops) {
+ // Split illegal MVT::v16i8->i64 vector reductions into two legal v8i16->i64
+ // reductions. The operands are extended with MVEEXT, but as they are
----------------
Perhaps a silly question on the use of 'illegal'. Type `v16i8` isn't an illegal type, it is just not supported for these VMLALV instructions. Thus, I was wondering if 'unsupported' would be better, to avoid possible confusion with type legalization in general if that makes sense?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105680/new/
https://reviews.llvm.org/D105680
More information about the llvm-commits
mailing list