Skip to content

BUG: Fix build on ppc64 when the baseline set to Power9 or higher #24806

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

Merged
merged 4 commits into from
Nov 7, 2023

Conversation

seiko2plus
Copy link
Member

closes #24789

@seiko2plus seiko2plus added the 09 - Backport-Candidate PRs tagged should be backported label Sep 26, 2023
@seiko2plus seiko2plus marked this pull request as draft September 26, 2023 09:46
@seiko2plus seiko2plus force-pushed the issue_24789 branch 4 times, most recently from 0e81137 to 431b7dd Compare October 25, 2023 19:47
…Half/Double VSX3 instructions by the assembler
  This fix addresses two issues:
   * Corrects the use of unsupported instructions by the assembler
     in half-precision to double-precision conversion.
   * Resolves a code error related to variable naming during conversion.
@rgommers
Copy link
Member

rgommers commented Nov 6, 2023

@seiko2plus is this still marked as Draft on purpose?

@seiko2plus
Copy link
Member Author

is this still marked as Draft on purpose?

I had to test it locally against an assembler that doesn't support VSX3 scalar double-precision conversion. It's ready for review now.

@charris charris merged commit 14f2257 into numpy:main Nov 7, 2023
@charris
Copy link
Member

charris commented Nov 7, 2023

Thanks Sayed.

@charris charris removed the 09 - Backport-Candidate PRs tagged should be backported label Nov 11, 2023
charris added a commit to charris/numpy that referenced this pull request Nov 11, 2023
Backport of numpy#24806.

This backport was already made in numpy#25083, but that didn't make
use of the linux_qemu.yml action, so this tweaks it a bit.
charris added a commit to charris/numpy that referenced this pull request Nov 11, 2023
Backport of numpy#24806.

This backport was already made in numpy#25083, but that didn't make
use of the linux_qemu.yml action, so this tweaks it a bit.
charris added a commit to charris/numpy that referenced this pull request Nov 11, 2023
Backport of numpy#24806.

This backport was already made in numpy#25083, but that didn't make
use of the linux_qemu.yml action, so this tweaks it a bit.
charris added a commit to charris/numpy that referenced this pull request Nov 11, 2023
Backport of numpy#24806.

This backport was already made in numpy#25083, but that didn't make
use of the linux_qemu.yml action, so this tweaks it a bit.
@Flamefire
Copy link
Contributor

Flamefire commented Dec 13, 2023

FWIW: 7a84442 fixes a test failure in test_half_conversions caused by a difference of 512 in indices 31745-32255 & 64513-65023 (all NaNs) of the test for float64 on Sapphire Rapids (AVX512)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BLD: PPC issues with new CPU dispatcher
4 participants