Skip to content

Improve examples #16

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jesuisYves
Copy link

The final log in 2-benchmark has been changed.
Comma only appeared in cases where there were a few optimisations in a single opt call:
closureInstance....... 330061782 no preopt, interp interp opt, turbofan

An arrow has been added to distinct optimization steps in the final log:
closureInstance....... 213392979 no preopt -> interp -> interp -> opt, turbofan

closureInstance in 3-instantiation has been fixed. Previously it returned a function instead of an object instance.

In 4-array data1 elements didn't have age property, so the query filter always failed.
This issue has already been addressed in #15, but I believe it is better to create a separate prototype for objects rather than using the same one as for arrays.

Comma only appeared in cases where there were a few optimisations in a single `opt' call:
`closureInstance....... 330061782 no preopt, interp interp opt, turbofan`

An arrow has been added to distinct optimization steps in the final log:
`closureInstance....... 213392979 no preopt -> interp -> interp -> opt, turbofan`
`closureInstance` used to return a function instead of an object instance
Assigned the prototype with `age` property for `data1` elements
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.

1 participant