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

Issue 154101: code review 154101: add support for pre arm v6 cas. set GOARM=5 to enable. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years, 5 months ago by kaib
Modified:
14 years, 5 months ago
Reviewers:
rsc
Visibility:
Public.

Description

add support for pre arm v6 cas. set GOARM=5 to enable.

Patch Set 1 #

Patch Set 2 : code review 154101: add support for pre arm v6 cas. set GOARM=5 to enable. #

Total comments: 2

Patch Set 3 : code review 154101: add support for pre arm v6 cas. set GOARM=5 to enable. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+45 lines, -0 lines) Patch
M src/pkg/runtime/Makefile View 1 chunk +3 lines, -1 line 0 comments Download
A src/pkg/runtime/arm/cas5.s View 1 chunk +43 lines, -0 lines 0 comments Download
M src/pkg/runtime/arm/cas6.s View 0 chunks +-1 lines, --1 lines 0 comments Download

Messages

Total messages: 3
kaib
Hello rsc, I'd like you to review the following change.
14 years, 5 months ago (2009-11-12 23:19:20 UTC) #1
rsc
LGTM http://codereview.appspot.com/154101/diff/1001/1002 File src/pkg/runtime/Makefile (right): http://codereview.appspot.com/154101/diff/1001/1002#newcode31 src/pkg/runtime/Makefile:31: GOARM ?= 6 drop spaces to match surrounding ...
14 years, 5 months ago (2009-11-12 23:21:57 UTC) #2
kaib
14 years, 5 months ago (2009-11-12 23:23:24 UTC) #3
*** Submitted as http://code.google.com/p/go/source/detail?r=b55da7e4a1a5 ***

add support for pre arm v6 cas. set GOARM=5 to enable.

R=rsc
http://codereview.appspot.com/154101
Sign in to reply to this message.

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