New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 827995 link

Starred by 2 users
Status: Assigned
Owner:
ishell@chromium.org
Cc:
machenb...@chromium.org
npm@chromium.org
ahaas@chromium.org
█ ivo.mark...@mips.com
v8-autoroll@chromium.org
verwa...@chromium.org
ishell@chromium.org
u...@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

11.6%-13.6% regression in v8.browsing_mobile at 545699:545706

Project Member Reported by ulan@google.com, Apr 2 2018 
See the link to graphs below.
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Apr 2 2018 

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

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


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

android-nexus5
android-nexus5X
android-nexus6
android-nexus7v2
android-webview-nexus6
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Apr 3 2018

Cc: ishell@chromium.org verwa...@chromium.org ahaas@chromium.org machenb...@chromium.org
Owner: machenb...@chromium.org
Status: Assigned (was: Untriaged)
📍 Found significant differences after each of 2 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/11c6b57f440000

[ic] Use Map as transition handlers instead of StoreHandler objects. by ishell@chromium.org
https://chromium.googlesource.com/v8/v8/+/78c6bbd930b9138002cfb52642b90b9aab104a78

Revert "Force proper code aligment in wasm serializer." by machenbach@chromium.org
https://chromium.googlesource.com/v8/v8/+/2746f658b70a890675e2ba919a876471d4ee0047

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

Comment 4 by verwa...@chromium.org, Aug 8

Cc: -ishell@chromium.org
Owner: ishell@chromium.org
It seems like at least the ARM generated code for IC stub with the map as transition handler gets slower:

Of the total renderer time, the "execute" RCS bucket goes up:

https://chromeperf.appspot.com/report?sid=b757f426a91064d85373134b8c51231a37cd86623a04176f7465487d419d17d7&start_rev=543304&end_rev=548399

Comment 5 by verwa...@chromium.org, Aug 8

Cc: npm@chromium.org ishell@chromium.org v8-autoroll@chromium.org ivo.mark...@mips.com
 Issue 826826  has been merged into this issue.
►

Sign in to add a comment