[Webkit-unassigned] [Bug 32430] [v8, dom] Add a pointer field to C++ Node class

[bugzilla-daemon at webkit.org]

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





--- Comment #15 from Sam Weinig <sam at webkit.org>  2009-12-15 21:47:47 PST ---
(In reply to comment #14)
> (In reply to comment #13)
> > (In reply to comment #7)
> > > This bug is quite devoid of any actual details of the speedup you got.  What
> > > was the win?  What did you test on?  Why is this the right tradeoff?
> > 
> > Sorry, Sam, didn't want to overload with details.
> > 
> > The speedup is ~10% (tested on Windows and Linux) for overall score for DOM
> > core of Dromaeo.
> 
> At what cost?  What was the change in memory use?

Another question: Is putting this pointer on Node really the right choice? We
have discussed specializing Element or  even HTMLElement to avoid the cost for
every Node (Text nodes for instance). Did you measure which type of Nodes this
would be the biggest win for?

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