[webkit-reviews] review requested: [Bug 162125] Enable IC for put_by_id_with_this : [Attachment 308185] Proposed Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 25 18:43:23 PDT 2017


Caio Lima <ticaiolima at gmail.com> has asked  for review:
Bug 162125: Enable IC for put_by_id_with_this
https://bugs.webkit.org/show_bug.cgi?id=162125

Attachment 308185: Proposed Patch

https://bugs.webkit.org/attachment.cgi?id=308185&action=review




--- Comment #18 from Caio Lima <ticaiolima at gmail.com> ---
Created attachment 308185

  --> https://bugs.webkit.org/attachment.cgi?id=308185&action=review

Proposed Patch

This Version is enabling put_by_id_with_this for setter and value cases. Here
are some benchmarks improved with this patch:

super-set-value 		      ^ definitely 9.2902x faster
super-setter			      ^ definitely 5.9915x faster
untyped-string-from-char-code	      ^ definitely 1.0725x faster
super-get-by-id-with-this-monomorphic ^ definitely 1.1543x faster


More information about the webkit-reviews mailing list