Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(311)

Issue 2639303002: Merged: [heap] Use CAS to update old to new slots in Scavenger. (Closed)

Created:
3 years, 11 months ago by Hannes Payer (out of office)
Modified:
3 years, 11 months ago
Reviewers:
Michael Lippautz
CC:
v8-reviews_googlegroups.com, v8-merges_googlegroups.com, ulan
Target Ref:
refs/pending/branch-heads/5.6
Project:
v8
Visibility:
Public.

Description

Merged: [heap] Use CAS to update old to new slots in Scavenger. Revision: fdff4b072ee15f8dbca60e86fdd4d054f034cb7f BUG=chromium:676635 LOG=N NOTRY=true NOPRESUBMIT=true NOTREECHECKS=true R=mlippautz@chromium.org Review-Url: https://codereview.chromium.org/2639303002 . Cr-Commit-Position: refs/branch-heads/5.6@{#79} Cr-Branched-From: bdd3886218dfe76e8560eb8a18401942452ae859-refs/heads/5.6.326@{#1} Cr-Branched-From: 879f6599eee6e1dfcbe9a24bf688b261c03e9558-refs/heads/master@{#41014} Committed: https://chromium.googlesource.com/v8/v8/+/dd44aa6d859fd3e1af12c6add0e4ac6194a2709a

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -2 lines) Patch
M src/heap/scavenger.cc View 1 chunk +6 lines, -2 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
Hannes Payer (out of office)
3 years, 11 months ago (2017-01-18 15:42:46 UTC) #1
Hannes Payer (out of office)
NOTE: This script will no longer automatically update include/v8-version.h and create a tag. This is ...
3 years, 11 months ago (2017-01-18 15:42:48 UTC) #2
Michael Lippautz
lgtm
3 years, 11 months ago (2017-01-18 15:44:18 UTC) #3
Hannes Payer (out of office)
3 years, 11 months ago (2017-01-18 15:46:53 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
dd44aa6d859fd3e1af12c6add0e4ac6194a2709a.

Powered by Google App Engine
This is Rietveld 408576698