Skip to content

POWER: Fix --optarch=GENERIC for GCC#2073

Merged
boegel merged 4 commits intoeasybuilders:developfrom
geimer:power_optarch_generic_gcc
Jan 7, 2017
Merged

POWER: Fix --optarch=GENERIC for GCC#2073
boegel merged 4 commits intoeasybuilders:developfrom
geimer:power_optarch_generic_gcc

Conversation

@geimer
Copy link
Copy Markdown
Contributor

@geimer geimer commented Jan 5, 2017

Fixes #2064

Note that this PR builds on top of #1974 (the diff will get lean and mean once #1974 is merged)

@geimer
Copy link
Copy Markdown
Contributor Author

geimer commented Jan 5, 2017

With this branch, I was able to robot-build HPL-2.1-foss-2016b.eb on our BG/Q frontend (POWER7) with EASYBUILD_OPTARCH=GENERIC set in the environment. I only had to tweak the FFTW easyconfig a bit (due to the hardcoded --enable-sse2 configure option and a missing libquadmath). The resulting xhpl binary ran fine, though I don't have the possibility to test whether it would also run on a different POWER-based system (i.e., whether it is "generic").

@boegel boegel added this to the 3.1.0 milestone Jan 5, 2017
@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 6, 2017

@geimer please sync with develop now that #1974 is merged?

Seems like you'll also need to deal with a merge conflict in systemtools.py...

@geimer
Copy link
Copy Markdown
Contributor Author

geimer commented Jan 6, 2017

@boegel: Synced. And git didn't report any conflicts...

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 6, 2017

@geimer OK, great. GitHub mentioned there was a conflict, but I guess it just got confused.

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 6, 2017

@JackPerdue Can you take a look at these changes, and provide some feedback?

Looks good to me, but this may impact you (even though the changes look backwards-compatible to me).

@JackPerdue
Copy link
Copy Markdown
Contributor

A quick test:
gcc -mcpu=native hello.c
gcc -mcpu=powerpc64 hello.c
xlc -qtune=auto -qmaxmem=1 hello.c
all work on our Power7 cluster (our BG/Q is gone).

I didn't do a full test with EB, but it looks good to me at this point..

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 7, 2017

OK, thanks for the feedback @JackPerdue!

I see no reason to hold this back any further, thanks a lot for tackling this @geimer, going in!

@boegel boegel merged commit 7a413bd into easybuilders:develop Jan 7, 2017
@geimer geimer deleted the power_optarch_generic_gcc branch January 8, 2017 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants