[PATCH] D57317: [DAGCombine] Deduplicate addcarry node using commutativity.

[Amaury SECHET via Phabricator via llvm-commits]

deadalnix marked an inline comment as done.
deadalnix added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:2527
+  if (CSENode)
+    return SDValue(CSENode, 0);
+
----------------
RKSimon wrote:
> We already do this in DAGCombiner::combine for plain commutative binops, is it worth generalizing that code?
The problem is that this is not a binary operation so it doesn't fit in the existing machinery. If this end up being common enough, we can generalize.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57317