[PATCH] D33374: Do not legalize large setcc with setcce, introduce setcccarry and do it with usubo/setcccarry.

[Amaury SECHET via Phabricator via llvm-commits]

deadalnix added inline comments.


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:34510
 
-    if (Carry.getOpcode() == ISD::SETCC ||
-        Carry.getOpcode() == X86ISD::SETCC ||
+    if (Carry.getOpcode() == X86ISD::SETCC ||
         Carry.getOpcode() == X86ISD::SETCC_CARRY) {
----------------
RKSimon wrote:
> I'm curious why you're dropping ISD::SETCC support here - is the change independent of the rest of the patch?
It never worked and is invalid. It could indeed be part of another diff.


https://reviews.llvm.org/D33374