RESOLVED FIXED 75188
RenderStyle: Inline the destructor.
https://bugs.webkit.org/show_bug.cgi?id=75188
Summary RenderStyle: Inline the destructor.
Andreas Kling
Reported 2011-12-23 22:28:26 PST
The empty RenderStyle destructor gets a little hot sometimes (0.4% on HTML5 spec load.) We should inline it to remove the function call.
Attachments
Patch (1.92 KB, patch)
2011-12-23 22:32 PST, Andreas Kling
no flags
Andreas Kling
Comment 1 2011-12-23 22:32:52 PST
Andreas Kling
Comment 2 2011-12-24 10:32:15 PST
Comment on attachment 120488 [details] Patch Clearing flags on attachment: 120488 Committed r103656: <http://trac.webkit.org/changeset/103656>
Andreas Kling
Comment 3 2011-12-24 10:32:24 PST
All reviewed patches have been landed. Closing bug.
Andreas Kling
Comment 4 2011-12-24 12:15:39 PST
Darin Adler
Comment 5 2011-12-24 12:22:24 PST
Comment on attachment 120488 [details] Patch You could have just removed the declaration of ~RenderStyle altogether. I think that would be a bit more elegant and equally effective.
Note You need to log in before you can comment on or make changes to this bug.