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

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Mar 27 09:01:24 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 305470: WIP - Proposed Patch

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




--- Comment #5 from Caio Lima <ticaiolima at gmail.com> ---
Created attachment 305470

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

WIP - Proposed Patch

I noticed that IC was not being properly handled in put_by_id_with_this in the
last patch because slot object wasn't properly configured in
```ordinarySetSlow```. Now in this version I'm proposing a change that
configure the slot properly and make possible IC PuByIdWithThis.

The results seem promising and I'm posting them here soon.


More information about the webkit-reviews mailing list