[PATCH] D155929: [RISCV] Use the first element of source as the start value of reduction.

[Luke Lau via Phabricator via llvm-commits]

luke added a comment.

I can't speak for the fmin/fmax parts, but for the integer ops this makes sense to me



================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:7610-7611
+// Function to extract the first element of Vec. For fixed vector Vec, this
+// converts it to scalabe vector before extraction. It could let the following
+// optimizations not have to serve fixed vector.
+static SDValue getFirstElement(SDValue Vec, SelectionDAG &DAG,
----------------
What happens it it's not converted to a scalable vector?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155929