[PATCH] D29564: [DAGCombiner] (add X, (adde Y, 0, Carry)) -> (adde X, Y, Carry)
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Feb 5 15:41:10 PST 2017
- Previous message: [PATCH] D29564: [DAGCombiner] (add X, (adde Y, 0, Carry)) -> (adde X, Y, Carry)
- Next message: [PATCH] D29564: [DAGCombiner] (add X, (adde Y, 0, Carry)) -> (adde X, Y, Carry)
- Messages sorted by:
[ date ]
[ thread ]
[ subject ]
[ author ]
RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.
LGTM with one style minor
================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:1840
+ N1->hasOneUse() &&
+ isNullConstant(N1.getOperand(1)))
+ return DAG.getNode(ISD::ADDE, DL, DAG.getVTList(VT, MVT::Glue),
----------------
clang-format this
https://reviews.llvm.org/D29564
- Previous message: [PATCH] D29564: [DAGCombiner] (add X, (adde Y, 0, Carry)) -> (adde X, Y, Carry)
- Next message: [PATCH] D29564: [DAGCombiner] (add X, (adde Y, 0, Carry)) -> (adde X, Y, Carry)
- Messages sorted by:
[ date ]
[ thread ]
[ subject ]
[ author ]
More information about the llvm-commits
mailing list