[PATCH] D99509: [RISCV] Add legality check for vectoring redunction

Fraser Cormack via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 29 08:21:01 PDT 2021


frasercrmck added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.h:132
+                                   ElementCount VF) const {
+    if (!VF.isScalable())
+      return true;
----------------
We support the fixed-length vector reductions. Can we add support for that too?


================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.h:151
+    case RecurKind::UMax:
+    case RecurKind::FMin:
+    case RecurKind::FMax:
----------------
We don't support the fmin/fmax reductions yet. I suspect this would crash in the backend?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99509



More information about the llvm-commits mailing list