From ffca74b8b82f0af0d213bffa59c40b10449fd05f Mon Sep 17 00:00:00 2001 From: "Peyton, Jonathan L" Date: Tue, 2 Feb 2021 10:38:33 -0600 Subject: [PATCH] [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 --- openmp/runtime/src/kmp_affinity.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/openmp/runtime/src/kmp_affinity.cpp b/openmp/runtime/src/kmp_affinity.cpp index 0161aca..a94dbd1 100644 --- a/openmp/runtime/src/kmp_affinity.cpp +++ b/openmp/runtime/src/kmp_affinity.cpp @@ -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) { -- 2.7.4