Skip to content

feature detection - #187

Merged
odraccir1 merged 1 commit into
masterfrom
ra/small_prime_fft_cmake
Oct 28, 2026
Merged

odraccir1 merged 1 commit into
masterfrom
ra/small_prime_fft_cmake

Conversation

@odraccir1

@odraccir1 odraccir1 commented Oct 28, 2026 •

Copy link
Copy Markdown
Collaborator

Ordinary CMake builds could leave existing small-prime FFT and CRT/export optimizations disabled, even on supported hardware. The kernels require AVX2/FMA and, for the eight-lane paths, AVX512F/DQ. The previous configuration did not consistently propagate these complete requirements to downstream targets linking crypto3::math.

This change:

• Reuses CPU/OS and compiler checks to enable complete supported instruction bundles.
• Exports the required flags through crypto3::math, keeping IFMA detection independent of DQ.
• Refreshes feature detection during reconfiguration and preserves unsupported-host and cross-compilation fallbacks.
• Documents configuration requirements and binary portability limits.
• Adds opt-in consumer tests checking actual kernel-selection macros and transitive flags. Numerical correctness tests alone would not detect silently disabled optimizations.

@odraccir1
odraccir1 requested a review from spaceships October 28, 2026 21:49
@odraccir1
odraccir1 merged commit 1509ea6 into master Oct 28, 2026
2 checks passed
@odraccir1
odraccir1 deleted the ra/small_prime_fft_cmake branch October 28, 2026 21:59
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.

1 participant