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

Issue 5030004: Update v8_benchmark after V8 roll at r64892. (Closed)

Created:
10 years, 1 month ago by chase
Modified:
9 years, 7 months ago
Reviewers:
Nicolas Sylvain
CC:
chromium-reviews, pam+watch_chromium.org
Visibility:
Public.

Description

Update v8_benchmark after V8 roll at r64892. BUG=63267 TEST=v8 benchmark step is green TBR=nsylvain@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=66200

Patch Set 1 #

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

Messages

Total messages: 2 (0 generated)
chase
10 years, 1 month ago (2010-11-16 00:30:49 UTC) #1
Nicolas Sylvain
10 years, 1 month ago (2010-11-16 00:45:34 UTC) #2
lgtm

On Mon, Nov 15, 2010 at 4:30 PM, <chase@chromium.org> wrote:

> Reviewers: Nicolas Sylvain,
>
> Description:
> Update v8_benchmark after V8 roll at r64892.
>
> BUG=63267
> TEST=v8 benchmark step is green
> TBR=nsylvain@chromium.org
>
> Please review this at http://codereview.chromium.org/5030004/
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src/tools/perf_expectations
>
> Affected files:
>  M tools/perf_expectations/perf_expectations.json
>
>
> Index: tools/perf_expectations/perf_expectations.json
> diff --git a/tools/perf_expectations/perf_expectations.json
> b/tools/perf_expectations/perf_expectations.json
> index
>
eb6f0679ced35966566e76e2a50e133162defaae..509052ed672374108f9ede5a6e5c2db1c1582ef2
> 100644
> --- a/tools/perf_expectations/perf_expectations.json
> +++ b/tools/perf_expectations/perf_expectations.json
> @@ -16,7 +16,7 @@
>  "linux-release-hardy/morejs/times/t": {"reva": 63860, "revb": 64266,
> "improve": 121, "regress": 281},
>  "linux-release-hardy/moz/times/t": {"improve": -190, "regress": 20},
>  "linux-release-hardy/startup/warm/t": {"improve": -11, "regress": 93},
> - "linux-release-hardy/v8_benchmark/score/score": {"improve": 750,
> "regress": 450},
> + "linux-release-hardy/v8_benchmark/score/score": {"reva": 64893, "revb":
> 66188, "improve": 828, "regress": 130},
>  "linux-release/dhtml/times/t": {"improve": -22, "regress": 8},
>  "linux-release/intl1/times/t": {"improve": -18, "regress": 12},
>  "linux-release/intl2/times/t": {"improve": -18, "regress": 12},
> @@ -37,21 +37,21 @@
>  "mac-release-10.5/morejs/times/t": {"improve": -210, "regress": -60},
>  "mac-release-10.5/moz/times/t": {"improve": -40, "regress": 200},
>  "mac-release-10.5/startup/warm/t": {"improve": 20, "regress": 80},
> - "mac-release-10.5/v8_benchmark/score/score": {"improve": 1420, "regress":
> 1120},
> + "mac-release-10.5/v8_benchmark/score/score": {"reva": 64892, "revb":
> 66187, "improve": 1413, "regress": 1040},
>  "mac-release-10.6/dhtml/times/t": {"improve": -55, "regress": 25},
>  "mac-release-10.6/intl1/times/t": {"reva": 63692, "revb": 64027,
> "improve": -20733, "regress": 23605},
>  "mac-release-10.6/intl2/times/t": {"reva": 63692, "revb": 64027,
> "improve": -17204, "regress": 19694},
>  "mac-release-10.6/morejs/times/t": {"improve": -200, "regress": -40},
>  "mac-release-10.6/moz/times/t": {"improve": -30, "regress": 200},
>  "mac-release-10.6/startup/times/t": {"improve": -2, "regress": 22},
> - "mac-release-10.6/v8_benchmark/score/score": {"improve": 1730, "regress":
> 1360},
> + "mac-release-10.6/v8_benchmark/score/score": {"reva": 64893, "revb":
> 66191, "improve": 1670, "regress": 1049},
>  "vista-release-dual-core/dhtml/times/t": {"improve": -22, "regress": 45},
>  "vista-release-dual-core/intl1/times/t": {"reva": 63721, "revb": 64027,
> "improve": 178, "regress": 574},
>  "vista-release-dual-core/intl2/times/t": {"improve": -180, "regress":
> 215},
>  "vista-release-dual-core/morejs/times/t": {"improve": -130, "regress":
> -60},
>  "vista-release-dual-core/moz/times/t": {"improve": -3, "regress": 133},
>  "vista-release-dual-core/startup/times/t": {"improve": -15, "regress":
> 15},
> - "vista-release-dual-core/v8_benchmark/score/score": {"improve": 1650,
> "regress": 1070},
> + "vista-release-dual-core/v8_benchmark/score/score": {"reva": 64892,
> "revb": 66190, "improve": 1715, "regress": 772},
>  "vista-release-single-core/dhtml/times/t": {"improve": -90, "regress":
> 90},
>  "vista-release-single-core/intl1/times/t": {"improve": -510, "regress":
> 510},
>  "vista-release-single-core/intl2/times/t": {"improve": -105, "regress":
> 105},
> @@ -72,7 +72,7 @@
>  "xp-release-dual-core/moz/vm_peak_b/vm_pk_b": {"improve": 600000,
> "regress": 4200000},
>  "xp-release-dual-core/moz/vm_peak_r/vm_pk_r": {"reva": 63660, "revb":
> 64027, "improve": -14949581, "regress": -7358259},
>  "xp-release-dual-core/startup/warm/t": {"reva": 63759, "revb": 64027,
> "improve": -4, "regress": 13},
> - "xp-release-dual-core/v8_benchmark/score/score": {"improve": 1500,
> "regress": 1120},
> + "xp-release-dual-core/v8_benchmark/score/score": {"reva": 64892, "revb":
> 66189, "improve": 1455, "regress": 937},
>  "xp-release-single-core/dhtml/times/t": {"improve": -21, "regress": 21},
>  "xp-release-single-core/intl1/times/t": {"improve": 510, "regress": 930},
>  "xp-release-single-core/intl2/times/t": {"improve": 225, "regress": 375},
>
>
>

Powered by Google App Engine
This is Rietveld 408576698