Bug 64863 - Switch isPrimitiveValue() virtual function to inline one.
: Switch isPrimitiveValue() virtual function to inline one.
Status: RESOLVED DUPLICATE of bug 71668
: WebKit
CSS
: 528+ (Nightly build)
: All All
: P2 Normal
Assigned To:
:
:
:
: 64262
  Show dependency treegraph
 
Reported: 2011-07-20 05:08 PST by
Modified: 2012-01-08 15:27 PST (History)


Attachments
Switch isPrimitiveValue virtual function to inline (8.53 KB, patch)
2011-07-20 05:11 PST, Tamas Czene
eric: review-
webkit.review.bot: commit‑queue-
Review Patch | Details | Formatted Diff | Diff


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2011-07-20 05:08:57 PST
Removing the virtual functions - removing the calls - makes measurable performance progression on CSS.

We measured the following performance progressions on some very popular websites, for example:
msn.com: 33%
yandex.ru: 3,3%

The number of variables created during the test.
m_isPrimitiveValue 261596
The max number of variables in the memory during the test.
m_isPrimitiveValue 468
------- Comment #1 From 2011-07-20 05:11:35 PST -------
Created an attachment (id=101453) [details]
Switch isPrimitiveValue virtual function to inline
------- Comment #2 From 2011-07-20 05:42:40 PST -------
(From update of attachment 101453 [details])
Attachment 101453 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/9200235

New failing tests:
css2.1/t1202-counter-16-f.html
css2.1/t1202-counters-03-b.html
css2.1/t1202-counter-12-b.html
css2.1/t1202-counter-08-b.html
css2.1/t1202-counter-02-b.html
css2.1/t1202-counter-14-b.html
css2.1/t1202-counter-00-b.html
css2.1/t1202-counter-09-b.html
css2.1/t1202-counter-15-b.html
css2.1/t1202-counter-05-b.html
css2.1/t1202-counter-07-b.html
css2.1/t1202-counter-06-b.html
css2.1/t1202-counter-01-b.html
css2.1/t1202-counter-03-b.html
css2.1/t1202-counters-02-b.html
css2.1/t1202-counter-04-b.html
css2.1/t1202-counters-01-b.html
css2.1/t1202-counter-13-b.html
css2.1/t1202-counter-11-b.html
css2.1/t1202-counters-00-b.html
------- Comment #3 From 2011-07-20 13:25:44 PST -------
See my comments in https://bugs.webkit.org/show_bug.cgi?id=64262.
------- Comment #4 From 2011-09-12 17:00:07 PST -------
(From update of attachment 101453 [details])
How much of a memory regression is this?
------- Comment #5 From 2011-10-11 17:09:45 PST -------
I believe there are some wasted bits that could be put to good use for a perf enhancement without increasing the memory overhead. Please give a chance to upload an RFC patch and some repeatable numbers.
------- Comment #6 From 2012-01-08 15:27:53 PST -------
The action requested in this bug was taken in:
http://trac.webkit.org/changeset/99468

*** This bug has been marked as a duplicate of bug 71668 ***