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: Make GeckoElement::has_class faster. #19787
Conversation
I've verified that current libxul, at least for Linux, doesn't manage to pull the branch out.
Heads up! This PR modifies the following files:
|
@bors-servo r+ |
📌 Commit c44e038 has been approved by |
style: Make GeckoElement::has_class faster. By force-inlining the fast path, and pulling out a branch that rust didn't manage to pull out. <!-- 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/19787) <!-- Reviewable:end -->
💔 Test failed - mac-dev-unit |
It's easier, and probably faster, than trying to inline the function call to Gecko. This symbol is in libxul.so as of right now.
@bors-servo r=bholley
|
📌 Commit e55bb3e has been approved by |
style: Make GeckoElement::has_class faster. By force-inlining the fast path, and pulling out a branch that rust didn't manage to pull out. <!-- 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/19787) <!-- 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 |
By force-inlining the fast path, and pulling out a branch that rust didn't manage to pull out.
This change is