From: Xi Ruoyao Date: Fri, 24 Nov 2023 08:14:05 +0000 (+0800) Subject: Do not define L_ENDIAN (for now) when we guessed linux64-loongarch64 X-Git-Tag: openssl-3.3.0-alpha1~561 X-Git-Url: https://git.openssl.org/gitweb/?p=openssl.git;a=commitdiff_plain;h=d7c0fc5b1a7b5cb2219f8d89a861f3879582fc16 Do not define L_ENDIAN (for now) when we guessed linux64-loongarch64 In 160f48941d14 I made L_ENDIAN defined when the system is guessed to be linux64-loongarch64. Unfortunately now I found it problematic: 1. This should be added into Configurations/10-main.conf, not here. Having it here causes a different configuration when linux64-loongarch64 is explicitly specified than guessed. 2. With LTO enabled, this causes many test failures on linux64-loongarch64 due to #12247. So I think we should remove it for now (master and 3.2 branch), and reintroduce it to Configurations/10-main.conf when we finally sort out #12247. Reviewed-by: Matt Caswell Reviewed-by: Tomas Mraz (Merged from https://github.com/openssl/openssl/pull/22812) --- diff --git a/util/perl/OpenSSL/config.pm b/util/perl/OpenSSL/config.pm index 8125f48b14..cda764b4ed 100755 --- a/util/perl/OpenSSL/config.pm +++ b/util/perl/OpenSSL/config.pm @@ -691,7 +691,6 @@ EOF $disable = []; } return { target => "linux64-loongarch64", - defines => [ 'L_ENDIAN' ], disable => $disable, }; } ],