diff options
| author | Andrey Matyukov <andrey.matyukov@intel.com> | 2021-03-24 10:05:29 +0300 |
|---|---|---|
| committer | Tomas Mraz <tomas@openssl.org> | 2021-03-30 19:05:35 +0200 |
| commit | 788a72e92fc0b78c2ca12df90e45cccbaf7a800e (patch) | |
| tree | 017633f8b8bf5bd5863906c13edb6ec112018772 | |
| parent | 1ee1a169c310f3dc735a3d83298686cf70aa427a (diff) | |
Increase minimum clang version requirement for rsaz-avx512.pl
The reason is that clang-6 does not enable proper -march flags by
default for assembly modules (rsaz-avx512.pl requires avx512ifma, avx512dq,
avx512vl, avx512f). This is not true for newer clang versions - clang-7 and
further work ok.
For older clang versions users who want to get optimization from this
file, we have a note in the OPENSSL_ia32cap.pod with the workaround that
proposes having a wrapper that forces using external assembler.
Fixes #14668: clang-6.0.0 build broken
Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/14671)
| -rw-r--r-- | crypto/bn/asm/rsaz-avx512.pl | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crypto/bn/asm/rsaz-avx512.pl b/crypto/bn/asm/rsaz-avx512.pl index 04effabffa..d22eeef5da 100644 --- a/crypto/bn/asm/rsaz-avx512.pl +++ b/crypto/bn/asm/rsaz-avx512.pl @@ -49,7 +49,7 @@ if (!$avx512 && $win64 && ($flavour =~ /nasm/ || $ENV{ASM} =~ /nasm/) && } if (!$avx512 && `$ENV{CC} -v 2>&1` =~ /((?:clang|LLVM) version|.*based on LLVM) ([0-9]+\.[0-9]+)/) { - $avx512ifma = ($2>=6.0); + $avx512ifma = ($2>=7.0); } open OUT,"| \"$^X\" \"$xlate\" $flavour \"$output\"" |