On 2015/08/13 18:04:50, adamk wrote:
The CL description says this builtin is "useless", but the fast path for
arrays
was added as a fix for a performance bug:
https://code.google.com/p/v8/issues/detail?id=3701
If this removal is part of a broader effort to improve the handling
of 'in',
that's probably fine. Just wanted to call out that this fast path is
there for
a
reason.
Thanks a lot for the pointer. I kept the fast path in the runtime, and will
measure with
the microbenchmark whether it's worth keeping a fast path that bypasses the
runtime.
https://codereview.chromium.org/1295433002/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
