[PATCH] D30400: For Thumb1, lower ADDC/ADDE/SUBC/SUBE via the glueless ARMISD nodes, same as already done for ARM and Thumb2.

A. Skrobov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 28 06:36:53 PST 2017


tyomitch added inline comments.


================
Comment at: lib/Target/ARM/ARMISelDAGToDAG.cpp:3240
+  case ARMISD::SUBE:
+    if (Subtarget->hasV6Ops()) {
+
----------------
rengolin wrote:
> Why can't you leave this as an early break?
Exactly because I want it to fall through to the next case, if the condition doesn't hold.


https://reviews.llvm.org/D30400





More information about the llvm-commits mailing list