[llvm] r318010 - [X86] Attempt to fix signed and unsigned comparison warning.

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 12 18:19:13 PST 2017


Author: ctopper
Date: Sun Nov 12 18:19:13 2017
New Revision: 318010

URL: http://llvm.org/viewvc/llvm-project?rev=318010&view=rev
Log:
[X86] Attempt to fix signed and unsigned comparison warning.

Modified:
    llvm/trunk/lib/Target/X86/X86ISelLowering.cpp

Modified: llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86ISelLowering.cpp?rev=318010&r1=318009&r2=318010&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86ISelLowering.cpp Sun Nov 12 18:19:13 2017
@@ -19771,7 +19771,7 @@ SDValue X86TargetLowering::LowerINTRINSI
       // (1) With suppress-all-exceptions (sae) or rounding mode- 6 operands
       // (2) With rounding mode and sae - 7 operands.
       bool HasRounding = IntrWithRoundingModeOpcode != 0;
-      if (Op.getNumOperands() == (5 + HasRounding)) {
+      if (Op.getNumOperands() == (5U + HasRounding)) {
         if (HasRounding) {
           SDValue Rnd = Op.getOperand(5);
           if (!isRoundModeCurDirection(Rnd))
@@ -19784,7 +19784,7 @@ SDValue X86TargetLowering::LowerINTRINSI
                                     Mask, passThru, Subtarget, DAG);
       }
 
-      assert(Op.getNumOperands() == (6 + HasRounding) &&
+      assert(Op.getNumOperands() == (6U + HasRounding) &&
              "Unexpected intrinsic form");
       SDValue RoundingMode = Op.getOperand(5);
       if (HasRounding) {




More information about the llvm-commits mailing list