[Openmp-commits] [openmp] 3f48d34 - [OpenMP][runtime] Fix comparison of integer expressions of different signedness (#128204)
via Openmp-commits
openmp-commits at lists.llvm.org
Sun Mar 2 23:55:01 PST 2025
Author: foxtran
Date: 2025-03-03T08:54:57+01:00
New Revision: 3f48d34dfffac81bc73db626438f531c5324f85b
URL: https://github.com/llvm/llvm-project/commit/3f48d34dfffac81bc73db626438f531c5324f85b
DIFF: https://github.com/llvm/llvm-project/commit/3f48d34dfffac81bc73db626438f531c5324f85b.diff
LOG: [OpenMP][runtime] Fix comparison of integer expressions of different signedness (#128204)
This PR fixes warning which occurs if one compiles OpenMP runtime with
GCC:
```
warning: comparison of integer expressions of different signedness: 'kmp_intptr_t' {aka 'long int'} and 'long unsigned int' [-Wsign-compare]
```
Added:
Modified:
openmp/runtime/src/kmp_taskdeps.cpp
Removed:
################################################################################
diff --git a/openmp/runtime/src/kmp_taskdeps.cpp b/openmp/runtime/src/kmp_taskdeps.cpp
index 392c8a2b19333..41173e9e1de6e 100644
--- a/openmp/runtime/src/kmp_taskdeps.cpp
+++ b/openmp/runtime/src/kmp_taskdeps.cpp
@@ -764,7 +764,7 @@ kmp_int32 __kmpc_omp_task_with_deps(ident_t *loc_ref, kmp_int32 gtid,
for (i = 0; i < ndeps; i++) {
ompt_deps[i].variable.ptr = (void *)dep_list[i].base_addr;
- if (dep_list[i].base_addr == KMP_SIZE_T_MAX)
+ if (dep_list[i].base_addr == (kmp_intptr_t)KMP_SIZE_T_MAX)
ompt_deps[i].dependence_type = ompt_dependence_type_out_all_memory;
else if (dep_list[i].flags.in && dep_list[i].flags.out)
ompt_deps[i].dependence_type = ompt_dependence_type_inout;
@@ -781,7 +781,7 @@ kmp_int32 __kmpc_omp_task_with_deps(ident_t *loc_ref, kmp_int32 gtid,
}
for (i = 0; i < ndeps_noalias; i++) {
ompt_deps[ndeps + i].variable.ptr = (void *)noalias_dep_list[i].base_addr;
- if (noalias_dep_list[i].base_addr == KMP_SIZE_T_MAX)
+ if (noalias_dep_list[i].base_addr == (kmp_intptr_t)KMP_SIZE_T_MAX)
ompt_deps[ndeps + i].dependence_type =
ompt_dependence_type_out_all_memory;
else if (noalias_dep_list[i].flags.in && noalias_dep_list[i].flags.out)
More information about the Openmp-commits
mailing list