[PATCH] D116804: [x86] use SETCC_CARRY instead of SBB node for select lowering

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 7 08:24:56 PST 2022


craig.topper added a comment.

> why does it require a parameter for X86::COND_B? I changed that parameter in an experimental patch, and it didn't alter the asm.

I believe there is some code in lowering that uses it. I think it made it easier to handle it along with X86ISD::SETCC if it has a condition code operand in the same place.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116804/new/

https://reviews.llvm.org/D116804



More information about the llvm-commits mailing list