Skip to content

Conversation

@skirpichev
Copy link
Contributor

@skirpichev skirpichev commented Nov 6, 2025

@skirpichev skirpichev added the needs backport to 3.14 bugs and security fixes label Nov 6, 2025
@skirpichev skirpichev marked this pull request as ready for review November 6, 2025 04:47
@skirpichev skirpichev changed the title gh-141042: fix sNaN's (un)packing for mixed floating-point formats gh-141042: fix sNaN's packing for mixed floating-point formats Nov 6, 2025
@skirpichev
Copy link
Contributor Author

CC @vstinner

@skirpichev skirpichev requested a review from vstinner November 11, 2025 00:05
Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

LGTM. Let's check on buildbots.

@vstinner vstinner added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Nov 11, 2025
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @vstinner for commit 3f6b0ef 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F141107%2Fmerge

If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Nov 11, 2025
@skirpichev
Copy link
Contributor Author

We have no RISC-V buildbot, maybe @stefanor can assist with testing? (Though, I don't expect surprises here.)

@skirpichev
Copy link
Contributor Author

failures:

@stefanor

This comment was marked as resolved.

@stefanor
Copy link
Contributor

stefanor commented Nov 11, 2025

FWIW, this branch passes test_capi tests on hppa

@skirpichev
Copy link
Contributor Author

@stefanor, thanks. I think now it should be ok - I forgot to move around code, that "fix" NaN's type.

@stefanor
Copy link
Contributor

Yes, that now passes on riscv64.

@vstinner vstinner merged commit 23d85a2 into python:main Nov 12, 2025
46 checks passed
@miss-islington-app
Copy link

Thanks @skirpichev for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 12, 2025
…ythonGH-141107)

(cherry picked from commit 23d85a2)

Co-authored-by: Sergey B Kirpichev <skirpichev@gmail.com>
@bedevere-app
Copy link

bedevere-app bot commented Nov 12, 2025

GH-141459 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.14 bugs and security fixes label Nov 12, 2025
@vstinner
Copy link
Member

Merged. Thanks for the fix.

vstinner pushed a commit that referenced this pull request Nov 12, 2025
…GH-141107) (#141459)

gh-141042: fix sNaN's packing for mixed floating-point formats (GH-141107)
(cherry picked from commit 23d85a2)

Co-authored-by: Sergey B Kirpichev <skirpichev@gmail.com>
@skirpichev skirpichev deleted the fix-mixed-packing/141042 branch November 13, 2025 13:18
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.

4 participants