Commit dd3f341
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 e137c11 commit dd3f341
1 file changed
Lines changed: 0 additions & 14 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -3603,20 +3603,6 @@ void Init(int* argc, | |||
| 3603 | 3603 | DispatchDebugMessagesAsyncCallback); | |
| 3604 | 3604 | uv_unref(reinterpret_cast<uv_handle_t*>(&dispatch_debug_messages_async)); | |
| 3605 | 3605 | ||
| 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 | - | ||
| 3620 | 3606 | #if defined(NODE_V8_OPTIONS) | |
| 3621 | 3607 | // Should come before the call to V8::SetFlagsFromCommandLine() | |
| 3622 | 3608 | // so the user can disable a flag --foo at run-time by passing | |
0 commit comments