Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(168)

Issue 1351063002: Oilpan: Remove 'inline' from Heap::allocateOnHeapIndex

Created:
5 years, 3 months ago by haraken
Modified:
5 years, 3 months ago
Reviewers:
keishi
CC:
blink-reviews, oilpan-reviews, kouhei+heap_chromium.org, Mads Ager (chromium)
Target Ref:
refs/heads/master
Project:
blink
Visibility:
Public.

Description

Oilpan: Remove 'inline' from Heap::allocateOnHeapIndex For some reason I don't know, this CL improves performance of document.createElement("div") by 10% and thus fix regressions observed in Dromaeo/dom-modify.html. BUG=

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -0 lines) Patch
M Source/platform/heap/Heap.h View 2 chunks +2 lines, -0 lines 0 comments Download
M Source/platform/heap/Heap.cpp View 1 chunk +8 lines, -0 lines 0 comments Download

Messages

Total messages: 10 (1 generated)
haraken
PTAL keishi@: Would you measure the performance of Bindings/create-element.html with the following settings? - ToT ...
5 years, 3 months ago (2015-09-17 10:26:49 UTC) #2
sof
Does this one perturb blink_perf.dom.textarea-dom numbers also?
5 years, 3 months ago (2015-09-17 10:47:52 UTC) #3
haraken
On 2015/09/17 10:47:52, sof wrote: > Does this one perturb blink_perf.dom.textarea-dom numbers also? Unfortunately the ...
5 years, 3 months ago (2015-09-17 10:51:53 UTC) #4
keishi
Here are my results. Bindings/create-element.html regression gone. CSS tests are better too. no change for ...
5 years, 3 months ago (2015-09-18 05:12:08 UTC) #5
haraken
On 2015/09/18 05:12:08, keishi wrote: > Here are my results. > Bindings/create-element.html regression gone. > ...
5 years, 3 months ago (2015-09-18 05:24:38 UTC) #6
sof
On 2015/09/18 05:24:38, haraken wrote: > On 2015/09/18 05:12:08, keishi wrote: > > Here are ...
5 years, 3 months ago (2015-09-18 05:44:06 UTC) #7
haraken
On 2015/09/18 05:44:06, sof wrote: > On 2015/09/18 05:24:38, haraken wrote: > > On 2015/09/18 ...
5 years, 3 months ago (2015-09-18 05:46:35 UTC) #8
keishi
On 2015/09/18 05:46:35, haraken wrote: > On 2015/09/18 05:44:06, sof wrote: > > On 2015/09/18 ...
5 years, 3 months ago (2015-09-18 05:53:39 UTC) #9
haraken
5 years, 3 months ago (2015-09-24 01:28:16 UTC) #10
On 2015/09/18 05:53:39, keishi wrote:
> On 2015/09/18 05:46:35, haraken wrote:
> > On 2015/09/18 05:44:06, sof wrote:
> > > On 2015/09/18 05:24:38, haraken wrote:
> > > > On 2015/09/18 05:12:08, keishi wrote:
> > > > > Here are my results.
> > > > > Bindings/create-element.html regression gone.
> > > > > CSS tests are better too.
> > > > > no change for textarea-dom
> > > > >
> > > >
> > >
> >
>
https://www.googledrive.com/host/0B4aiM9jljUy8UEh0QjR3SUVXaVE/linux-0917-resu...
> > > > 
> > > > Thanks! Then PTAL :)
> > > 
> > > What level of confidence do you have that the observed improvements will
> carry
> > > over to other platforms? (I'm fine with landing this, but that we keep
this
> > > change in mind when looking & reconciling perf data on other platforms
> later.)
> > 
> > keishi-san: Would you just check if this CL improves create-elememt.html in
> Mac?
> 
> ok

Reuploaded to https://codereview.chromium.org/1367753004/ after the repo merge.

Powered by Google App Engine
This is Rietveld 408576698