build: re-enable openssl arm for arm64
In #23913 it looked like arm64 testing was flaky, and as a result a number of changes were made including disabling openssl asm for that architecture. This PR is limited to re-enabling the one change to turn on openssl asm for arm64, in the hopes that either it works now, or that this specific change introduces a reproducible condition. See: https://github.com/nodejs/node/pull/23913 PR-URL: https://github.com/nodejs/node/pull/28180 Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Sam Roberts <vieuxtech@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com>
This commit is contained in:
parent
43e5478e2f
commit
16f673ebcc
@ -28,6 +28,7 @@
|
||||
'clang%': 0,
|
||||
|
||||
'openssl_fips%': '',
|
||||
'openssl_no_asm%': 0,
|
||||
|
||||
# Some STL containers (e.g. std::vector) do not preserve ABI compatibility
|
||||
# between debug and non-debug mode.
|
||||
@ -78,12 +79,6 @@
|
||||
##### end V8 defaults #####
|
||||
|
||||
'conditions': [
|
||||
['target_arch=="arm64"', {
|
||||
# Disabled pending https://github.com/nodejs/node/issues/23913.
|
||||
'openssl_no_asm%': 1,
|
||||
}, {
|
||||
'openssl_no_asm%': 0,
|
||||
}],
|
||||
['OS == "win"', {
|
||||
'os_posix': 0,
|
||||
'v8_postmortem_support%': 0,
|
||||
|
Loading…
x
Reference in New Issue
Block a user