[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]

tyomitch added inline comments.


================
Comment at: lib/Target/ARM/ARMISelDAGToDAG.cpp:3299
+              Replacement[] = {Res, GlueOut};
+      Select(RHS.getNode());
+      CurDAG->ReplaceAllUsesWith(N, Replacement);
----------------
efriedma wrote:
> Do you actually need to call Select() explicitly here?  Instruction selection should pick it up automatically, I think.
No, it doesn't re-lower nodes created by `ARMDAGToDAGISel::Select()`: it is assumed to only output lowered nodes.


https://reviews.llvm.org/D30400