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

Fixes for ZSeries with GCC 9 or earlier or Clang 18 or earlier #2411

Merged
merged 1 commit into from
Dec 14, 2024

Conversation

johnplatts
Copy link
Contributor

Resolves issue #2409.

Made fixes to F32->F64 PromoteTo and F64->F32 DemoteTo to work around compiler errors on Z14/Z15 with GCC 8 or GCC 9.

Also made fixes to Reverse on Z14 with GCC 8 to work around the missing vec_reve intrinsic on Z14 with GCC 8 or earlier.

Also marked Z15 target as broken with GCC 8 or earlier as the Z15 target requires GCC 9 or later.

Also marked Z14 and Z15 targets as broken with Clang 18 or earlier due to LLVM bugs in LLVM 18 or earlier on Z14/Z15 that were fixed in LLVM 19, including a bug in LLVM 18 and earlier that was fixed in commit llvm/llvm-project@7e4c6e9 (which made its way into LLVM 19 and later).

Copy link
Member

@jan-wassenberg jan-wassenberg left a comment

Choose a reason for hiding this comment

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

Great, thanks for working around the compiler issues :)

@copybara-service copybara-service bot merged commit 065050e into google:master Dec 14, 2024
37 of 40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants