https://codereview.chromium.org/947683002/diff/140001/src/collection.js
File src/collection.js (right):

https://codereview.chromium.org/947683002/diff/140001/src/collection.js#newcode462
src/collection.js:462: $MapGet = MapGet;
This is where I copy MapGet into the global scope.

https://codereview.chromium.org/947683002/diff/140001/src/harmony-templates.js
File src/harmony-templates.js (right):

https://codereview.chromium.org/947683002/diff/140001/src/harmony-templates.js#newcode24
src/harmony-templates.js:24: var obj = %_CallFunction(callSiteCache,
hash, $MapGet);
On 2015/03/25 15:46:24, caitp wrote:
On 2015/03/25 15:43:06, adamk wrote:
> On 2015/03/25 12:56:27, caitp wrote:
> > Couldn't we just call MapGet() (and others) directly here?
>
> MapGet is now inside a closure, so it's not visible here.

I mean, you have a reference to $MapGet, otherwise %_CallFunction()
wouldn't
work either.

Not sure what you're saying. I could have called "$MapGet" "MapGet"
instead, but $MapGet seemed a little clearer to me.

https://codereview.chromium.org/947683002/

--
--
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