diff options
author | Shigeki Ohtsu <ohtsu@ohtsu.org> | 2018-04-23 15:26:28 +0300 |
---|---|---|
committer | James M Snell <jasnell@gmail.com> | 2018-04-23 21:45:29 +0300 |
commit | b935e6371008aa508d1a05b67ef0f6e8b8f4d55c (patch) | |
tree | b020902f36d0667f442f8851cf4ba48b3e5111dd /BUILDING.md | |
parent | adc3e8ad87436c86702f88726f29e651226ba9cd (diff) |
build: limit assembler version check on x86
The current openssl checks assembler version only x86_64 or ia32
target arch for use of AES-NI, AVX and AVX2.
This requires --openssl-no-asm option during configure when an older
assembler version is found only on x86_64 or ia32.
PR-URL: https://github.com/nodejs/node/pull/20226
Fixes: https://github.com/nodejs/node/issues/19944
Refs: https://github.com/nodejs/node/pull/20217
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Diffstat (limited to 'BUILDING.md')
-rw-r--r-- | BUILDING.md | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/BUILDING.md b/BUILDING.md index 618427eaefb..9690e826c39 100644 --- a/BUILDING.md +++ b/BUILDING.md @@ -94,14 +94,16 @@ Depending on host platform, the selection of toolchains may vary. #### OpenSSL asm support OpenSSL-1.1.0 requires the following asssembler version for use of asm -support. +support on x86_64 and ia32. * gas (GNU assembler) version 2.23 or higher * xcode version 5.0 or higher * llvm version 3.3 or higher * nasm version 2.10 or higher in Windows -Otherwise, `--openssl-no-asm` is added with warning in configure. +Otherwise `configure` will fail with an error. This can be avoided by +either providing a newer assembler as per the list above or by +using the `--openssl-no-asm` flag. *Note:* The forthcoming OpenSSL-1.1.1 will require higher version. Please refer |