Skip to content

gh-148474: Fix _Py_hexlify_simd compilation with older clang#148475

Merged
kumaraditya303 merged 4 commits into
python:mainfrom
jmroot:pystrhex
Apr 16, 2026
Merged

gh-148474: Fix _Py_hexlify_simd compilation with older clang#148475
kumaraditya303 merged 4 commits into
python:mainfrom
jmroot:pystrhex

Conversation

@jmroot

@jmroot jmroot commented Apr 12, 2026

Copy link
Copy Markdown
Contributor

Apple clang prior to around Xcode 9 does not automatically generate a vector from a scalar shift operator RHS in this case. Doing it explicitly works with all versions and arguably also makes it clearer what is actually happening.

Apple clang prior to around Xcode 9 does not automatically generate a
vector from a scalar shift operator RHS in this case. Doing it
explicitly works with all versions and arguably also makes it clearer
what is actually happening.
Comment thread Python/pystrhex.c
Comment thread Misc/NEWS.d/next/Build/2026-04-12-22-54-16.gh-issue-148474.ouIO8R.rst Outdated

@picnixz picnixz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

@picnixz

picnixz commented Apr 13, 2026

Copy link
Copy Markdown
Member

I hope that my NEWS entry change is correct (I don't know if we had the c file directives correctly set up)

@kumaraditya303 kumaraditya303 merged commit 5f28e6b into python:main Apr 16, 2026
98 of 100 checks passed
@jmroot jmroot deleted the pystrhex branch April 16, 2026 06:01
ljfp pushed a commit to ljfp/cpython that referenced this pull request Apr 25, 2026
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