From 6c1b26a12e2ef3b72062e074b025592dae88a6ce Mon Sep 17 00:00:00 2001 From: Ali Ijaz Sheikh Date: Sun, 23 Aug 2015 12:53:05 -0700 Subject: [PATCH] src: enable vector ics on arm again The flag is no longer supported by V8 4.5, and the original issue [1] on ARMv6 no longer manifests with (at least) 4.5.103.20. [1] See https://code.google.com/p/v8/issues/detail?id=4338 PR-URL: https://github.com/nodejs/node/pull/2509 Reviewed-By: Ben Noordhuis --- src/node.cc | 5 ----- 1 file changed, 5 deletions(-) diff --git a/src/node.cc b/src/node.cc index f27541bc32394a..13e517f8de568d 100644 --- a/src/node.cc +++ b/src/node.cc @@ -3615,11 +3615,6 @@ void Init(int* argc, // 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); - // See https://github.com/nodejs/node/pull/2220#issuecomment-126200059 - // and https://code.google.com/p/v8/issues/detail?id=4338 - // TODO(targos): Remove this workaround when v8:4338 has been fixed and the - // patch back-ported. - V8::SetFlagsFromString("--novector_ics", sizeof("--novector_ics") - 1); #endif #if defined(NODE_V8_OPTIONS)