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

Fix Linux build issues #25

Merged
merged 5 commits into from
Nov 22, 2024
Merged

Fix Linux build issues #25

merged 5 commits into from
Nov 22, 2024

Conversation

Andreas-Krebbel
Copy link
Member

The current version of libzdnn does not build on Linux with recent GCCs (>=14). Unfortunately the data types of a few intrinsics changed in way which triggers compile errors when compiling without -flax-vector-conversions. Since using that option for building the entire library would probably paper over other potential errors as well, I rather went with a GCC version check for that particular macro. Clang was already using the GCC 14 intrinsics, so no change required here.

I've tried to keep the z/OS could unchanged, but could not test it. Please verify that it still works there. Thanks!

zdnn/zdnn_private.h Outdated Show resolved Hide resolved
Signed-off-by: Andreas Krebbel <[email protected]>
Signed-off-by: Nicholas Marion <[email protected]>
Copy link
Member

@n-marion n-marion left a comment

Choose a reason for hiding this comment

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

Confirmed runs well on zOS and internal tests.

Copy link
Member

@stbjones stbjones left a comment

Choose a reason for hiding this comment

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

approved

@n-marion n-marion merged commit 8823d82 into IBM:main Nov 22, 2024
1 check passed
n-marion added a commit that referenced this pull request Nov 22, 2024
* Fix Linux build issues
* Add missing prototypes to testsupport.h
* Get rid of type-aliasing warning
* Go down to z14 to support older distros
* Resolve CPPCheck errors.

---------

Signed-off-by: Andreas Krebbel <[email protected]>
Co-authored-by: Nicholas Marion <[email protected]>
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