We consult VM::stringCache when creating a JSString, but since bug 142115 we never insert anything into it. Removing this results in almost-significant improvements in the VueJS, Vanilla-ES2015, and jQuery sub-tests of Speedometer 2 (of 0.5-2%, 0.03 <= p <= 0.05), and an almost significant 0.2% improvement in the overall score (p = 0.06).
Created attachment 449849 [details] Patch
Comment on attachment 449849 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=449849&action=review > Source/JavaScriptCore/ChangeLog:9 > + We consult VM::stringCache when creating a JSString, but since > + bug 142115 we never insert anything into it. 6 years, wow I wonder if the patch making it work again would make things even faster. I suppose it comes with the risk of memory use regression.
Committed r288537 (246369@main): <https://commits.webkit.org/246369@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 449849 [details].
<rdar://problem/88006205>