[PATCH] D141870: [DAG] Fold Op(vecreduce(a), vecreduce(b)) into vecreduce(Op(a,b))

[Dave Green via Phabricator via llvm-commits]

dmgreen added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:5521
 
+  // Fold and(vecreduce(x), vecreduce(y)) -> vecreduce(and(x, y))
+  auto ReductionOpcode = [](unsigned Opcode) {
----------------
dzhidzhoev wrote:
> Should here be "min/max/...(vecreduce..."?
Yep! Thanks. Updated in 14914fb1573f0393979492238735beecde65b3bb.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141870