[Webkit-unassigned] [Bug 27762] [v8] Cache atomic strings in externalized v8 strings

[bugzilla-daemon at webkit.org]

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





--- Comment #16 from Christian Plesner Hansen <christian.plesner.hansen at gmail.com>  2009-07-28 11:34:58 PDT ---
(In reply to comment #15)
> I believe both UString and StringImpl know how to cache their hash values.  So
> add() is just a single lookup in the AtomicString hash table, you don't have to
> re-hash the string.

We currently cache a StringImpl so if they do hash caching we already get the
benefits of that.  The lookup itself still shows up high on our profiles.

Also, to answer your question from before: I think add() is as fast for us as
for JSC.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.