On 2014/11/19 21:38:57, arv wrote:
https://codereview.chromium.org/742643003/diff/60001/src/harmony-templates.js
File src/harmony-templates.js (right):
https://codereview.chromium.org/742643003/diff/60001/src/harmony-templates.js#newcode39
src/harmony-templates.js:39: %MapSet(callSiteCache, hash, new
InternalArray(siteObj));
Lets not use the strange overloading of params for the array constructor.
var array = new InternalArray(1);
array[0] = siteObj;
Done --- I think the "." in the hash string separators can be removed too,
will
do that in a sec.
There's a TODO in the tests above a failing test, which can be fixed easily
once
multi-byte handling is fixed.
https://codereview.chromium.org/742643003/
--
--
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.