-
-
Notifications
You must be signed in to change notification settings - Fork 669
Slightly improve pass pipeline #1498
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
Conversation
@@ -1656,7 +1655,7 @@ export class Module { | |||
passes.push("simplify-locals"); | |||
passes.push("vacuum"); | |||
|
|||
passes.push("inlining-optimizing"); | |||
passes.push("inlining"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just curious: What's the different between inlining
and inlining-optimizing
? Is the latter unnecessary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems inlining-optimizing
also apply constant propagation but only for specific part or something like this
Great news! Time of bootstrap compilation reduced from |
🎉 This PR is included in version 0.15.2 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Move
code-folding
little bit lower in pipeline.Remove and simplify some inline passes.
All this changes slightly improve codegen quality and increase optimization speed