[PATCH] D74750: [X86] Rewrite LowerBRCOND to remove dead code and handle ISD::SETCC and overflow ops directly.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 18 10:07:44 PST 2020


RKSimon added inline comments.


================
Comment at: llvm/lib/Target/X86/X86ISelLowering.cpp:23142
+           Opc == ISD::USUBO || Opc == ISD::SMULO || Opc == ISD::UMULO));
 }
 
----------------
craig.topper wrote:
> RKSimon wrote:
> > Maybe put this in ISDOpcodes.h ?
> The dependency on SDValue probably means it needs to go in SelectionDAGNodes.h?
Sorry, I didn't notice the getResNo..........


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74750





More information about the llvm-commits mailing list