[OpenMP] Fix sign comparison warnings from GCC
authorPeyton, Jonathan L <jonathan.l.peyton@intel.com>
Tue, 2 Feb 2021 16:38:33 +0000 (10:38 -0600)
committerPeyton, Jonathan L <jonathan.l.peyton@intel.com>
Tue, 2 Feb 2021 16:52:16 +0000 (10:52 -0600)
New affinity patch introduced legitimate sign-compare warnings that
clang doesn't report but GCC-10 does. This removes the warnings by
changing two variables types to unsigned.

Differential Revision: https://reviews.llvm.org/D95818

openmp/runtime/src/kmp_affinity.cpp

index 0161aca..a94dbd1 100644 (file)
@@ -109,8 +109,8 @@ static int __kmp_affinity_remove_radix_one_levels(AddrUnsPair *addrP, int nTh,
     }
     bool radix1 = true;
     bool all_same = true;
-    int id1 = addrP[0].first.labels[top_index1];
-    int id2 = addrP[0].first.labels[top_index2];
+    unsigned id1 = addrP[0].first.labels[top_index1];
+    unsigned id2 = addrP[0].first.labels[top_index2];
     int pref1 = preference[type1];
     int pref2 = preference[type2];
     for (int hwidx = 1; hwidx < nTh; ++hwidx) {