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.

Sign up for GitHub

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

Jump to bottom

Deoptimization reasons #18

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

Deoptimization reasons #18

wants to merge 2 commits into from

Conversation

P0lip
Copy link
Collaborator

@P0lip P0lip commented Jul 23, 2017

WIP

To be changed:

  • Format reasons in the old way (without AccessCheck, Hole etc.)
  • Add/port a few common examples

P0lip
P0lip commented Jul 23, 2017
View reviewed changes
README.md Outdated
# V8 bailout reasons
# V8 bailout/deopts reasons

A list of Crankshaft/Ignition+TF bailout/deopts reasons with examples, explanations and advices.
Copy link
Collaborator Author

@P0lip P0lip Jul 23, 2017 •
edited

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No idea how to go around this, any clue? I think it should be written in a more clear and meaningful way.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A list of V8 bailout/deopts reasons with examples, explanations and advices. Includes both the former optimizing compiler Crankshaft and the current compilation infrastructure Ignition + TurboFan.

What about this?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds reasonable 👍

@vsemozhetbyt
Copy link
Contributor

This can help: https://www.nearform.com/blog/node-js-is-getting-a-new-v8-with-turbofan/

@P0lip
Copy link
Collaborator Author

P0lip commented Aug 2, 2017

Yes, the article is quite good.
I've just added a few reasons, though still plenty are about to come.
I will tell you once it's done :)

@vhf
Copy link
Owner

vhf commented Aug 2, 2017

Awesome, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vhf vhf vhf left review comments

Assignees
No one assigned
Labels
enhancement question
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@P0lip @vsemozhetbyt @vhf