Skip to content

Commit 522ad12

Browse files
committed
mpi: remove references to MPI_LB and MPI_UB (removed in MPI 3.0)
Signed-off-by: Gilles Gouaillardet <[email protected]>
1 parent 954f9ab commit 522ad12

File tree

16 files changed

+184
-326
lines changed

16 files changed

+184
-326
lines changed

ompi/datatype/ompi_datatype_create_subarray.c

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -40,12 +40,6 @@ int32_t ompi_datatype_create_subarray(int ndims,
4040
int32_t i, step, end_loop;
4141
MPI_Aint size, displ, extent;
4242

43-
/**
44-
* If the oldtype contains the original MPI_LB and MPI_UB markers then we
45-
* are forced to follow the MPI standard suggestion and reset these 2
46-
* markers (MPI 3.0 page 96 line 37). Otherwise we can simply resize the
47-
* datatype.
48-
*/
4943
ompi_datatype_type_extent( oldtype, &extent );
5044

5145
/* If the ndims is zero then return the NULL datatype */

ompi/datatype/ompi_datatype_internal.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -92,9 +92,6 @@
9292
#define OMPI_DATATYPE_MPI_C_DOUBLE_COMPLEX 0x2B
9393
#define OMPI_DATATYPE_MPI_C_LONG_DOUBLE_COMPLEX 0x2C
9494

95-
#define OMPI_DATATYPE_MPI_LB 0x2D
96-
#define OMPI_DATATYPE_MPI_UB 0x2E
97-
9895
/*
9996
* Datatypes from the MPI 3.0 standard
10097
*/

ompi/datatype/ompi_datatype_module.c

Lines changed: 74 additions & 81 deletions
Original file line numberDiff line numberDiff line change
@@ -68,8 +68,6 @@ ompi_predefined_datatype_t ompi_mpi_datatype_null =
6868

6969
ompi_predefined_datatype_t ompi_mpi_unavailable = OMPI_DATATYPE_INIT_PREDEFINED (UNAVAILABLE, 0);
7070

71-
ompi_predefined_datatype_t ompi_mpi_lb = OMPI_DATATYPE_INIT_PREDEFINED (LB, 0);
72-
ompi_predefined_datatype_t ompi_mpi_ub = OMPI_DATATYPE_INIT_PREDEFINED (UB, 0);
7371
ompi_predefined_datatype_t ompi_mpi_char = OMPI_DATATYPE_INIT_PREDEFINED (CHAR, OMPI_DATATYPE_FLAG_DATA_C);
7472
ompi_predefined_datatype_t ompi_mpi_signed_char = OMPI_DATATYPE_INIT_PREDEFINED (SIGNED_CHAR, OMPI_DATATYPE_FLAG_DATA_C | OMPI_DATATYPE_FLAG_DATA_INT );
7573
ompi_predefined_datatype_t ompi_mpi_unsigned_char = OMPI_DATATYPE_INIT_PREDEFINED (UNSIGNED_CHAR, OMPI_DATATYPE_FLAG_DATA_C | OMPI_DATATYPE_FLAG_DATA_INT );
@@ -363,9 +361,6 @@ const ompi_datatype_t* ompi_datatype_basicDatatypes[OMPI_DATATYPE_MPI_MAX_PREDEF
363361
[OMPI_DATATYPE_MPI_C_DOUBLE_COMPLEX] = &ompi_mpi_c_double_complex.dt,
364362
[OMPI_DATATYPE_MPI_C_LONG_DOUBLE_COMPLEX] = &ompi_mpi_c_long_double_complex.dt,
365363

366-
[OMPI_DATATYPE_MPI_LB] = &ompi_mpi_lb.dt,
367-
[OMPI_DATATYPE_MPI_UB] = &ompi_mpi_ub.dt,
368-
369364
/* MPI 3.0 types */
370365
[OMPI_DATATYPE_MPI_COUNT] = &ompi_mpi_count.dt,
371366

@@ -557,98 +552,96 @@ int32_t ompi_datatype_init( void )
557552
MOOG(datatype_null, 0);
558553
MOOG(byte, 1);
559554
MOOG(packed, 2);
560-
MOOG(ub, 3);
561-
MOOG(lb, 4);
562-
MOOG(character, 5);
563-
MOOG(logical, 6);
564-
MOOG(integer, 7);
565-
MOOG(integer1, 8);
566-
MOOG(integer2, 9);
567-
MOOG(integer4, 10);
568-
MOOG(integer8, 11);
569-
MOOG(integer16, 12);
570-
MOOG(real, 13);
571-
MOOG(real4, 14);
572-
MOOG(real8, 15);
573-
MOOG(real16, 16);
574-
MOOG(dblprec, 17);
575-
MOOG(cplex, 18);
576-
MOOG(complex8, 19);
577-
MOOG(complex16, 20);
578-
MOOG(complex32, 21);
579-
MOOG(dblcplex, 22);
580-
MOOG(2real, 23);
581-
MOOG(2dblprec, 24);
582-
MOOG(2integer, 25);
583-
MOOG(2cplex, 26);
584-
MOOG(2dblcplex, 27);
585-
MOOG(real2, 28);
586-
MOOG(logical1, 29);
587-
MOOG(logical2, 30);
588-
MOOG(logical4, 31);
589-
MOOG(logical8, 32);
555+
MOOG(character, 3);
556+
MOOG(logical, 4);
557+
MOOG(integer, 5);
558+
MOOG(integer1, 6);
559+
MOOG(integer2, 7);
560+
MOOG(integer4, 8);
561+
MOOG(integer8, 9);
562+
MOOG(integer16, 10);
563+
MOOG(real, 11);
564+
MOOG(real4, 12);
565+
MOOG(real8, 13);
566+
MOOG(real16, 14);
567+
MOOG(dblprec, 15);
568+
MOOG(cplex, 16);
569+
MOOG(complex8, 17);
570+
MOOG(complex16, 18);
571+
MOOG(complex32, 19);
572+
MOOG(dblcplex, 20);
573+
MOOG(2real, 21);
574+
MOOG(2dblprec, 22);
575+
MOOG(2integer, 23);
576+
MOOG(2cplex, 24);
577+
MOOG(2dblcplex, 25);
578+
MOOG(real2, 26);
579+
MOOG(logical1, 27);
580+
MOOG(logical2, 28);
581+
MOOG(logical4, 29);
582+
MOOG(logical8, 30);
590583

591584
/* Now the C types */
592585

593-
MOOG(wchar, 33);
594-
MOOG(char, 34);
595-
MOOG(unsigned_char, 35);
596-
MOOG(signed_char, 36);
597-
MOOG(short, 37);
598-
MOOG(unsigned_short, 38);
599-
MOOG(int, 39);
600-
MOOG(unsigned, 40);
601-
MOOG(long, 41);
602-
MOOG(unsigned_long, 42);
603-
MOOG(long_long_int, 43);
604-
MOOG(unsigned_long_long, 44);
605-
606-
MOOG(float, 45);
607-
MOOG(double, 46);
608-
MOOG(long_double, 47);
609-
610-
MOOG(float_int, 48);
611-
MOOG(double_int, 49);
612-
MOOG(longdbl_int, 50);
613-
MOOG(long_int, 51);
614-
MOOG(2int, 52);
615-
MOOG(short_int, 53);
586+
MOOG(wchar, 31);
587+
MOOG(char, 32);
588+
MOOG(unsigned_char, 33);
589+
MOOG(signed_char, 34);
590+
MOOG(short, 35);
591+
MOOG(unsigned_short, 36);
592+
MOOG(int, 37);
593+
MOOG(unsigned, 38);
594+
MOOG(long, 39);
595+
MOOG(unsigned_long, 40);
596+
MOOG(long_long_int, 41);
597+
MOOG(unsigned_long_long, 42);
598+
599+
MOOG(float, 43);
600+
MOOG(double, 44);
601+
MOOG(long_double, 45);
602+
603+
MOOG(float_int, 46);
604+
MOOG(double_int, 47);
605+
MOOG(longdbl_int, 48);
606+
MOOG(long_int, 49);
607+
MOOG(2int, 50);
608+
MOOG(short_int, 51);
616609

617610
/* C++ types */
618611

619-
MOOG(cxx_bool, 54);
620-
MOOG(cxx_cplex, 55);
621-
MOOG(cxx_dblcplex, 56);
622-
MOOG(cxx_ldblcplex, 57);
612+
MOOG(cxx_bool, 52);
613+
MOOG(cxx_cplex, 53);
614+
MOOG(cxx_dblcplex, 54);
615+
MOOG(cxx_ldblcplex, 55);
623616

624617
/* MPI 2.2 types */
625-
MOOG(int8_t, 58);
626-
MOOG(uint8_t, 59);
627-
MOOG(int16_t, 60);
628-
MOOG(uint16_t, 61);
629-
MOOG(int32_t, 62);
630-
MOOG(uint32_t, 63);
631-
MOOG(int64_t, 64);
632-
MOOG(uint64_t, 65);
633-
MOOG(aint, 66);
634-
MOOG(offset, 67);
635-
MOOG(c_bool, 68);
636-
MOOG(c_float_complex, 69);
637-
MOOG(c_double_complex, 70);
638-
MOOG(c_long_double_complex, 71);
618+
MOOG(int8_t, 56);
619+
MOOG(uint8_t, 57);
620+
MOOG(int16_t, 58);
621+
MOOG(uint16_t, 59);
622+
MOOG(int32_t, 60);
623+
MOOG(uint32_t, 61);
624+
MOOG(int64_t, 62);
625+
MOOG(uint64_t, 63);
626+
MOOG(aint, 64);
627+
MOOG(offset, 65);
628+
MOOG(c_bool, 66);
629+
MOOG(c_float_complex, 67);
630+
MOOG(c_double_complex, 68);
631+
MOOG(c_long_double_complex, 69);
639632

640633
/* MPI 3.0 types */
641-
MOOG(count, 72);
634+
MOOG(count, 70);
642635

643636
/* Datatype missing in old Open MPI */
644-
MOOG(complex4, 73);
637+
MOOG(complex4, 71);
645638

646639
/* Datatypes proposed to the MPI Forum in June 2017 for proposal in
647640
* the MPI 4.0 standard. As of February 2019, it is not accepted yet.
648641
* See https://github.com/mpi-forum/mpi-issues/issues/65 */
649-
MOOG(short_float, 74);
650-
MOOG(c_short_float_complex, 75);
651-
MOOG(cxx_sfltcplex, 76);
642+
MOOG(short_float, 72);
643+
MOOG(c_short_float_complex, 73);
644+
MOOG(cxx_sfltcplex, 74);
652645

653646
/**
654647
* Now make sure all non-contiguous types are marked as such.

ompi/include/mpif-values.pl

Lines changed: 69 additions & 71 deletions
Original file line numberDiff line numberDiff line change
@@ -122,80 +122,78 @@ sub write_file {
122122

123123
$handles->{MPI_BYTE} = 1;
124124
$handles->{MPI_PACKED} = 2;
125-
$handles->{MPI_UB} = 3;
126-
$handles->{MPI_LB} = 4;
127-
$handles->{MPI_CHARACTER} = 5;
128-
$handles->{MPI_LOGICAL} = 6;
129-
$handles->{MPI_INTEGER} = 7;
130-
$handles->{MPI_INTEGER1} = 8;
131-
$handles->{MPI_INTEGER2} = 9;
132-
$handles->{MPI_INTEGER4} = 10;
133-
$handles->{MPI_INTEGER8} = 11;
134-
$handles->{MPI_INTEGER16} = 12;
135-
$handles->{MPI_REAL} = 13;
136-
$handles->{MPI_REAL4} = 14;
137-
$handles->{MPI_REAL8} = 15;
138-
$handles->{MPI_REAL16} = 16;
139-
$handles->{MPI_DOUBLE_PRECISION} = 17;
140-
$handles->{MPI_COMPLEX} = 18;
141-
$handles->{MPI_COMPLEX8} = 19;
142-
$handles->{MPI_COMPLEX16} = 20;
143-
$handles->{MPI_COMPLEX32} = 21;
144-
$handles->{MPI_DOUBLE_COMPLEX} = 22;
145-
$handles->{MPI_2REAL} = 23;
146-
$handles->{MPI_2DOUBLE_PRECISION} = 24;
147-
$handles->{MPI_2INTEGER} = 25;
148-
$handles->{MPI_2COMPLEX} = 26;
149-
$handles->{MPI_2DOUBLE_COMPLEX} = 27;
150-
$handles->{MPI_REAL2} = 28;
151-
$handles->{MPI_LOGICAL1} = 29;
152-
$handles->{MPI_LOGICAL2} = 30;
153-
$handles->{MPI_LOGICAL4} = 31;
154-
$handles->{MPI_LOGICAL8} = 32;
155-
$handles->{MPI_WCHAR} = 33;
156-
$handles->{MPI_CHAR} = 34;
157-
$handles->{MPI_UNSIGNED_CHAR} = 35;
158-
$handles->{MPI_SIGNED_CHAR} = 36;
159-
$handles->{MPI_SHORT} = 37;
160-
$handles->{MPI_UNSIGNED_SHORT} = 38;
161-
$handles->{MPI_INT} = 39;
162-
$handles->{MPI_UNSIGNED} = 40;
163-
$handles->{MPI_LONG} = 41;
164-
$handles->{MPI_UNSIGNED_LONG} = 42;
165-
$handles->{MPI_LONG_LONG_INT} = 43;
125+
$handles->{MPI_CHARACTER} = 3;
126+
$handles->{MPI_LOGICAL} = 4;
127+
$handles->{MPI_INTEGER} = 5;
128+
$handles->{MPI_INTEGER1} = 6;
129+
$handles->{MPI_INTEGER2} = 7;
130+
$handles->{MPI_INTEGER4} = 8;
131+
$handles->{MPI_INTEGER8} = 9;
132+
$handles->{MPI_INTEGER16} = 10;
133+
$handles->{MPI_REAL} = 11;
134+
$handles->{MPI_REAL4} = 12;
135+
$handles->{MPI_REAL8} = 13;
136+
$handles->{MPI_REAL16} = 14;
137+
$handles->{MPI_DOUBLE_PRECISION} = 15;
138+
$handles->{MPI_COMPLEX} = 16;
139+
$handles->{MPI_COMPLEX8} = 17;
140+
$handles->{MPI_COMPLEX16} = 18;
141+
$handles->{MPI_COMPLEX32} = 19;
142+
$handles->{MPI_DOUBLE_COMPLEX} = 20;
143+
$handles->{MPI_2REAL} = 21;
144+
$handles->{MPI_2DOUBLE_PRECISION} = 22;
145+
$handles->{MPI_2INTEGER} = 23;
146+
$handles->{MPI_2COMPLEX} = 24;
147+
$handles->{MPI_2DOUBLE_COMPLEX} = 25;
148+
$handles->{MPI_REAL2} = 26;
149+
$handles->{MPI_LOGICAL1} = 27;
150+
$handles->{MPI_LOGICAL2} = 28;
151+
$handles->{MPI_LOGICAL4} = 29;
152+
$handles->{MPI_LOGICAL8} = 30;
153+
$handles->{MPI_WCHAR} = 31;
154+
$handles->{MPI_CHAR} = 32;
155+
$handles->{MPI_UNSIGNED_CHAR} = 33;
156+
$handles->{MPI_SIGNED_CHAR} = 34;
157+
$handles->{MPI_SHORT} = 35;
158+
$handles->{MPI_UNSIGNED_SHORT} = 36;
159+
$handles->{MPI_INT} = 37;
160+
$handles->{MPI_UNSIGNED} = 38;
161+
$handles->{MPI_LONG} = 39;
162+
$handles->{MPI_UNSIGNED_LONG} = 40;
163+
$handles->{MPI_LONG_LONG_INT} = 41;
166164
$handles->{MPI_LONG_LONG} = $handles->{MPI_LONG_LONG_INT};
167-
$handles->{MPI_UNSIGNED_LONG_LONG} = 44;
168-
$handles->{MPI_FLOAT} = 45;
169-
$handles->{MPI_DOUBLE} = 46;
170-
$handles->{MPI_LONG_DOUBLE} = 47;
171-
$handles->{MPI_FLOAT_INT} = 48;
172-
$handles->{MPI_DOUBLE_INT} = 49;
173-
$handles->{MPI_LONG_DOUBLE_INT} = 50;
174-
$handles->{MPI_LONG_INT} = 51;
175-
$handles->{MPI_2INT} = 52;
176-
$handles->{MPI_SHORT_INT} = 53;
177-
$handles->{MPI_CXX_BOOL} = 54;
178-
$handles->{MPI_CXX_FLOAT_COMPLEX} = 55;
165+
$handles->{MPI_UNSIGNED_LONG_LONG} = 42;
166+
$handles->{MPI_FLOAT} = 43;
167+
$handles->{MPI_DOUBLE} = 44;
168+
$handles->{MPI_LONG_DOUBLE} = 45;
169+
$handles->{MPI_FLOAT_INT} = 46;
170+
$handles->{MPI_DOUBLE_INT} = 47;
171+
$handles->{MPI_LONG_DOUBLE_INT} = 48;
172+
$handles->{MPI_LONG_INT} = 49;
173+
$handles->{MPI_2INT} = 50;
174+
$handles->{MPI_SHORT_INT} = 51;
175+
$handles->{MPI_CXX_BOOL} = 52;
176+
$handles->{MPI_CXX_FLOAT_COMPLEX} = 53;
179177
$handles->{MPI_CXX_COMPLEX} = $handles->{MPI_CXX_FLOAT_COMPLEX};
180-
$handles->{MPI_CXX_DOUBLE_COMPLEX} = 56;
181-
$handles->{MPI_CXX_LONG_DOUBLE_COMPLEX} = 57;
182-
$handles->{MPI_INT8_T} = 58;
183-
$handles->{MPI_UINT8_T} = 59;
184-
$handles->{MPI_INT16_T} = 60;
185-
$handles->{MPI_UINT16_T} = 61;
186-
$handles->{MPI_INT32_T} = 62;
187-
$handles->{MPI_UINT32_T} = 63;
188-
$handles->{MPI_INT64_T} = 64;
189-
$handles->{MPI_UINT64_T} = 65;
190-
$handles->{MPI_AINT} = 66;
191-
$handles->{MPI_OFFSET} = 67;
192-
$handles->{MPI_C_BOOL} = 68;
193-
$handles->{MPI_C_COMPLEX} = 69;
178+
$handles->{MPI_CXX_DOUBLE_COMPLEX} = 54;
179+
$handles->{MPI_CXX_LONG_DOUBLE_COMPLEX} = 55;
180+
$handles->{MPI_INT8_T} = 56;
181+
$handles->{MPI_UINT8_T} = 57;
182+
$handles->{MPI_INT16_T} = 58;
183+
$handles->{MPI_UINT16_T} = 59;
184+
$handles->{MPI_INT32_T} = 60;
185+
$handles->{MPI_UINT32_T} = 61;
186+
$handles->{MPI_INT64_T} = 62;
187+
$handles->{MPI_UINT64_T} = 63;
188+
$handles->{MPI_AINT} = 64;
189+
$handles->{MPI_OFFSET} = 65;
190+
$handles->{MPI_C_BOOL} = 66;
191+
$handles->{MPI_C_COMPLEX} = 67;
194192
$handles->{MPI_C_FLOAT_COMPLEX} = $handles->{MPI_C_COMPLEX};
195-
$handles->{MPI_C_DOUBLE_COMPLEX} = 70;
196-
$handles->{MPI_C_LONG_DOUBLE_COMPLEX} = 71;
197-
$handles->{MPI_COUNT} = 72;
198-
$handles->{MPI_COMPLEX4} = 73;
193+
$handles->{MPI_C_DOUBLE_COMPLEX} = 68;
194+
$handles->{MPI_C_LONG_DOUBLE_COMPLEX} = 69;
195+
$handles->{MPI_COUNT} = 70;
196+
$handles->{MPI_COMPLEX4} = 71;
199197

200198
$handles->{MPI_MESSAGE_NO_PROC} = 1;
201199

ompi/mca/coll/portals4/coll_portals4_component.c

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -111,9 +111,6 @@ ptl_datatype_t ompi_coll_portals4_atomic_datatype [OMPI_DATATYPE_MPI_MAX_PREDEFI
111111
[OMPI_DATATYPE_MPI_C_DOUBLE_COMPLEX] = COLL_PORTALS4_NO_DTYPE,
112112
[OMPI_DATATYPE_MPI_C_LONG_DOUBLE_COMPLEX] = COLL_PORTALS4_NO_DTYPE,
113113

114-
[OMPI_DATATYPE_MPI_LB] = COLL_PORTALS4_NO_DTYPE,
115-
[OMPI_DATATYPE_MPI_UB] = COLL_PORTALS4_NO_DTYPE,
116-
117114
/* MPI 3.0 types */
118115
[OMPI_DATATYPE_MPI_COUNT] = COLL_PORTALS4_NO_DTYPE,
119116

0 commit comments

Comments
 (0)