Skip to content

Commit 20721c1

Browse files
committed
coll/ftagree: Fix warnings about unused variables
Fix warnings in non-debug builds about unused variables by folding into debug areas. Signed-off-by: Brian Barrett <bbarrett@amazon.com>
1 parent 633fbf3 commit 20721c1

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

ompi/mca/coll/ftagree/coll_ftagree_earlyreturning.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1623,11 +1623,12 @@ static void era_decide(era_value_t *decided_value, era_agreement_info_t *ci)
16231623
ompi_communicator_t *comm;
16241624
era_rank_item_t *rl;
16251625
int r, s, dead_size;
1626-
void *value;
16271626

16281627
assert( 0 != ci->agreement_id.ERAID_FIELDS.agreementid );
16291628

16301629
#if OPAL_ENABLE_DEBUG
1630+
void *value;
1631+
16311632
r = era_parent(ci);
16321633
if( opal_hash_table_get_value_uint64(&era_passed_agreements,
16331634
ci->agreement_id.ERAID_KEY, &value) == OMPI_SUCCESS ) {
@@ -2636,7 +2637,6 @@ static void era_cb_fn(struct mca_btl_base_module_t* btl,
26362637
const mca_btl_base_receive_descriptor_t* descriptor)
26372638
{
26382639
era_incomplete_msg_t *incomplete_msg = NULL;
2639-
mca_btl_base_tag_t tag = descriptor->tag;
26402640
era_msg_header_t *msg_header;
26412641
era_frag_t *frag;
26422642
uint64_t src_hash;
@@ -2646,7 +2646,7 @@ static void era_cb_fn(struct mca_btl_base_module_t* btl,
26462646
int *new_dead;
26472647
int *ack_failed;
26482648

2649-
assert(MCA_BTL_TAG_FT_AGREE == tag);
2649+
assert(MCA_BTL_TAG_FT_AGREE == descriptor->tag);
26502650
assert(1 == descriptor->des_segment_count);
26512651

26522652
frag = (era_frag_t*)descriptor->des_segments->seg_addr.pval;

0 commit comments

Comments
 (0)