Skip to content

Commit ac6d734

Browse files
committed
coll/han: fix compiler warnings
Fix an unused variable, and explictly cast uint64_t* to size_t* Note that the latter won't be needed after embiggening collectives for large count support. Signed-off-by: Wenduo Wang <[email protected]>
1 parent 6564419 commit ac6d734

File tree

3 files changed

+2
-3
lines changed

3 files changed

+2
-3
lines changed

ompi/mca/coll/han/coll_han_gatherv.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -332,7 +332,7 @@ int mca_coll_han_gatherv_intra(const void *sbuf, int scount, struct ompi_datatyp
332332
}
333333

334334
if (max_data_size > (size_t) INT_MAX) {
335-
datatype_size = coll_han_utils_gcd(low_data_size, low_size);
335+
datatype_size = coll_han_utils_gcd((const size_t *) low_data_size, low_size);
336336
}
337337

338338
low_rcounts = malloc(low_size * sizeof(int));

ompi/mca/coll/han/coll_han_reduce.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,6 @@ mca_coll_han_reduce_intra(const void *sbuf,
132132
int *vranks = han_module->cached_vranks;
133133
int low_rank = ompi_comm_rank(low_comm);
134134
int low_size = ompi_comm_size(low_comm);
135-
int up_rank = ompi_comm_rank(up_comm);
136135

137136
int root_low_rank;
138137
int root_up_rank;

ompi/mca/coll/han/coll_han_scatterv.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -343,7 +343,7 @@ int mca_coll_han_scatterv_intra(const void *sbuf, const int *scounts, const int
343343
}
344344

345345
if (max_data_size > (size_t) INT_MAX) {
346-
datatype_size = coll_han_utils_gcd(low_data_size, low_size);
346+
datatype_size = coll_han_utils_gcd((const size_t *) low_data_size, low_size);
347347
}
348348

349349
low_scounts = malloc(low_size * sizeof(int));

0 commit comments

Comments
 (0)