8000 [3.14] gh-135755: rename undocumented `HACL_CAN_COMPILE_SIMD{128,256}` macros (GH-135847) by picnixz · Pull Request #136045 · python/cpython · GitHub
[go: up one dir, main page]

Skip to content

8000 [3.14] gh-135755: rename undocumented HACL_CAN_COMPILE_SIMD{128,256} macros (GH-135847) #136045

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

Draft
wants to merge 1 commit into
base: 3.14
Choose a base branch
from

Conversation

picnixz
Copy link
Member
@picnixz picnixz commented Jun 27, 2025

Rename undocumented HACL_CAN_COMPILE_SIMD{128,256} macros to _Py_HACL_CAN_COMPILE_VEC{128,256}. These macros are private. (cherry picked from commit 1e975ae)

…8,256}` macros (pythonGH-135847)

Rename undocumented `HACL_CAN_COMPILE_SIMD{128,256}` macros
to `_Py_HACL_CAN_COMPILE_VEC{128,256}`. These macros are private.
(cherry picked from commit 1e975ae)

Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
@picnixz
Copy link
Member Author
picnixz commented Jun 27, 2025

Ok, so blake2module actually changed because of my recent refactor. I'm almost tempted to backport the refactor from blake2 to 3.14 as well because any changes in blake2 will lead to conflicts. OTOH, it's a long rewrite and I think it's better if we make manual backports in the future. I doubt we'll need to touch more those modules (I mean, they are pretty... stable in general).

@picnixz
Copy link
Member Author
picnixz commented Jun 27, 2025

!buildbot FIPS only

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @picnixz for commit 25dabcd 🤖

Results will be shown at:

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

The command will test the builders whose names match following regular expression: FIPS only

The builders matched are:

  • AMD64 RHEL8 FIPS Only Blake2 Builtin Hash PR
  • AMD64 CentOS9 FIPS Only Blake2 Builtin Hash PR

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
0