<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_REOPENED "
title="REOPENED - LLInt should support other types of prototype GetById caching."
href="https://bugs.webkit.org/show_bug.cgi?id=158083#c120">Comment # 120</a>
on <a class="bz_bug_link
bz_status_REOPENED "
title="REOPENED - LLInt should support other types of prototype GetById caching."
href="https://bugs.webkit.org/show_bug.cgi?id=158083">bug 158083</a>
from <span class="vcard"><a class="email" href="mailto:ticaiolima@gmail.com" title="Caio Lima <ticaiolima@gmail.com>"> <span class="fn">Caio Lima</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=158083#c118">comment #118</a>)
<span class="quote">> Comment on <span class=""><a href="attachment.cgi?id=291039&action=diff" name="attach_291039" title="Patch">attachment 291039</a> <a href="attachment.cgi?id=291039&action=edit" title="Patch">[details]</a></span>
> Patch
>
> I remember Keith did a patch similar to this in the LLInt for prototype
> values and we didn't see a measurable speedup from it.
>
> Should we be doing optimizations like this in the LLInt? Is it worth the
> complexity and maintenance </span >
I think I can't agree more. It is bringing a complex with practically no performance gains. I remember last time I ran Speedometer in x86_64 and I didn't saw gains. The point is that we can get better improvements if JIT=false, but it doesn't represent real world usage.
<span class="quote">> What's the best benchmark result we have from this? Do we have any results
> that are not tailored micro benchmarks?</span >
I don't.
What do you think if we just cache getters and forget about customGetter inline? The patch will be much more simpler and we also can take advantage on "super" access in a further case. I can take a look on that and see if it is worth or not.</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>