New issue
Advanced search Search tips

Issue 785723 link

Starred by 1 user
Status: Duplicate
Owner:
cbruni@chromium.org
Closed: Nov 2017
Cc:
machenb...@chromium.org
verwa...@chromium.org
ishell@chromium.org
u...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

A zero-to-nonzero to 4.8% regression in v8.runtimestats.browsing_mobile at 514098:514800

Project Member Reported by bmeurer@google.com, Nov 16 2017 
See the link to graphs below.
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Nov 16 2017 

All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=785723

(For debugging:) Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?sid=8ad88f63a72da6cb1c33d3d47e5fdaf2d5c6826158c06086a6708f5b470952a4


Bot(s) for this bug's original alert(s):

android-nexus5
android-nexus6
android-webview-nexus5X
android-webview-nexus6
chromium-rel-mac-retina
chromium-rel-mac11
chromium-rel-mac11-air
chromium-rel-mac11-pro
chromium-rel-mac12-mini-8gb
chromium-rel-win10
chromium-rel-win7-dual
chromium-rel-win7-gpu-ati
chromium-rel-win7-gpu-intel
chromium-rel-win7-gpu-nvidia
chromium-rel-win7-x64-dual
win-high-dpi
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Nov 16 2017

Cc: u...@chromium.org machenb...@chromium.org
Owner: u...@chromium.org
Status: Assigned (was: Untriaged)
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/13b6abc9f80000

[heap] Re-enable concurrent marking.
By ulan@chromium.org · Tue Nov 07 13:11:10 2017
v8 @ 6bb1d47e6e82f75ead36cdab209afc5dafa8329c

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions
Project Member

Comment 5 by 42576172...@developer.gserviceaccount.com, Nov 20 2017

Cc: ishell@chromium.org verwa...@chromium.org
Owner: verwa...@chromium.org
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/12bd636df80000

[ic] Migrate StoreIC setter support to data driven handlers
By verwaest@chromium.org · Tue Nov 07 12:52:10 2017
v8 @ 7084bd255057ffa47ffc0a1b3b33a7d704672458

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions

Comment 6 by u...@chromium.org, Nov 20 2017 

The bisect in #5 is for the increase of JS duration.

The alerts were incorrectly merged for two separate regressions:
- Increase in average GC time (coming from my CL)
- Increase JS time (coming from "Migrate StoreIC setter support to data driven handlers").

Comment 7 by verwa...@chromium.org, Nov 21 2017

Mergedinto: 782550
Owner: cbruni@chromium.org
Status: Duplicate (was: Assigned)
►

Sign in to add a comment