From: Michaël Zasso Date: Fri, 28 Aug 2015 07:02:32 +0000 (+0200) Subject: src: re-enable fast math on arm X-Git-Tag: v4.0.0~25 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=dd3f3417c7dcec0603a7fab2341c2d055a810653;p=platform%2Fupstream%2Fnodejs.git src: re-enable fast math on arm Ref: https://github.com/nodejs/node/issues/1376 Ref: https://github.com/nodejs/node/pull/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: https://github.com/nodejs/node/pull/2592 Reviewed-By: Rod Vagg Reviewed-By: Ben Noordhuis Reviewed-By: Ali Ijaz Sheikh --- diff --git a/src/node.cc b/src/node.cc index 13e517f..46ce047 100644 --- a/src/node.cc +++ b/src/node.cc @@ -3603,20 +3603,6 @@ void Init(int* argc, DispatchDebugMessagesAsyncCallback); uv_unref(reinterpret_cast(&dispatch_debug_messages_async)); -#if defined(__ARM_ARCH_6__) || \ - defined(__ARM_ARCH_6J__) || \ - defined(__ARM_ARCH_6K__) || \ - defined(__ARM_ARCH_6M__) || \ - defined(__ARM_ARCH_6T2__) || \ - defined(__ARM_ARCH_6ZK__) || \ - defined(__ARM_ARCH_6Z__) - // See https://github.com/nodejs/node/issues/1376 - // and https://code.google.com/p/v8/issues/detail?id=4019 - // TODO(bnoordhuis): Remove test/parallel/test-arm-math-exp-regress-1376.js - // and this workaround when v8:4019 has been fixed and the patch back-ported. - V8::SetFlagsFromString("--nofast_math", sizeof("--nofast_math") - 1); -#endif - #if defined(NODE_V8_OPTIONS) // Should come before the call to V8::SetFlagsFromCommandLine() // so the user can disable a flag --foo at run-time by passing