Skip to content

Commit bfe0240

Browse files
committed
opal/datatype: only use opal_pack_general[_checksum] if CONVERTOR_SEND_CONVERSION && ! CONVERTOR_HOMOGENEOUS
(cherry picked from commit cd829e4)
1 parent 8be543a commit bfe0240

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

opal/datatype/opal_convertor.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
* All rights reserved.
1313
* Copyright (c) 2009 Oak Ridge National Labs. All rights reserved.
1414
* Copyright (c) 2011 NVIDIA Corporation. All rights reserved.
15-
* Copyright (c) 2013 Research Organization for Information Science
15+
* Copyright (c) 2013-2016 Research Organization for Information Science
1616
* and Technology (RIST). All rights reserved.
1717
* $COPYRIGHT$
1818
*
@@ -595,7 +595,7 @@ int32_t opal_convertor_prepare_for_send( opal_convertor_t* convertor,
595595
OPAL_CONVERTOR_PREPARE( convertor, datatype, count, pUserBuf );
596596

597597
if( convertor->flags & CONVERTOR_WITH_CHECKSUM ) {
598-
if( !(convertor->flags & CONVERTOR_HOMOGENEOUS) ) {
598+
if( CONVERTOR_SEND_CONVERSION == (convertor->flags & (CONVERTOR_SEND_CONVERSION|CONVERTOR_HOMOGENEOUS)) ) {
599599
convertor->fAdvance = opal_pack_general_checksum;
600600
} else {
601601
if( datatype->flags & OPAL_DATATYPE_FLAG_CONTIGUOUS ) {
@@ -609,7 +609,7 @@ int32_t opal_convertor_prepare_for_send( opal_convertor_t* convertor,
609609
}
610610
}
611611
} else {
612-
if( !(convertor->flags & CONVERTOR_HOMOGENEOUS) ) {
612+
if( CONVERTOR_SEND_CONVERSION == (convertor->flags & (CONVERTOR_SEND_CONVERSION|CONVERTOR_HOMOGENEOUS)) ) {
613613
convertor->fAdvance = opal_pack_general;
614614
} else {
615615
if( datatype->flags & OPAL_DATATYPE_FLAG_CONTIGUOUS ) {

0 commit comments

Comments
 (0)