summaryrefslogtreecommitdiff
path: root/BUILDING.md
diff options
context:
space:
mode:
authorShigeki Ohtsu <ohtsu@ohtsu.org>2018-04-23 21:26:28 +0900
committerJames M Snell <jasnell@gmail.com>2018-04-23 11:43:18 -0700
commit3bcd8576fcb800077c922233da251a2299b66923 (patch)
tree79c8a8bc489676b26eb7d85c18842783b354ae88 /BUILDING.md
parentde96899f239f4b5cda5a597f70d871a95e6f9ac3 (diff)
downloadandroid-node-v8-3bcd8576fcb800077c922233da251a2299b66923.tar.gz
android-node-v8-3bcd8576fcb800077c922233da251a2299b66923.tar.bz2
android-node-v8-3bcd8576fcb800077c922233da251a2299b66923.zip
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.md6
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