diff options
| author | Shigeki Ohtsu <ohtsu@ohtsu.org> | 2018-04-23 21:26:28 +0900 |
|---|---|---|
| committer | James M Snell <jasnell@gmail.com> | 2018-04-23 11:43:18 -0700 |
| commit | 3bcd8576fcb800077c922233da251a2299b66923 (patch) | |
| tree | 79c8a8bc489676b26eb7d85c18842783b354ae88 /BUILDING.md | |
| parent | de96899f239f4b5cda5a597f70d871a95e6f9ac3 (diff) | |
| download | node-new-3bcd8576fcb800077c922233da251a2299b66923.tar.gz | |
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 618427eaef..9690e826c3 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 |