Switch up the conditional statement, add OPAL_LIKELY.
Thanks to Jeff for the suggestion. This commit was SVN r21082.
Этот коммит содержится в:
родитель
1663e51d9d
Коммит
aae5d3c825
@ -4082,10 +4082,7 @@ static int drain_message_copy_remove(ompi_crcp_bkmrk_pml_drain_message_ref_t *dr
|
|||||||
}
|
}
|
||||||
|
|
||||||
/* The buffer could be NULL - More likely when doing a count=0 type of message (e.g., Barrier) */
|
/* The buffer could be NULL - More likely when doing a count=0 type of message (e.g., Barrier) */
|
||||||
if( NULL == buf ) {
|
if( OPAL_LIKELY(NULL != buf) ) {
|
||||||
OPAL_OUTPUT_VERBOSE((20, mca_crcp_bkmrk_component.super.output_handle,
|
|
||||||
"crcp:bkmrk: drain_message_copy_remove(): Skip copy - NULL buffer"));
|
|
||||||
} else {
|
|
||||||
if( 0 != (ret = ompi_ddt_copy_content_same_ddt(datatype, count,
|
if( 0 != (ret = ompi_ddt_copy_content_same_ddt(datatype, count,
|
||||||
(void*)buf, drain_content_ref->buffer) ) ) {
|
(void*)buf, drain_content_ref->buffer) ) ) {
|
||||||
opal_output( mca_crcp_bkmrk_component.super.output_handle,
|
opal_output( mca_crcp_bkmrk_component.super.output_handle,
|
||||||
@ -4094,6 +4091,10 @@ static int drain_message_copy_remove(ompi_crcp_bkmrk_pml_drain_message_ref_t *dr
|
|||||||
exit_status = ret;
|
exit_status = ret;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
else {
|
||||||
|
OPAL_OUTPUT_VERBOSE((20, mca_crcp_bkmrk_component.super.output_handle,
|
||||||
|
"crcp:bkmrk: drain_message_copy_remove(): Skip copy - NULL buffer"));
|
||||||
|
}
|
||||||
|
|
||||||
/* Remove the message from the list */
|
/* Remove the message from the list */
|
||||||
drain_content_ref->request = NULL;
|
drain_content_ref->request = NULL;
|
||||||
|
Загрузка…
x
Ссылка в новой задаче
Block a user