From: Simon Pilgrim <llvm-dev@redking.me.uk>
Date: Sun, 26 Mar 2017 11:17:05 +0000 (+0000)
Subject: Fix MSVC signed/unsigned comparison warnings.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=29226b40633bdc058f6bdcb28fadb896adb9662d;p=llvm

Fix MSVC signed/unsigned comparison warnings.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@298804 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/include/llvm/IR/Instructions.h b/include/llvm/IR/Instructions.h
index 3e94ee443b8..b8606a08eca 100644
--- a/include/llvm/IR/Instructions.h
+++ b/include/llvm/IR/Instructions.h
@@ -3129,13 +3129,14 @@ public:
 
     /// Resolves case value for current case.
     ConstantIntTy *getCaseValue() {
-      assert(Index < SI->getNumCases() && "Index out the number of cases.");
+      assert((unsigned)Index < SI->getNumCases() &&
+             "Index out the number of cases.");
       return reinterpret_cast<ConstantIntTy*>(SI->getOperand(2 + Index*2));
     }
 
     /// Resolves successor for current case.
     BasicBlockTy *getCaseSuccessor() {
-      assert((Index < SI->getNumCases() ||
+      assert(((unsigned)Index < SI->getNumCases() ||
               Index == DefaultPseudoIndex) &&
              "Index out the number of cases.");
       return SI->getSuccessor(getSuccessorIndex());
@@ -3146,7 +3147,8 @@ public:
 
     /// Returns TerminatorInst's successor index for current case successor.
     unsigned getSuccessorIndex() const {
-      assert((Index == DefaultPseudoIndex || Index < SI->getNumCases()) &&
+      assert((Index == DefaultPseudoIndex ||
+              (unsigned)Index < SI->getNumCases()) &&
              "Index out the number of cases.");
       return Index != DefaultPseudoIndex ? Index + 1 : 0;
     }
@@ -3154,7 +3156,7 @@ public:
     Self &operator+=(ptrdiff_t N) {
       // Check index correctness after addition.
       // Note: Index == getNumCases() means end().
-      assert(Index + N >= 0 && Index + N <= SI->getNumCases() &&
+      assert(Index + N >= 0 && (unsigned)(Index + N) <= SI->getNumCases() &&
              "Index out the number of cases.");
       Index += N;
       return *this;
@@ -3162,7 +3164,7 @@ public:
     Self &operator-=(ptrdiff_t N) {
       // Check index correctness after subtraction.
       // Note: Index == getNumCases() means end().
-      assert(Index - N >= 0 && Index - N <= SI->getNumCases() &&
+      assert(Index - N >= 0 && (unsigned)(Index - N) <= SI->getNumCases() &&
              "Index out the number of cases.");
       Index -= N;
       return *this;
@@ -3191,7 +3193,8 @@ public:
 
     /// Sets the new value for current case.
     void setValue(ConstantInt *V) {
-      assert(Index < SI->getNumCases() && "Index out the number of cases.");
+      assert((unsigned)Index < SI->getNumCases() &&
+             "Index out the number of cases.");
       SI->setOperand(2 + Index*2, reinterpret_cast<Value*>(V));
     }