New issue
Advanced search Search tips

Issue 736687 link

Starred by 3 users

Issue metadata

Status: WontFix
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

15.8%-63.5% regression in v8.runtimestats.browsing_mobile at 481958:481987

Project Member Reported by bmeurer@google.com, Jun 26 2017

Issue description

See the link to graphs below.
 
Project Member

Comment 2 by 42576172...@developer.gserviceaccount.com, Jun 26 2017

Cc: cbruni@chromium.org
Owner: cbruni@chromium.org

=== Auto-CCing suspected CL author cbruni@chromium.org ===

Hi cbruni@chromium.org, the bisect results pointed to your CL, please take a look at the
results.


=== BISECT JOB RESULTS ===
Perf regression found with culprit

Suspected Commit
  Author : Camillo Bruni
  Commit : c503b80595ded0e30ca9264689e026de07ce6f93
  Date   : Fri Jun 23 13:12:06 2017
  Subject: [runtime] Enable eager boilerplate creation again

Bisect Details
  Configuration: android_webview_arm64_aosp_perf_bisect
  Benchmark    : v8.runtimestats.browsing_mobile
  Metric       : Parse:duration_avg/browse_social/browse_social_twitter
  Change       : 74.88% | 121.519166667 -> 212.511333333

Revision                           Result                  N
chromium@481957                    121.519 +- 8.71159      6      good
chromium@481961                    133.578 +- 56.06        6      good
chromium@481963                    137.249 +- 66.5138      6      good
chromium@481963,v8@2b99d09e62      123.202 +- 8.58116      6      good
chromium@481963,v8@c503b80595      210.17 +- 14.2035       6      bad       <--
chromium@481963,v8@28d6d14ad5      205.535 +- 98.6229      9      bad
chromium@481964                    211.59 +- 85.939        6      bad
chromium@481965                    201.002 +- 28.4786      6      bad
chromium@481972                    206.936 +- 33.7955      6      bad
chromium@481987                    212.511 +- 31.3935      6      bad

To Run This Test
  src/tools/perf/run_benchmark -v --browser=android-webview --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=browse.social.twitter v8.runtimestats.browsing_mobile

Debug Info
  https://chromeperf.appspot.com/buildbucket_job_status/8975750926825388176

Is this bisect wrong?
  https://chromeperf.appspot.com/bad_bisect?try_job_id=5071130596999168


| O O | Visit http://www.chromium.org/developers/speed-infra/perf-bug-faq
|  X  | for more information addressing perf regression bugs. For feedback,
| / \ | file a bug with component Speed>Bisection.  Thank you!
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Jun 26 2017

Issue 736694 has been merged into this issue.

Comment 4 by jochen@chromium.org, Jun 27 2017

Status: Assigned (was: Untriaged)
Project Member

Comment 5 by 42576172...@developer.gserviceaccount.com, Jun 28 2017

Cc: sullivan@chromium.org
 Issue 737375  has been merged into this issue.
Project Member

Comment 7 by sheriffbot@chromium.org, Jul 12 2017

Labels: Hotlist-Google

Comment 8 by cbruni@chromium.org, Aug 17 2017

Status: WontFix (was: Assigned)
These are expected regressions related to tweaking on how to handle javascript object literal boilerplates.
The initial CL was too aggressive (1. improvement) and got partially reverted (1. regression) and then partially enabled again (2. improvement).

Sign in to add a comment