Chromium Code Reviews
|
|
Chromium Code Reviews
Issue
1803573002:
Revert of Enable parallel pointer updates after evacuation. (Closed)
|
Created:
4 years, 9 months ago by ulan Modified:
4 years, 9 months ago Reviewers:
Michael Lippautz CC:
v8-reviews_googlegroups.com, Michael Hablich Base URL:
https://chromium.googlesource.com/v8/v8.git@master Target Ref:
refs/pending/heads/master Project:
v8 Visibility:
Public. |
DescriptionRevert of Enable parallel pointer updates after evacuation. (patchset #1 id:1 of https://codereview.chromium.org/1784643010/ )
Reason for revert:
Speculative revert to check if this CL is causing GC stress failures.
Original issue's description:
> Enable parallel pointer updates after evacuation.
>
> BUG=chromium:578883
> LOG=NO
>
> Committed: https://crrev.com/7217fcc0bf5b0f62947f92128ec76855d50e91da
> Cr-Commit-Position: refs/heads/master@{#34712}
TBR=mlippautz@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:578883
LOG=NO
Committed: https://crrev.com/70632e51c03d3dc415314d4b63811fec65c7dc63
Cr-Commit-Position: refs/heads/master@{#34741}
Patch Set 1 #
Messages
Total messages: 16 (7 generated)
Created Revert of Enable parallel pointer updates after evacuation.
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1803573002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1803573002/1
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: v8_presubmit on tryserver.v8 (JOB_FAILED, http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/12305)
lgtm
The CQ bit was checked by mlippautz@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1803573002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1803573002/1
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: v8_presubmit on tryserver.v8 (JOB_FAILED, http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/12308)
Description was changed from ========== Revert of Enable parallel pointer updates after evacuation. (patchset #1 id:1 of https://codereview.chromium.org/1784643010/ ) Reason for revert: Speculative revert to check if this CL is causing GC stress failures. Original issue's description: > Enable parallel pointer updates after evacuation. > > BUG=chromium:578883 > LOG=NO > > Committed: https://crrev.com/7217fcc0bf5b0f62947f92128ec76855d50e91da > Cr-Commit-Position: refs/heads/master@{#34712} TBR=mlippautz@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=chromium:578883 ========== to ========== Revert of Enable parallel pointer updates after evacuation. (patchset #1 id:1 of https://codereview.chromium.org/1784643010/ ) Reason for revert: Speculative revert to check if this CL is causing GC stress failures. Original issue's description: > Enable parallel pointer updates after evacuation. > > BUG=chromium:578883 > LOG=NO > > Committed: https://crrev.com/7217fcc0bf5b0f62947f92128ec76855d50e91da > Cr-Commit-Position: refs/heads/master@{#34712} TBR=mlippautz@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=chromium:578883 LOG=NO ==========
The CQ bit was checked by ulan@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1803573002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1803573002/1
Message was sent while issue was closed.
Description was changed from ========== Revert of Enable parallel pointer updates after evacuation. (patchset #1 id:1 of https://codereview.chromium.org/1784643010/ ) Reason for revert: Speculative revert to check if this CL is causing GC stress failures. Original issue's description: > Enable parallel pointer updates after evacuation. > > BUG=chromium:578883 > LOG=NO > > Committed: https://crrev.com/7217fcc0bf5b0f62947f92128ec76855d50e91da > Cr-Commit-Position: refs/heads/master@{#34712} TBR=mlippautz@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=chromium:578883 LOG=NO ========== to ========== Revert of Enable parallel pointer updates after evacuation. (patchset #1 id:1 of https://codereview.chromium.org/1784643010/ ) Reason for revert: Speculative revert to check if this CL is causing GC stress failures. Original issue's description: > Enable parallel pointer updates after evacuation. > > BUG=chromium:578883 > LOG=NO > > Committed: https://crrev.com/7217fcc0bf5b0f62947f92128ec76855d50e91da > Cr-Commit-Position: refs/heads/master@{#34712} TBR=mlippautz@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=chromium:578883 LOG=NO ==========
Message was sent while issue was closed.
Committed patchset #1 (id:1)
Message was sent while issue was closed.
Description was changed from ========== Revert of Enable parallel pointer updates after evacuation. (patchset #1 id:1 of https://codereview.chromium.org/1784643010/ ) Reason for revert: Speculative revert to check if this CL is causing GC stress failures. Original issue's description: > Enable parallel pointer updates after evacuation. > > BUG=chromium:578883 > LOG=NO > > Committed: https://crrev.com/7217fcc0bf5b0f62947f92128ec76855d50e91da > Cr-Commit-Position: refs/heads/master@{#34712} TBR=mlippautz@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=chromium:578883 LOG=NO ========== to ========== Revert of Enable parallel pointer updates after evacuation. (patchset #1 id:1 of https://codereview.chromium.org/1784643010/ ) Reason for revert: Speculative revert to check if this CL is causing GC stress failures. Original issue's description: > Enable parallel pointer updates after evacuation. > > BUG=chromium:578883 > LOG=NO > > Committed: https://crrev.com/7217fcc0bf5b0f62947f92128ec76855d50e91da > Cr-Commit-Position: refs/heads/master@{#34712} TBR=mlippautz@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=chromium:578883 LOG=NO Committed: https://crrev.com/70632e51c03d3dc415314d4b63811fec65c7dc63 Cr-Commit-Position: refs/heads/master@{#34741} ==========
Message was sent while issue was closed.
Patchset 1 (id:??) landed as https://crrev.com/70632e51c03d3dc415314d4b63811fec65c7dc63 Cr-Commit-Position: refs/heads/master@{#34741} |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
