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

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 27 14:17:04 PST 2017


efriedma added inline comments.


================
Comment at: lib/Target/ARM/ARMISelDAGToDAG.cpp:3306
+        else {
+          assert(imm < 256);
+          Opc = isAdd ? ARM::tADDi8 : ARM::tSUBi8;
----------------
This assertion seems suspicious... why is it true in general?


https://reviews.llvm.org/D30400





More information about the llvm-commits mailing list