[PATCH] D35192: [ARM] Use ADDCARRY / SUBCARRY

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 27 11:37:12 PDT 2017


efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.

LGTM, with minor comments.

Please make sure to update the commit message to account for the changes during the review.



================
Comment at: lib/Target/ARM/ARMISelLowering.cpp:7477
+      Result = DAG.getNode(ARMISD::SUBE, DL, VTs, Op.getOperand(0),
+              Op.getOperand(1), Carry.getValue(1));
+
----------------
Weird indentation (maybe try clang-format?)


https://reviews.llvm.org/D35192





More information about the llvm-commits mailing list