[PATCH] D122563: [RISCV] Add DAGCombine to fold base operation and reduction.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:8454
+  case ISD::FADD:
+    return performFADDCombine(N, DAG);
+  case ISD::UMAX:
----------------
craig.topper wrote:
> Can we just lump all these together calling combineBinOpToReduce directly until we need specific combines for them?
This wasn't addressed. I don't think we want individual performUMAXCombine, performUMINCombine, etc. functions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122563