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.

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

[JSC] Use std::span::data in super hot function #28262

Conversation

@Constellation Constellation self-assigned this May 8, 2024
@Constellation Constellation requested a review from a team as a code owner May 8, 2024 00:03
@Constellation Constellation added the JavaScriptCore For bugs in JavaScriptCore, the JS engine used by WebKit, other than kxmlcore issues. label May 8, 2024
Copy link
Contributor

@kmiller68 kmiller68 left a comment

Choose a reason for hiding this comment

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

r=me

@Constellation Constellation added the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label May 8, 2024
https://bugs.webkit.org/show_bug.cgi?id=273852
rdar://127703218

Reviewed by Keith Miller.

With 277836@main and 278441@main, we observed large regression in JetStream2/cdjs.
This patch attempts to recover that regression.

* Source/JavaScriptCore/runtime/IntlObjectInlines.h:
(JSC::compareASCIIWithUCADUCETLevel3):
(JSC::compareASCIIWithUCADUCET):

Canonical link: https://commits.webkit.org/278486@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/JSC-Use-stdspandata-in-super-hot-function branch from 7767ef8 to ed761c3 Compare May 8, 2024 01:54
@webkit-commit-queue
Copy link
Collaborator

Committed 278486@main (ed761c3): https://commits.webkit.org/278486@main

Reviewed commits have been landed. Closing PR #28262 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit ed761c3 into WebKit:main May 8, 2024
@webkit-commit-queue webkit-commit-queue removed the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label May 8, 2024
@Constellation Constellation deleted the eng/JSC-Use-stdspandata-in-super-hot-function branch May 8, 2024 03:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
JavaScriptCore For bugs in JavaScriptCore, the JS engine used by WebKit, other than kxmlcore issues.
Projects
None yet
4 participants