Skip to content
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

Power9 CMake fixes #636

Closed
wants to merge 2 commits into from
Closed

Conversation

scothalverson
Copy link

@scothalverson scothalverson commented Jun 14, 2019

Background

  • I'm trying to get Draco to build on IBM Power9 systems

Purpose of Pull Request

  • This adds two fixes that allow detecting Spectrum MPI and detecting that GFortran is built for PowerPC

Status

@KineticTheory
Copy link
Collaborator

@scothalverson Thanks for this patch. We are doing a release right now, so I will review these changes as compared to a different set of changes that I have on a branch. Either way, we will fix any build issues wrt rzansel and sierra.

@KineticTheory KineticTheory self-requested a review June 16, 2019 19:15
@KineticTheory KineticTheory self-assigned this Jun 16, 2019
@KineticTheory KineticTheory added this to the Draco-7_2_0 milestone Jun 16, 2019
@scothalverson
Copy link
Author

Thanks KT. That works for me. Its non-blocking for me, but I figured I wouldn't be the only one working with Power9.

@KineticTheory
Copy link
Collaborator

@scothalverson I'm going to close your PR as it is superseded by #637.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants