[PATCH] D75533: [ARM][LowOverheadLoops] Handle reductions

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 1 00:29:46 PDT 2020


samparker marked an inline comment as done.
samparker added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMLowOverheadLoops.cpp:693
+  case ARM::MVE_VCTP8:
+    if (Reduce->getOpcode() != ARM::MVE_VADDi8)
+      return false;
----------------
SjoerdMeijer wrote:
> Do we have a (negative) test case with a float reduction?
The vectorizer doesn't seem to want to produce vector float reductions loops, so I've left them out on testing.


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

https://reviews.llvm.org/D75533





More information about the llvm-commits mailing list