]> git.ipfire.org Git - thirdparty/openssl.git/commitdiff
Add aix64-gcc-as architecture and p2align callback
authorDaniel Bevenius <daniel.bevenius@gmail.com>
Mon, 7 Jun 2021 04:01:54 +0000 (06:01 +0200)
committerPauli <pauli@openssl.org>
Tue, 8 Jun 2021 08:52:53 +0000 (18:52 +1000)
This commit adds an architecture named aix64-gcc-as which can generate
assembler source code compatible with AIX assembler (as) instead of the
GNU Assembler (gas). This architecture name is then used in a callback
for the .p2align directive which is not available in AIX as.

The motivation for this addition came out of an issue we ran into when
working on upgrading OpenSSL in Node.js. We ran into the following
compilation error on one of the CI machines that uses AIX:

  05:39:05 Assembler:
  05:39:05 crypto/bn/ppc64-mont-fixed.s: line 4: Error In Syntax

This machine is using AIX Version 7.2 and does not have gas installed
and the .p2align directive is causing this error. After asking around if
it would be possible to install GAS on this machine I learned that AIX
GNU utils are not maintained as well as the native AIX ones and we
(Red Hat/IBM) have run into issues with the GNU utils in the past and if
possible it would be preferable to be able to use the AIX native
assembler.

Refs: https://github.com/nodejs/node/pull/38512

Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/15638)

Configurations/10-main.conf
crypto/perlasm/ppc-xlate.pl

index 138ad8a6ae0c2c3b1e5471489b87574d34b7cc4a..dfff5ea4ae6219e020443dee37badd70f1c825c0 100644 (file)
@@ -1212,6 +1212,10 @@ my %targets = (
         AR               => add("-X64"),
         RANLIB           => add("-X64"),
     },
+    "aix64-gcc-as" => {
+        inherit_from     => [ "aix64-gcc" ],
+        perlasm_scheme   => "aix64-as",
+    },
     "aix-cc" => {
         inherit_from     => [ "aix-common" ],
         CC               => "cc",
index 14fd06dec494a4698940d750c62bc13184eea56c..c81fb6a47ac66d1555142cf2d7b1204c02fcd485 100755 (executable)
@@ -83,6 +83,10 @@ my $text = sub {
     $ret = ".abiversion        2\n".$ret       if ($flavour =~ /linux.*64(le|v2)/);
     $ret;
 };
+my $p2align = sub {
+    my $ret = ($flavour =~ /aix64-as/) ? "" : ".p2align $line";
+    $ret;
+};
 my $machine = sub {
     my $junk = shift;
     my $arch = shift;