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.

Reply via email to