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

OpenBLAS 0.3.1 Compile Fails with IBM XL 16.1.0 Compiler on POWER8 #1699

Closed
nmhamster opened this issue Jul 25, 2018 · 7 comments
Closed

OpenBLAS 0.3.1 Compile Fails with IBM XL 16.1.0 Compiler on POWER8 #1699

nmhamster opened this issue Jul 25, 2018 · 7 comments

Comments

@nmhamster
Copy link

Bug when trying to compile OpenBLAS 0.3.1 download using IBM XL 16.1 (C/C++ and Fortran compilers) on POWER8. It appears inline assembly does not compile correctly.

[ 26%] Building C object kernel/CMakeFiles/kernel.dir/CMakeFiles/sasum_k.c.o
In file included from /ascldap/users/sdhammo/openblas/OpenBLAS-0.3.1/build-power8/kernel/CMakeFiles/sasum_k.c:7:
In file included from /ascldap/users/sdhammo/openblas/OpenBLAS-0.3.1/kernel/power/sasum.c:50:
/ascldap/users/sdhammo/openblas/OpenBLAS-0.3.1/kernel/power/sasum_microk_power8.c:172:8: error: unknown register name 'vs32' in asm
       "vs32","vs33","vs34","vs35","vs36","vs37","vs38","vs39",
@martin-frbg
Copy link
Collaborator

Known problem, see #1248 - according to #1078 (comment) this is a limitation of the XL compiler.

@trudeaun
Copy link

IBM XL 16.1 uses clang technology, and so far, "vs" register names are unsupported by clang. As a workaround, you can use v0, …, v7, instead of "vs32", …, "vs39".

@martin-frbg
Copy link
Collaborator

martin-frbg commented Jul 25, 2018

Good to know - was this already the case with xlc 13, which was the subject of the earlier issues ? (I note another problem back then was lack of support for constraints like "=wa" (t0)" in this same file, sasum_mikrok_power.c , which kept me from doing the vs mass rename - #1078 (comment) )

@trudeaun
Copy link

For "vs" register names, my comment applies to xlc 13 as well. For the second problem related to constraints like "wa", unfortunately even the latest version of XL (16.1) does not support it yet. This feature request is in XL's backlog and hopefully it can be implemented soon.

@martin-frbg
Copy link
Collaborator

So are there equivalent constraints that xlc supports ? So far it looks like David Edelsohn's comment is still valid ...

@trudeaun
Copy link

Unfortunately, no. The currently supported constraints are listed on Knowledge Center. Sorry about that.

@martin-frbg
Copy link
Collaborator

I have added an entry to the faq in the wiki (thought I had done that already earlier).

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

No branches or pull requests

3 participants