[Webkit-unassigned] [Bug 162124] Unite op_get_by_id_with_this with op_get_by_id

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=162124

--- Comment #10 from Caio Lima <ticaiolima at gmail.com> ---
Created attachment 289331
  --> https://bugs.webkit.org/attachment.cgi?id=289331&action=review
Benchmarks results

It seems that the current Patch keeps benchmarks neutral and over ~20% improvements in super-get-by-id micro benchmarks:

   super-get-by-id-with-this-monomorphic             36.5942+-0.7969     ^     29.8105+-0.3676        ^ definitely 1.2276x faster
   super-get-by-id-with-this-polymorphic             35.1846+-2.1380     ^     28.1864+-0.6060        ^ definitely 1.2483x faster

General results:
Geomean of preferred means:
   <scaled-result>                                   27.6482+-0.1572           27.5743+-0.1447          might be 1.0027x faster

I suspect we can improve it a little more in compilation phase, since it can be possible remove if(m_base->isSuperNode()).

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160920/10c18dc6/attachment.html>