[llvm] [RISCV] Move vnclip patterns into DAGCombiner. (PR #93728)

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Wed May 29 16:24:56 PDT 2024


topperc wrote:

> LGTM
> 
> Aside - This is starting to look a bit like isSaturatingMinMax in DAGCombine. This version handles the _VL variants, and that one handles the select variants, so neither is a subset of the other. Do we have any room to optimize the saturating fp_to_sint patterns the DAG is working on? I don't see anything but figured it's worth asking.. Alternatively, is there anything we can do to optimize/recognize the clamp idiom on it's own?

Looks like there are some vnclips in test/CodeGen/RISCV/rvv/fpclamptosat_vec.ll but also some min/max/vnsrl.

https://github.com/llvm/llvm-project/pull/93728


More information about the llvm-commits mailing list