]> git.proxmox.com Git - mirror_edk2.git/commit - CryptoPkg/Library/OpensslLib/OpensslLib.inf
CryptoPkg/OpensslLib AARCH64: clear XIP CC flags
authorArd Biesheuvel <ard.biesheuvel@linaro.org>
Fri, 14 Jul 2017 17:09:03 +0000 (18:09 +0100)
committerArd Biesheuvel <ard.biesheuvel@linaro.org>
Sat, 15 Jul 2017 12:36:58 +0000 (13:36 +0100)
commite38eb2595b86434f2d488b0d575af71f07b6428f
tree1b3a7529a3c375e5b94664128953fdfd59d3f8a3
parent0df6c8c157af9510e21bff7bb8aa1f461d04707b
CryptoPkg/OpensslLib AARCH64: clear XIP CC flags

Commit 0df6c8c157af ("BaseTools/tools_def AARCH64: avoid SIMD registers
in XIP code") updated the compiler flags used by AARCH64 when building
modules (including BASE libraries) that may execute before the MMU is
enabled.

This broke the build for OpensslLib/OpensslLibCrypto because the SIMD
register file is shared with the FPU, and since OpenSSL contains some
references to float/double types (which are mostly unused for UEFI btw),
disabling floating point prevents the compiler from building OpenSSL
at all. So for OpensslLib[Crypto], we need to override the XIP CC flags,
to remove the -mgeneral-regs-only compiler flag again.

When introducing the support for XIP CC flags, we were aware that this
would affect BASE libraries as well, but were not expecting this to
have any performance impact. However, in the case of software crypto,
it makes sense not to needlessly inhibit the compiler's ability to
generate fast code, and even if OpenssLib is a BASE library, it is
guaranteed not to run with the MMU off. So omit -mstrict-align from the
local XIP CC flags override as well.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
Acked-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Long Qin <qin.long@intel.com>
CryptoPkg/Library/OpensslLib/OpensslLib.inf
CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf