Skip to content

MAINT: Use VQSORT_COMPILER_COMPATIBLE to determine if we should use VQSort#28202

Merged
ngoldbaum merged 1 commit intonumpy:mainfrom
Mousius:vqsort-compiler-check
Jan 21, 2025
Merged

MAINT: Use VQSORT_COMPILER_COMPATIBLE to determine if we should use VQSort#28202
ngoldbaum merged 1 commit intonumpy:mainfrom
Mousius:vqsort-compiler-check

Conversation

@Mousius
Copy link
Copy Markdown
Member

@Mousius Mousius commented Jan 20, 2025

Previously we copied the compiler detection across from the Highway header, now we don't have to manually keep it up to date.

Previously we copied the compiler detection across from the Highway header, now we don't have to manually keep it up to date.
@ngoldbaum
Copy link
Copy Markdown
Member

I can build NumPy with both TSAN and ASAN on this branch, so I think this should be working as intended.

Thanks so much for getting this fixed upstream and then updated here.

@ngoldbaum ngoldbaum merged commit 1af7cb3 into numpy:main Jan 21, 2025
@charris
Copy link
Copy Markdown
Member

charris commented Jan 21, 2025

I am going to backport this for the highway update, see #28030.

@charris charris added the 09 - Backport-Candidate PRs tagged should be backported label Jan 21, 2025
@charris charris removed the 09 - Backport-Candidate PRs tagged should be backported label Jan 23, 2025
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.

3 participants