-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Do not coalesce unspillable tmps #10742
Do not coalesce unspillable tmps #10742
Conversation
EWS run on previous version of this PR (hash 1a17163) |
1a17163
to
3470cc3
Compare
EWS run on previous version of this PR (hash 3470cc3) |
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.
r=me
3470cc3
to
ef2c550
Compare
Perf is neutral on JS2 AS, landing |
EWS run on current version of this PR (hash ef2c550) |
https://bugs.webkit.org/show_bug.cgi?id=253006 rdar://105878987 Reviewed by Yusuke Suzuki. We have been encountering multiple test cases where we make a graph uncolourable by coalescing too aggressively. If a tmp is marked as unspillable because its live range is too small, it will never be un-marked if it gets coalesced. Let's try not coalescing these registers ever. * Source/JavaScriptCore/b3/air/AirAllocateRegistersByGraphColoring.cpp: Canonical link: https://commits.webkit.org/260969@main
ef2c550
to
c83189e
Compare
Committed 260969@main (c83189e): https://commits.webkit.org/260969@main Reviewed commits have been landed. Closing PR #10742 and removing active labels. |
c83189e
ef2c550
π§ͺ ios-wk2π§ͺ api-macπ§ͺ gtk-wk2π§ͺ api-iosπ§ͺ api-gtkπ π§ͺ jscπ§ͺ mac-wk2π§ͺ mac-AS-debug-wk2