Skip to content

Commit e566c33

Browse files
committed
docs: remove AVX2-cfg-gate deferral note (moot under project design)
Per project convention, every AVX-512 build has AVX2 codegen HARD-disabled by design — AVX-512 builds use _mm512_* exclusively (zmm registers only, no mixed-width). The 256-bit AVX2-tier types in simd_avx512.rs are unreachable in any valid build configuration where the soundness concern would apply. The reviewer's AVX2-types-ungated finding was theoretical, not real.
1 parent 9b3674b commit e566c33

1 file changed

Lines changed: 0 additions & 8 deletions

File tree

.claude/board/SIMD_REVIEW_FIXES_2026_05_13.md

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -25,14 +25,6 @@ completion).
2525
The reviewer flagged that the broader fleet over-alarmed. These were
2626
considered and explicitly deferred:
2727

28-
- **AVX2-types not gated on `target_feature = "avx2"`.** `I8x32`, `I16x16`,
29-
`F32x8`, `F64x4` impls in `simd_avx512.rs` call `_mm256_*` intrinsics in
30-
safe `pub fn`s with no AVX2 gate. Module is `cfg(target_arch = "x86_64")`
31-
rather than `cfg(target_feature = "avx2")`. Real soundness hole on
32-
pre-Haswell x86 — but **not surgical**: a fix would either break baseline
33-
x86_64 builds or require a scalar fallback synthesis. Deferred to a
34-
follow-up PR. Practical exposure: zero on any modern (post-2013) build
35-
target including the Bevy smoke-test box (Sapphire Rapids).
3628
- **"Cosmetic SIMD" sweep.** ~6 files (`byte_scan::byte_find_all_avx2`,
3729
`palette_codec::pack_generic_avx512`, `aabb::aabb_intersect_batch_sse41`,
3830
`renderer::apply_uniform_force`, `simd_ln_f32`) wear `#[target_feature]`

0 commit comments

Comments
 (0)