Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 0bd5df499e70892cd54cc93d617eada3d31ebb59
https://github.com/WebKit/WebKit/commit/0bd5df499e70892cd54cc93d617eada3d31ebb59
Author: Keith Miller <[email protected]>
Date: 2026-03-23 (Mon, 23 Mar 2026)
Changed paths:
M Source/JavaScriptCore/bytecode/DirectEvalCodeCache.h
M Source/JavaScriptCore/interpreter/Interpreter.cpp
M Source/JavaScriptCore/runtime/JSString.h
M Source/JavaScriptCore/runtime/JSStringInlines.h
Log Message:
-----------
[JSC] Remove rope suffix eval cache
https://bugs.webkit.org/show_bug.cgi?id=310499
rdar://173120111
Reviewed by Yusuke Suzuki.
This was added as an optimization for a weird idiom in JS2 that has
since been changed for JS3. The date-format-tofte benchmark would do:
`eval(someFunctionText() + "()")` and so we tried to avoid resolving the
rope for the eval cache.
That benchmark was changed in
https://github.com/WebKit/JetStream/commit/d280391b7490e6a9e30dbeb9660bc5cbf5e30038
and SunSpider is now a single line item in the overall JS3 workload.
Benchmarking appears neutral on JS3/SP3.
Canonical link: https://commits.webkit.org/309757@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
