20240910-configure-enable-all-asm-and-sp-cleanup #7967
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
configure.ac
:--enable-all-asm
(enablessp-asm
and eitherintelasm
orarmasm
), and add it to--enable-all-crypto
.--enable-all-asm
in--enable-all-crypto
enable-all-crypto
settings in one place (--enable-all
now simply activates--enable-all-crypto
).enable-all
andenable-all-crypto
setup, correctly conditionalize several FIPS-v6-only features/algorithms.wolfcrypt/src/siphash.c
: fixes forbugprone-macro-parentheses
.wolfcrypt/src/sp_int.c
: fixbugprone-too-small-loop-variable
in_sp_mul()
.SP asm: fixes for several
bugprone-macro-parentheses
and-Wconversion
s.tested with
wolfssl-multi-test.sh ... super-quick-check quantum-safe-wolfssl-all-cppcheck clang-tidy-all-sp-all quantum-safe-wolfssl-all-clang-tidy quantum-safe-wolfssl-all-no-asm-clang-tidy quantum-safe-wolfssl-all-cross-aarch64-armasm-unittest-sanitizer quantum-safe-wolfssl-all-cross-armv7a-armasm-unittest
with--enable-all[-crypto]
updated to enable sp-asm and intelasm/armasm on full Gnu C targets.review+merge with https://github.com/wolfSSL/scripts/pull/423