[llvm] r229488 - [X86] Silence -Wsign-compare warnings.

Andrea Di Biagio Andrea_DiBiagio at sn.scee.net
Tue Feb 17 03:20:11 PST 2015


Author: adibiagio
Date: Tue Feb 17 05:20:11 2015
New Revision: 229488

URL: http://llvm.org/viewvc/llvm-project?rev=229488&view=rev
Log:
[X86] Silence -Wsign-compare warnings.

GCC 4.8 reported two new warnings due to comparisons
between signed and unsigned integer expressions. The new warnings were
accidentally introduced by revision 229480.
Added explicit casts to silence the warnings. No functional change intended.

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=229488&r1=229487&r2=229488&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86ISelLowering.cpp Tue Feb 17 05:20:11 2015
@@ -24946,7 +24946,7 @@ static SDValue VectorZextCombine(SDNode
   unsigned ResSize = N1.getValueType().getScalarSizeInBits();
   // Make sure the splat matches the mask we expect
   if (SplatBitSize > ResSize || 
-      (SplatValue + 1).exactLogBase2() != SrcSize)
+      (SplatValue + 1).exactLogBase2() != (int)SrcSize)
     return SDValue();
 
   // Make sure the input and output size make sense
@@ -24966,7 +24966,7 @@ static SDValue VectorZextCombine(SDNode
         break;
       }
     } else {
-      if (Shuffle->getMaskElt(i) != (i / ZextRatio)) {
+      if (Shuffle->getMaskElt(i) != (int)(i / ZextRatio)) {
         // Expected element number
         IsZext = false;
         break;





More information about the llvm-commits mailing list