[Webkit-unassigned] [Bug 158083] LLInt should support other types of prototype GetById caching.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Jun 11 14:48:14 PDT 2016


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

--- Comment #49 from Keith Miller <keith_miller at apple.com> ---
(In reply to comment #47)
> But I should have suggested something simpler yesterday.
> When you convert your get_by_id into your specialized version, you
> can stash the UniquedStringImpl* into the instruction stream itself if
> you have an unused instruction offset. Do you have such a free location?

This is a good idea. We have an invariant that changing a Accessor on an object requires transitioning the structure of that object. This means that you can use, for example, pc[4] to hold the function pointer and pc[5] to point to the UniquedStringImpl* of the property.

(In reply to comment #46)
> I would like to know if there is a way to test the changes in all
> architectures.

It can be pretty tricky. I would recommend having the bot do it since, IIRC, we test all the architectures we care about.

-- 
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/20160611/b6ceb7c3/attachment.html>


More information about the webkit-unassigned mailing list