src: enable vector ics on arm again
authorAli Ijaz Sheikh <ofrobots@google.com>
Sun, 23 Aug 2015 19:53:05 +0000 (12:53 -0700)
committerRod Vagg <rod@vagg.org>
Sun, 6 Sep 2015 11:39:05 +0000 (21:39 +1000)
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 <info@bnoordhuis.nl>
src/node.cc

index f27541b..13e517f 100644 (file)
@@ -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)