<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - We've regressed octane codeload by 8% since the revision before arrow functions"
href="https://bugs.webkit.org/show_bug.cgi?id=156252#c5">Comment # 5</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - We've regressed octane codeload by 8% since the revision before arrow functions"
href="https://bugs.webkit.org/show_bug.cgi?id=156252">bug 156252</a>
from <span class="vcard"><a class="email" href="mailto:gskachkov@gmail.com" title="GSkachkov <gskachkov@gmail.com>"> <span class="fn">GSkachkov</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=156252#c0">comment #0</a>)
<span class="quote">> It looks like arrow functions have slowed us down by 8-10%.
> I've ran Octane's code-load benchmarks from the revision
> before arrow functions vs ToT. Here is where we stand:</span >
....
<span class="quote">> If anybody has ideas on how to speed things up further, or how to speed up
> other parts of the parser, I'm all ears.</span >
I think there is a way how we can avoid at least one step back when parsing arrow function parameters.
We can create some variable that will answer if current expression can be arrow function, and will check it during parseConditionalExpression. This function cover both header for arrow function x=> and (x)=>. In this function we will check if symbol => follow after variable or (), if not it is definitely not arrow function, if yes it should be arrow function, and in case we can't parse it, it is mean we have syntax error.
Some like Proof of concept you can find in attach.
What do you think if it approach viable and can give some improvement is performance?</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>