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

add # of plugin visits #93

Merged
merged 1 commit into from
Aug 25, 2016
Merged

add # of plugin visits #93

merged 1 commit into from
Aug 25, 2016

Conversation

hzoo
Copy link
Member

@hzoo hzoo commented Aug 25, 2016

Not actually sure how useful this is anymore but figured I'd post it anyway.

./scripts/plugin-timing.js ./scripts/fixtures/jquery.js
pluginAlias                             time(ms) # visits time/visit(ms)
minify-simplify                         2461.853 53786    0.046
minify-mangle-names                     1785.353 1        1785.353
minify-constant-folding                 776.446  124070   0.006
minify-guarded-expressions              358.347  11517    0.031
minify-dead-code-elimination            250.410  1        250.410
minify-infinity                         109.801  56557    0.002
internal.shadowFunctions                105.693  57862    0.002
transform-undefined-to-void             99.388   56557    0.002
minify-type-constructors                59.929   7575     0.008
transform-simplify-comparison-operators 47.390   4845     0.010
transform-member-expression-literals    32.877   17182    0.002
minify-flip-comparisons                 17.091   4845     0.004
transform-merge-sibling-variables       7.737    1748     0.004
internal.blockHoist                     6.170    2484     0.002
transform-property-literals             4.608    1336     0.003
transform-minify-booleans               3.899    243      0.016
minify-replace                          0.186    1        0.186

@kangax
Copy link
Member

kangax commented Aug 25, 2016

I suppose it doesn't hurt :)

@kangax kangax merged commit 63ff75d into master Aug 25, 2016
@kangax kangax deleted the track-visits branch August 25, 2016 22:48
@hzoo
Copy link
Member Author

hzoo commented Aug 25, 2016

Also wow that's a lot of visits

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

Successfully merging this pull request may close these issues.

None yet

2 participants