-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Fix power10 gcc intrinsic check #4193
Conversation
@imciner2 Can you paste the gcc version that you are using? |
@@ -167,7 +167,7 @@ typedef __vector unsigned char vec_t; | |||
|
|||
#define INIT_1ACC() __builtin_mma_xxsetaccz(&acc0); | |||
|
|||
#if (defined(__GNUC__) && (__GNUC__ == 10)) | |||
#if (defined(__GNUC__) && (__GNUC__ >= 10)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
__builtin_vsx_build_pair() is the correct usage for gcc 11 and above.
The GCC version is
The errors I am seeing are specifically these:
|
__builtin_vsx_build_pair() is available from 11.3. Can you change it as.. |
__builtin_vsx_assemble_pair was only in GCC 10-11.2 and was replaced by __builtin_vsx_build_pair thereafter.
Yep, that works in my local testing, changed in the PR. |
__builtin_vsx_assemble_pair was introduced in GCC 10 as the replacement, so it should be used for all greater GCC versions as well, otherwise compilation with GCC 11 will fail.