commit a81998254880310b34e1f783d7a0d07b7d0cac5c
author Peyton, Jonathan L <jonathan.l.peyton@intel.com> Tue Feb 02 10:38:33 2021 -0600
committer Copybara-Service <copybara-worker@google.com> Fri Feb 05 03:24:08 2021 -0800
tree c6a880a43e535b63a776ae7def9c35d8823d89b2
parent 5fb66646bbbbabf5b5283ae5164cbd86e45f7e36

[OpenMP] Fix sign comparison warnings from GCC

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

GitOrigin-RevId: ffca74b8b82f0af0d213bffa59c40b10449fd05f

runtime/src/kmp_affinity.cpp

diff --git a/runtime/src/kmp_affinity.cpp b/runtime/src/kmp_affinity.cpp
index 0161aca..a94dbd1 100644
--- a/runtime/src/kmp_affinity.cpp
+++ b/runtime/src/kmp_affinity.cpp
@@ -109,8 +109,8 @@
     }
     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) {