-
Notifications
You must be signed in to change notification settings - Fork 885
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
v3.0.0rc1: XLC build errors with ompi_datatype_module.c #3811
Comments
@PHHargrove can you please give the inline path a try ? diff --git a/ompi/datatype/ompi_datatype_internal.h b/ompi/datatype/ompi_datatype_internal.h
index 4323f0c..f4f5848 100644
--- a/ompi/datatype/ompi_datatype_internal.h
+++ b/ompi/datatype/ompi_datatype_internal.h
@@ -467,7 +467,7 @@ extern const ompi_datatype_t* ompi_datatype_basicDatatypes[OMPI_DATATYPE_MPI_MAX
.name = OPAL_DATATYPE_INIT_NAME(TYPE ## SIZE), \
.desc = OPAL_DATATYPE_INIT_DESC_PREDEFINED(TYPE ## SIZE), \
.opt_desc = OPAL_DATATYPE_INIT_DESC_PREDEFINED(TYPE ## SIZE), \
- .ptypes = OPAL_DATATYPE_INIT_PTYPES_ARRAY(TYPE ## SIZE) \
+ .ptypes = OPAL_DATATYPE_INIT_PTYPES_ARRAY_UNAVAILABLE \
}
#define OMPI_DATATYPE_INIT_PREDEFINED_BASIC_TYPE_FORTRAN( TYPE, NAME, SIZE, ALIGN, FLAGS ) \ @bosilca does that sound like the right fix ? |
@ggouaillardet On the xlc-12.1 system the LP64 build gets all the way to a successful The xlc-12.1/ILP32 and xlc13-1/LP64 builds each yielded a new issue |
@ggouaillardet Did you file a PR corresponding to the patch you included above? |
👍 |
Still present in 3.0.0rc2 |
Josh Hursey is addressing this by discontinuing support for PPC BigEndian in configure for v3.x |
This has been resolved by PR #4103 |
From @PHHargrove on https://www.mail-archive.com/[email protected]/msg20275.html:
With xlc-12.1 and 13.1 (big-endian PPC64, both LP64 and ILP32) there are numerous problems compiling with ompi_datatype_module.c (a prefix of which are shown below).
The text was updated successfully, but these errors were encountered: