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

Issue 8198001: Rebase the expected executable size on Linux 64 bit after the (Closed)

Created:
9 years, 2 months ago by Erik Corry
Modified:
9 years, 2 months ago
Reviewers:
cmp, joth, Satish
CC:
chromium-reviews, pam+watch_chromium.org
Visibility:
Public.

Description

Rebase the expected executable size on Linux 64 bit after the incremental GC for V8 landed. Also rebase the expected number of static initializers, until we find out what caused it. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=104473

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M tools/perf_expectations/perf_expectations.json View 1 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
Erik Corry
9 years, 2 months ago (2011-10-07 13:08:55 UTC) #1
Satish
LGTM. Lets follow up on the size and static initializer increase offline
9 years, 2 months ago (2011-10-07 13:28:55 UTC) #2
joth
9 years, 2 months ago (2011-10-07 14:10:41 UTC) #3
joth
Just to close the loop on this one, are there bugs for the size and ...
9 years, 2 months ago (2011-10-07 17:54:20 UTC) #4
cmp
lgtm
9 years, 2 months ago (2011-10-07 18:00:30 UTC) #5
cmp
On 2011/10/07 17:54:20, joth wrote: > Just to close the loop on this one, are ...
9 years, 2 months ago (2011-10-07 18:06:02 UTC) #6
Erik Corry
9 years, 2 months ago (2011-10-07 18:23:02 UTC) #7
Yes 2 newest bugs on v8 bug tracker
On Oct 7, 2011 7:54 PM, <joth@chromium.org> wrote:

> Just to close the loop on this one, are there bugs for the size and static
> initializer increases? This looks a bit easy to get lost as is.
>
>
>
http://codereview.chromium.**org/8198001/<http://codereview.chromium.org/8198...
>

Powered by Google App Engine
This is Rietveld 408576698