Skip to content

Expect stable_features lint for stable SIMD types#2584

Merged
joshlf merged 1 commit intomainfrom
I7772b004a30ead788978e194181306fb98e092fa
Jun 10, 2025
Merged

Expect stable_features lint for stable SIMD types#2584
joshlf merged 1 commit intomainfrom
I7772b004a30ead788978e194181306fb98e092fa

Conversation

@joshlf
Copy link
Member

@joshlf joshlf commented Jun 10, 2025

Previously, x86 AVX-12 SIMD types were gated by the unstable
stdarch_x86_avx512 feature. As of this writing, that feature has been
stabilized on 1.89.0-nightly, staged for inclusion in 1.89.0 when it is
released as stable.

This means that the simd-nightly Cargo feature, which gates
stdarch_x86_avx512, will now cause the stable_features lint to
trigger when compiling on x86.

This commit enables expect(stable_features) when compiling with
simd-nightly for x86. Once 1.89.0 is released as stable, we will
follow up with a commit which uses version detection to enable support
for AVX-12 types, and we will remove our use of the stdarch_x86_avx512
feature entirely.

Makes progress on #2583


This PR is on branch simd-nightly-avx12.

Previously, x86 AVX-12 SIMD types were gated by the unstable
`stdarch_x86_avx512` feature. As of this writing, that feature has been
stabilized on 1.89.0-nightly, staged for inclusion in 1.89.0 when it is
released as stable.

This means that the `simd-nightly` Cargo feature, which gates
`stdarch_x86_avx512`, will now cause the `stable_features` lint to
trigger when compiling on x86.

This commit enables `expect(stable_features)` when compiling with
`simd-nightly` for x86. Once 1.89.0 is released as stable, we will
follow up with a commit which uses version detection to enable support
for AVX-12 types, and we will remove our use of the `stdarch_x86_avx512`
feature entirely.

Makes progress on #2583

gherrit-pr-id: I7772b004a30ead788978e194181306fb98e092fa
@joshlf joshlf force-pushed the I7772b004a30ead788978e194181306fb98e092fa branch from fe8b242 to d2ba384 Compare June 10, 2025 14:47
@joshlf joshlf requested a review from jswrenn June 10, 2025 14:47
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.91%. Comparing base (8fbfac9) to head (d2ba384).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2584   +/-   ##
=======================================
  Coverage   88.91%   88.91%           
=======================================
  Files          20       20           
  Lines        5312     5312           
=======================================
  Hits         4723     4723           
  Misses        589      589           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@joshlf joshlf added this pull request to the merge queue Jun 10, 2025
Merged via the queue into main with commit 053a5eb Jun 10, 2025
89 checks passed
@joshlf joshlf deleted the I7772b004a30ead788978e194181306fb98e092fa branch June 10, 2025 15:54
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