Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix compiler/linker support of SIMD instructions with -Wall #94

Merged
merged 10 commits into from
Aug 17, 2022

Conversation

emollier
Copy link
Contributor

@emollier emollier commented May 1, 2022

When -Wall is in use, the various compiler and linker tests related to SIMD extensions issue warnings about unused or unitialized variables detected as errors by autoconf. This notably crashes the ./configure step when an attempt of enforcing them with e.g. --enable-avx512=yes is made. This patch adjusts the source code compiled and linked to not issue these warnings anymore.

Affected SIMD extensions are avx512, altivec and neon. In the case of neon, but that's probably Debian specific since a different version of SIMDe is in use, getting the vectorization supported led to failure to build from source, so this patch is in a separate commit in case a revert were to be necessary.

In hope this helps,
Have a nice day, :)
Étienne.

jeffdaily and others added 10 commits February 26, 2020 18:32
When -Wall is in use, the various avx512 compiler tests issue
warnings which are detected as errors by autoconf.  This notably
crashes the ./configure step when an attempt of enforcing avx512
support is made.  This patch adjusts the source code compiled and
linked to not issue these warnings anymore.

Signed-off-by: Étienne Mollier <emollier@debian.org>
Similarly to previous commit concerning avx512, this patch enables
altivec support for powerpc hosts.

Signed-off-by: Étienne Mollier <emollier@debian.org>
Similarly to avx512 and Altivec, this fixes detection of neon support
by the compiler and linker.  These are provided as separated commits,
since the Neon patch shown afterwards, possibly specifically to
Debian, that parasail fails to compile with errors in the SIMDe layer.

Signed-off-by: Étienne Mollier <emollier@debian.org>
@jeffdaily jeffdaily changed the base branch from master to develop May 2, 2022 15:04
@jeffdaily jeffdaily merged commit d1a0447 into jeffdaily:develop Aug 17, 2022
@emollier emollier deleted the vectorization branch December 18, 2022 17:23
This pull request was closed.
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.

2 participants