New issue
Advanced search Search tips

Issue 782637 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Task



Sign in to add a comment

Tracking bug for lazy handler deserialization results

Project Member Reported by jgruber@chromium.org, Nov 8 2017

Issue description

Perf & stability results for the lazy handler deserialization trial run will be collected here.

Design doc: http://goo.gl/QxZBL2
Tracking bug:  v8:6624 
 
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 8 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/9664d017f7e219878d4b1a6c25be6c6f9b1b91bd

commit 9664d017f7e219878d4b1a6c25be6c6f9b1b91bd
Author: jgruber <jgruber@chromium.org>
Date: Wed Nov 08 13:09:07 2017

[snapshot] Flip --lazy-handler-deserialization

Flip this for a day or two to gather perf bot results & stability
issues. Results will be collected in  https://crbug.com/782637 .

TBR=hablich@chromium.org

Bug:  v8:6624 ,  chromium:782637 
Change-Id: I70eb5d6287d4723f935715ef80d9ba129dc282f0
Reviewed-on: https://chromium-review.googlesource.com/758375
Reviewed-by: Jakob Gruber <jgruber@chromium.org>
Commit-Queue: Jakob Gruber <jgruber@chromium.org>
Cr-Commit-Position: refs/heads/master@{#49222}
[modify] https://crrev.com/9664d017f7e219878d4b1a6c25be6c6f9b1b91bd/src/flag-definitions.h

Project Member

Comment 3 by bugdroid1@chromium.org, Nov 10 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b0996927fb63f1e2277c8d1cddf4cae2435bdedb

commit b0996927fb63f1e2277c8d1cddf4cae2435bdedb
Author: Jakob Gruber <jgruber@chromium.org>
Date: Fri Nov 10 10:46:33 2017

Revert "[snapshot] Flip --lazy-handler-deserialization"

This reverts commit 9664d017f7e219878d4b1a6c25be6c6f9b1b91bd.

Reason for revert: Planned revert, this is on Canary 3264 and V8
branch 6.4.245.

Original change's description:
> [snapshot] Flip --lazy-handler-deserialization
>
> Flip this for a day or two to gather perf bot results & stability
> issues. Results will be collected in  https://crbug.com/782637 .
>
> TBR=hablich@chromium.org
>
> Bug:  v8:6624 ,  chromium:782637 
> Change-Id: I70eb5d6287d4723f935715ef80d9ba129dc282f0
> Reviewed-on: https://chromium-review.googlesource.com/758375
> Reviewed-by: Jakob Gruber <jgruber@chromium.org>
> Commit-Queue: Jakob Gruber <jgruber@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#49222}

TBR=hablich@chromium.org,jgruber@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug:  v8:6624 ,  chromium:782637 
Change-Id: I2a6ee63f7191d186810d7352ee63b6b664e74157
Reviewed-on: https://chromium-review.googlesource.com/763427
Commit-Queue: Jakob Gruber <jgruber@chromium.org>
Reviewed-by: Jakob Gruber <jgruber@chromium.org>
Cr-Commit-Position: refs/heads/master@{#49290}
[modify] https://crrev.com/b0996927fb63f1e2277c8d1cddf4cae2435bdedb/src/flag-definitions.h

Cc: u...@chromium.org jgruber@chromium.org
 Issue 787453  has been merged into this issue.
Issue 787817 has been merged into this issue.
Status: Fixed (was: Assigned)
Experiment is over and the flag has been landed. gc-full-mark-compactor regressions from #4 seem to come from a shift in gc timing (thanks ulan for the investigation).

Sign in to add a comment