Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1371)

Issue 7837044: code review 7837044: runtime: replace lock() with casp() in the GC (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 2 months ago by atom
Modified:
12 years, 2 months ago
Reviewers:
CC:
golang-dev, dvyukov, rsc
Visibility:
Public.

Description

runtime: replace lock() with casp() in the GC Note: BitTarget will be removed by a forthcoming changeset.

Patch Set 1 #

Patch Set 2 : diff -r 93d02de84644 https://code.google.com/p/go/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+29 lines, -31 lines) Patch
M src/pkg/runtime/mgc0.c View 9 chunks +29 lines, -31 lines 0 comments Download

Messages

Total messages: 3
atom
Hello golang-dev@googlegroups.com (cc: rsc, dvyukov), I'd like you to review this change to https://code.google.com/p/go/
12 years, 2 months ago (2013-03-15 06:24:38 UTC) #1
dvyukov
LGTM
12 years, 2 months ago (2013-03-15 08:00:37 UTC) #2
atom
12 years, 2 months ago (2013-03-15 08:02:46 UTC) #3
*** Submitted as https://code.google.com/p/go/source/detail?r=1f8340c34b65 ***

runtime: replace lock() with casp() in the GC

Note: BitTarget will be removed by a forthcoming changeset.

R=golang-dev, dvyukov
CC=golang-dev, rsc
https://codereview.appspot.com/7837044
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b