Skip to content

Make Oracle compilers happy with PMIx 1.2.1+ #3062

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

Merged
merged 3 commits into from
Feb 28, 2017

Conversation

jjhursey
Copy link
Member

Fixes #3048

Cherry pick of these two PMIx v1.2 commits:

Signed-off-by: Gilles Gouaillardet <gilles@rist.or.jp>
(cherry picked from commit on PMIx v1.2 branch openpmix/openpmix@a2d431c)
Signed-off-by: Joshua Hursey <jhursey@us.ibm.com>
revamp the _ESH_{WR,RD,UN}LOCK macros to work around
a bug in the Oracle compilers that resulted in a crash
at compilation time

Signed-off-by: Gilles Gouaillardet <gilles@rist.or.jp>
(cherry picked from commit on PMIx v1.2 branch openpmix/openpmix@8587f27)
Signed-off-by: Joshua Hursey <jhursey@us.ibm.com>
@jjhursey jjhursey added the bug label Feb 28, 2017
@jjhursey jjhursey added this to the v2.1.0 milestone Feb 28, 2017
@jjhursey jjhursey requested a review from rhc54 February 28, 2017 16:16
@jjhursey
Copy link
Member Author

I think the only thing missing from this PR so far is a request to add a README to opal/mca/pmix/pmix112/ noting that this is the PMIx v1.2.1 release with these two cherry-picks. I'll do that shortly.

Signed-off-by: Joshua Hursey <jhursey@us.ibm.com>
@jjhursey
Copy link
Member Author

@jsquyres I added the README to the opal/mca/pmix/pmix112 directory identifying the release and which commits were cherry-picked. Let me know if it needs further updates.

@jsquyres jsquyres merged commit 6a2fb80 into open-mpi:v2.x Feb 28, 2017
@jjhursey jjhursey deleted the fix/v2.x/suncc-pmix branch February 28, 2017 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants