[PATCH] D22933: DAG: avoid truncating a sign extended operand when test equality against zero

Eli Friedman via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 28 17:16:47 PDT 2016


eli.friedman added a subscriber: eli.friedman.

================
Comment at: lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp:964
@@ -955,3 +963,3 @@
         OpR->getOpcode() == ISD::AssertSext &&
         cast<VTSDNode>(OpR->getOperand(1))->getVT() == NewRHS.getValueType()) {
       NewLHS = OpL;
----------------
Could you just use `OpL->ComputeNumSignBits() > ExtensionBits && OpR->ComputeNumSignBits() > ExtensionBits` or something like that here?


Repository:
  rL LLVM

https://reviews.llvm.org/D22933





More information about the llvm-commits mailing list