Skip to content

OpenBLAS with POWER support#1693

Merged
bartoldeman merged 1 commit intoeasybuilders:developfrom
bear-rsg:openblas-on-power
Apr 25, 2019
Merged

OpenBLAS with POWER support#1693
bartoldeman merged 1 commit intoeasybuilders:developfrom
bear-rsg:openblas-on-power

Conversation

@edmondac
Copy link
Copy Markdown
Contributor

For POWER8/9 we need to say 'TARGET=POWER8'

Copy link
Copy Markdown
Contributor

@bartoldeman bartoldeman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bartoldeman bartoldeman added this to the next release (3.9.1) milestone Apr 25, 2019
@bartoldeman bartoldeman merged commit 3f5a4da into easybuilders:develop Apr 25, 2019
@edmondac edmondac deleted the openblas-on-power branch April 25, 2019 13:19
@edmondac
Copy link
Copy Markdown
Contributor Author

Thanks!

@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 25, 2019

It would be nice to also have a custom sanity_check_step implementation in place in this custom OpenBLAS easyblock, so we can consider switching over the current OpenBLAS easyconfig files to use this easyblock (rather than the generic ConfigureMake easyblock).

Also auto-skipping configure_step would be nice (with a simple pass), so the skipsteps = ['configure'] from the easyconfig files can be omitted, and automatically adding certain build options like CC="$CC" FC="$F77"' and install options like PREFIX=%(installdir)s (and maybe also the BINARY, USE_OPENMP and USE_THREAD ones, at least by default).

@edmondac Are you up for looking into that?

@edmondac
Copy link
Copy Markdown
Contributor Author

Yep - I'll do that @boegel

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