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

Luke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 5 09:32:11 PDT 2021


luke957 added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.h:151
+    case RecurKind::UMax:
+    case RecurKind::FMin:
+    case RecurKind::FMax:
----------------
frasercrmck wrote:
> We don't support the fmin/fmax reductions yet. I suspect this would crash in the backend?
The added test case contains a fmin case. For opt -loop-vectorize, it seems no crash.


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