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

[Sanjay Patel via Phabricator via llvm-commits]

spatel added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:1200
+      TLI.shouldReassociateReduction(RedOpc, N0.getOperand(0).getValueType()))
+    return DAG.getNode(RedOpc, DL, VT,
+                       DAG.getNode(Opc, DL, N0.getOperand(0).getValueType(),
----------------
Flags param is not used?
They could propagate with:
  SelectionDAG::FlagInserter FlagsInserter(DAG, Flags);



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

https://reviews.llvm.org/D141870