Skip to content

sync_builtin: check for 64-bit atomic support #2698

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 1 commit into from
Jan 10, 2017

Conversation

hjelmn
Copy link
Member

@hjelmn hjelmn commented Jan 10, 2017

This commit adds an additional check for 64-bit atomic support for _sync
builtins. If 64-bit support is not available the opal_atomic
*_64 atomics
are disabled.

Signed-off-by: Nathan Hjelm [email protected]
(cherry picked from commit d99a978)
Signed-off-by: Nathan Hjelm [email protected]

This commit adds an additional check for 64-bit atomic support for __sync
builtins. If 64-bit support is not available the opal_atomic_*_64 atomics
are disabled.

Signed-off-by: Nathan Hjelm <[email protected]>
(cherry picked from commit d99a978)
Signed-off-by: Nathan Hjelm <[email protected]>
@hppritcha
Copy link
Member

experimenting with CFLAGS for disable-dlopen project
bot:lanl:retest

@jsquyres
Copy link
Member

@hppritcha Good to go.

@hppritcha
Copy link
Member

@jsquyres the lanl dlopen failed because I added CFLAGS="-O3" to the configury, so it hit the problem that #2681 is suppose to fix.

@hppritcha hppritcha merged commit 4f5a119 into open-mpi:v2.0.x Jan 10, 2017
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