Skip to content

Commit 439c1dd

Browse files
targosrvagg
authored andcommitted
src: re-enable fast math on arm
Ref: #1376 Ref: #1398 Issue fixed in V8: https://chromium.googlesource.com/v8/v8/+/81703350bbb9923d211fe5b79e90bd458b0916e2 V8-Bug: https://code.google.com/p/v8/issues/detail?id=4019 PR-URL: #2592 Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
1 parent 6c1b26a commit 439c1dd

File tree

1 file changed

+0
-14
lines changed

1 file changed

+0
-14
lines changed

src/node.cc

-14
Original file line numberDiff line numberDiff line change
@@ -3603,20 +3603,6 @@ void Init(int* argc,
36033603
DispatchDebugMessagesAsyncCallback);
36043604
uv_unref(reinterpret_cast<uv_handle_t*>(&dispatch_debug_messages_async));
36053605

3606-
#if defined(__ARM_ARCH_6__) || \
3607-
defined(__ARM_ARCH_6J__) || \
3608-
defined(__ARM_ARCH_6K__) || \
3609-
defined(__ARM_ARCH_6M__) || \
3610-
defined(__ARM_ARCH_6T2__) || \
3611-
defined(__ARM_ARCH_6ZK__) || \
3612-
defined(__ARM_ARCH_6Z__)
3613-
// See https://github.com/nodejs/node/issues/1376
3614-
// and https://code.google.com/p/v8/issues/detail?id=4019
3615-
// TODO(bnoordhuis): Remove test/parallel/test-arm-math-exp-regress-1376.js
3616-
// and this workaround when v8:4019 has been fixed and the patch back-ported.
3617-
V8::SetFlagsFromString("--nofast_math", sizeof("--nofast_math") - 1);
3618-
#endif
3619-
36203606
#if defined(NODE_V8_OPTIONS)
36213607
// Should come before the call to V8::SetFlagsFromCommandLine()
36223608
// so the user can disable a flag --foo at run-time by passing

0 commit comments

Comments
 (0)