Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Is the readme correct? #43

Open
kurtextrem opened this issue Dec 9, 2016 · 1 comment
Open

Is the readme correct? #43

kurtextrem opened this issue Dec 9, 2016 · 1 comment

Comments

@kurtextrem
Copy link

https://docs.google.com/presentation/d/1214p4CFjsF-NY4z9in0GEcJtjbyVQgU0A-UqEvovzCs/edit#slide=id.g192721b519_1_40
this suggests V8 doesn't only look for (, but only for (function(){})() (see slide 24)

@HorkyChen
Copy link

HorkyChen commented Apr 18, 2017

That looks like V8 had already finished some optimizations.

Same as this issue, I double checked latest implementation of V8, below pattern will be treated as "likely to be called" and should do eagerly parsing:
!function (){}()

Below is the related review:
Treat a '!' preceding a function literal as eager-compile hint.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants