[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