build: disable openssl asm on arm64 for now

bnoordhuis · kiyomizumia · commit 6f512bf048bf · 2018-11-15T21:18:26.000+09:00
There is reason to believe the generated assembly isn't working correctly so let's disable it for now pending further investigation. PR-URL: nodejs#24270 Refs: nodejs#23913 Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: Refael Ackermann <refack@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/common.gypi b/common.gypi
@@ -27,7 +27,6 @@
 
     'clang%': 0,
 
-    'openssl_no_asm%': 0,
     'openssl_fips%': '',
 
     # Reset this number to 0 on major V8 upgrades.
@@ -52,6 +51,12 @@
     'icu_use_data_file_flag%': 0,
 
     'conditions': [
+      ['target_arch=="arm64"', {
+        # Disabled pending https://github.com/nodejs/node/issues/23913.
+        'openssl_no_asm%': 1,
+      }, {
+        'openssl_no_asm%': 0,
+      }],
       ['GENERATOR=="ninja"', {
         'obj_dir': '<(PRODUCT_DIR)/obj',
         'conditions': [
diff --git a/configure.py b/configure.py
@@ -1183,9 +1183,11 @@ def configure_openssl(o):
   variables = o['variables']
   variables['node_use_openssl'] = b(not options.without_ssl)
   variables['node_shared_openssl'] = b(options.shared_openssl)
-  variables['openssl_no_asm'] = 1 if options.openssl_no_asm else 0
   variables['openssl_fips'] = ''
 
+  if options.openssl_no_asm:
+    variables['openssl_no_asm'] = 1
+
   if options.without_ssl:
     def without_ssl_error(option):
       error('--without-ssl is incompatible with %s' % option)
diff --git a/deps/openssl/openssl.gyp b/deps/openssl/openssl.gyp
@@ -1,7 +1,4 @@
 {
-  'variables': {
-    'openssl_no_asm%': 0,
-  },
   'targets': [
     {
       'target_name': 'openssl',

Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,4 @@`
`1`	`1`	`{`
`2`		`- 'variables': {`
`3`		`- 'openssl_no_asm%': 0,`
`4`		`- },`
`5`	`2`	`'targets': [`
`6`	`3`	`{`
`7`	`4`	`'target_name': 'openssl',`