Skip to content

Commit 5dfd4ab

Browse files
committed
coll/tuned: remove set-but-not-used variables
Signed-off-by: Gilles Gouaillardet <[email protected]>
1 parent fe64144 commit 5dfd4ab

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

ompi/mca/coll/tuned/coll_tuned_dynamic_rules.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,8 @@
1010
* Copyright (c) 2004-2005 The Regents of the University of California.
1111
* All rights reserved.
1212
* Copyright (c) 2011-2012 FUJITSU LIMITED. All rights reserved.
13+
* Copyright (c) 2017 Research Organization for Information Science
14+
* and Technology (RIST). All rights reserved.
1315
* $COPYRIGHT$
1416
*
1517
* Additional copyrights may follow
@@ -289,7 +291,7 @@ ompi_coll_com_rule_t* ompi_coll_tuned_get_com_rule_ptr (ompi_coll_alg_rule_t* ru
289291
ompi_coll_alg_rule_t* alg_p = (ompi_coll_alg_rule_t*) NULL;
290292
ompi_coll_com_rule_t* com_p = (ompi_coll_com_rule_t*) NULL;
291293
ompi_coll_com_rule_t* best_com_p = (ompi_coll_com_rule_t*) NULL;
292-
int i, best;
294+
int i;
293295

294296
if (!rules) { /* no rule base no resulting com rule */
295297
return ((ompi_coll_com_rule_t*)NULL);
@@ -305,13 +307,12 @@ ompi_coll_com_rule_t* ompi_coll_tuned_get_com_rule_ptr (ompi_coll_alg_rule_t* ru
305307

306308
/* make a copy of the first com rule */
307309
best_com_p = com_p = alg_p->com_rules;
308-
i = best = 0;
310+
i = 0;
309311

310312
while( i < alg_p->n_com_sizes ) {
311313
if (com_p->mpi_comsize > mpi_comsize) {
312314
break;
313315
}
314-
best = i;
315316
best_com_p = com_p;
316317
/* go to the next entry */
317318
com_p++;
@@ -344,7 +345,7 @@ int ompi_coll_tuned_get_target_method_params (ompi_coll_com_rule_t* base_com_rul
344345
{
345346
ompi_coll_msg_rule_t* msg_p = (ompi_coll_msg_rule_t*) NULL;
346347
ompi_coll_msg_rule_t* best_msg_p = (ompi_coll_msg_rule_t*) NULL;
347-
int i, best;
348+
int i;
348349

349350
/* No rule or zero rules */
350351
if( (NULL == base_com_rule) || (0 == base_com_rule->n_msg_sizes)) {
@@ -355,13 +356,12 @@ int ompi_coll_tuned_get_target_method_params (ompi_coll_com_rule_t* base_com_rul
355356

356357
/* make a copy of the first msg rule */
357358
best_msg_p = msg_p = base_com_rule->msg_rules;
358-
i = best = 0;
359+
i = 0;
359360

360361
while (i<base_com_rule->n_msg_sizes) {
361362
/* OPAL_OUTPUT((ompi_coll_tuned_stream,"checking mpi_msgsize %d against com_id %d msg_id %d index %d msg_size %d", */
362363
/* mpi_msgsize, msg_p->com_rule_id, msg_p->msg_rule_id, i, msg_p->msg_size)); */
363364
if (msg_p->msg_size <= mpi_msgsize) {
364-
best = i;
365365
best_msg_p = msg_p;
366366
/* OPAL_OUTPUT((ompi_coll_tuned_stream(":ok\n")); */
367367
}

0 commit comments

Comments
 (0)