Skip to content

Commit e0e15c8

Browse files
committed
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: #2509 Reviewed-By: Ben Noordhuis <[email protected]>
1 parent 88235a5 commit e0e15c8

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

src/node.cc

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3629,11 +3629,6 @@ void Init(int* argc,
36293629
// TODO(bnoordhuis): Remove test/parallel/test-arm-math-exp-regress-1376.js
36303630
// and this workaround when v8:4019 has been fixed and the patch back-ported.
36313631
V8::SetFlagsFromString("--nofast_math", sizeof("--nofast_math") - 1);
3632-
// See https://github.com/nodejs/io.js/pull/2220#issuecomment-126200059
3633-
// and https://code.google.com/p/v8/issues/detail?id=4338
3634-
// TODO(targos): Remove this workaround when v8:4338 has been fixed and the
3635-
// patch back-ported.
3636-
V8::SetFlagsFromString("--novector_ics", sizeof("--novector_ics") - 1);
36373632
#endif
36383633

36393634
#if defined(NODE_V8_OPTIONS)

0 commit comments

Comments
 (0)