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
style: Rework how precomputed pseudo stuff works, to avoid malloc/free churn. #17992
Conversation
Heads up! This PR modifies the following files:
|
r? @heycam |
(or @bholley) |
d15ef6b
to
70dfaf8
Compare
components/style/stylist.rs
Outdated
continue; | ||
} | ||
|
||
self.precomputed_pseudo_element_decls |
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 would be nice if we could handle the precomputed case separately, above the match, so we can can all of the normal rule collection bits together, but it's not a big deal.)
@bors-servo delegate+ Much more sensible, thanks. :-) |
✌️ @emilio can now approve this pull request |
…e churn. This showed up in a few profiles, and was an easy improvement. MozReview-Commit-ID: HVqATaSB2Ak Signed-off-by: Emilio Cobos Álvarez <emilio@crisal.io>
70dfaf8
to
121e2d5
Compare
Yup, agreed, done :) @bors-servo r=heycam |
📌 Commit 121e2d5 has been approved by |
style: Rework how precomputed pseudo stuff works, to avoid malloc/free churn. This showed up in a few profiles, and was an easy improvement. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/17992) <!-- Reviewable:end -->
☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-css2, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, windows-msvc-dev |
This showed up in a few profiles, and was an easy improvement.
This change is