-
Notifications
You must be signed in to change notification settings - Fork 1.6k
OpenBLAS 0.3.13 fails to build on ppc64le / gcc 8.2 #3074
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
Comments
Looks like fallout from #2994 (and yes RajalakshmiSR warned about this happening, I should not have merged antonblanchard's PR without further clarification from him probably. This did not come up in my tests back then though) |
FYI, 0.3.12 builds fine (also using Spack, on the same POWER8 host). I assume you already have access to POWER machines, but just in case, the GCC compile farm has several hw/sw POWER environments to test things: https://cfarm.tetaneutral.net/ |
Thx, I'm allowed to use the OpenPOWER minicloud at unicamp.br |
@martin-frbg Should we just back out 2994 for now? |
Either that, or put another gcc version check for >=11 around that __builtin_cpu_is() call. The other change in #2994 looked more important, and I can only assume I got fixated on whether "cc -Wl,--version" was actually safer than "ld --version". |
@jonglezb You posted debian compiler version, does not matter, no released gcc would work. |
Actually gcc 10.2 would work, as RajalakshmiSR pointed out to me in #3075 the relevant gcc patch was backported from mainline shortly before its release. |
Thanks, I just tested a build from the develop branch and it builds fine! |
I'm building OpenBLAS 0.3.13 using Spack on a POWER8 machine running Debian 10.
Build fails because of the new POWER10 dynamic support:
Here are the versions:
I browsed other issues, and as far as I can tell, POWER10 support is supposed to be disabled with old versions of GCC. Not sure what's up.
The text was updated successfully, but these errors were encountered: