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

Issue 169620043: [dev.cc] code review 169620043: runtime: convert netbsd/amd64 port to Go (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 4 months ago by jsing
Modified:
10 years, 4 months ago
Reviewers:
rsc
CC:
rsc, golang-codereviews
Visibility:
Public.

Description

runtime: convert netbsd/amd64 port to Go

Patch Set 1 #

Patch Set 2 : diff -r 56f3a790b67aff99cc0aa47b7d880fa1bd3b176b https://go.googlecode.com/hg/ #

Patch Set 3 : diff -r 9e4038f249b2fe62874b4113daf95c284ad7371d https://go.googlecode.com/hg/ #

Patch Set 4 : diff -r 74e3e900f1607ef411748ae3d9090920832ddcc6 https://go.googlecode.com/hg/ #

Total comments: 4

Patch Set 5 : diff -r db0f2b5fe958fca53fdd39c160547de620e120d2 https://go.googlecode.com/hg/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+310 lines, -385 lines) Patch
M src/runtime/defs1_netbsd_amd64.go View 1 3 chunks +16 lines, -4 lines 0 comments Download
M src/runtime/os1_netbsd.go View 1 2 chunks +166 lines, -269 lines 0 comments Download
M src/runtime/os1_netbsd_amd64.go View 1 1 chunk +9 lines, -11 lines 0 comments Download
M src/runtime/os2_netbsd.go View 1 1 chunk +11 lines, -24 lines 0 comments Download
M src/runtime/os_netbsd.go View 1 2 3 4 1 chunk +27 lines, -5 lines 0 comments Download
M src/runtime/signal_netbsd.go View 1 1 chunk +40 lines, -48 lines 0 comments Download
M src/runtime/signal_netbsd_amd64.go View 1 1 chunk +41 lines, -24 lines 0 comments Download

Messages

Total messages: 5
jsing
Hello rsc@golang.org (cc: golang-codereviews@googlegroups.com), I'd like you to review this change to the dev.cc branch ...
10 years, 4 months ago (2014-11-14 17:53:43 UTC) #1
jsing
PTAL
10 years, 4 months ago (2014-11-19 12:00:16 UTC) #2
rsc
LGTM https://codereview.appspot.com/169620043/diff/60001/src/runtime/os_netbsd.go File src/runtime/os_netbsd.go (right): https://codereview.appspot.com/169620043/diff/60001/src/runtime/os_netbsd.go#newcode24 src/runtime/os_netbsd.go:24: //go:noescape delete; meaningless on function with no pointer ...
10 years, 4 months ago (2014-11-21 19:47:36 UTC) #3
jsing
https://codereview.appspot.com/169620043/diff/60001/src/runtime/os_netbsd.go File src/runtime/os_netbsd.go (right): https://codereview.appspot.com/169620043/diff/60001/src/runtime/os_netbsd.go#newcode24 src/runtime/os_netbsd.go:24: //go:noescape On 2014/11/21 19:47:36, rsc wrote: > delete; meaningless ...
10 years, 4 months ago (2014-11-22 05:05:16 UTC) #4
jsing
10 years, 4 months ago (2014-11-22 05:05:43 UTC) #5
*** Submitted as https://code.google.com/p/go/source/detail?r=ccea2c6008f0 ***

[dev.cc] runtime: convert netbsd/amd64 port to Go

LGTM=rsc
R=rsc
CC=golang-codereviews
https://codereview.appspot.com/169620043
Sign in to reply to this message.

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