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

Issue 156055: install copy predefined (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
15 years, 7 months ago by ken3
Modified:
15 years, 7 months ago
Reviewers:
rsc
Visibility:
Public.

Description

install copy predefined did not test 386, but should work shouldnt matter if copy is not used

Patch Set 1 #

Patch Set 2 : code review 156055: install copy predefined #

Unified diffs Side-by-side diffs Delta from patch set Stats (+209 lines, -1 line) Patch
M src/cmd/gc/builtin.c.boot View 1 chunk +1 line, -0 lines 0 comments Download
M src/cmd/gc/go.h View 1 chunk +1 line, -0 lines 0 comments Download
M src/cmd/gc/lex.c View 1 chunk +1 line, -0 lines 0 comments Download
M src/cmd/gc/print.c View 2 chunks +2 lines, -0 lines 0 comments Download
M src/cmd/gc/runtime.go View 1 chunk +1 line, -0 lines 0 comments Download
M src/cmd/gc/subr.c View 1 chunk +1 line, -0 lines 0 comments Download
M src/cmd/gc/typecheck.c View 1 chunk +26 lines, -0 lines 0 comments Download
M src/cmd/gc/walk.c View 3 chunks +12 lines, -1 line 0 comments Download
M src/pkg/runtime/Makefile View 1 chunk +1 line, -0 lines 0 comments Download
A src/pkg/runtime/memmove_386.s View 1 chunk +65 lines, -0 lines 0 comments Download
A src/pkg/runtime/memmove_amd64.s View 1 chunk +65 lines, -0 lines 0 comments Download
M src/pkg/runtime/slice.c View 1 1 chunk +33 lines, -0 lines 0 comments Download

Messages

Total messages: 2
ken3
*** Submitted as http://code.google.com/p/go/source/detail?r=12c2cf12ee0f *** install copy predefined did not test 386, but should work ...
15 years, 7 months ago (2009-11-18 04:41:46 UTC) #1
rsc
15 years, 7 months ago (2009-11-18 06:02:07 UTC) #2
LGTM
Sign in to reply to this message.

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