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

Issue 5966075: code review 5966075: cmd/gc: inline slice[arr,str] in the frontend (mostly). (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 12 months ago by lvd
Modified:
11 years, 6 months ago
Reviewers:
CC:
rsc, ality, rog, minux1, dfc, golang-dev
Visibility:
Public.

Description

cmd/gc: inline slice[arr,str] in the frontend (mostly).

Patch Set 1 #

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

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

Patch Set 4 : diff -r 0051c7442fed https://go.googlecode.com/hg/ #

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

Patch Set 6 : diff -r 0051c7442fed https://go.googlecode.com/hg/ #

Patch Set 7 : diff -r 99292bad0d6d https://go.googlecode.com/hg/ #

Patch Set 8 : diff -r 99292bad0d6d https://go.googlecode.com/hg/ #

Patch Set 9 : diff -r 99292bad0d6d https://go.googlecode.com/hg/ #

Patch Set 10 : diff -r 99292bad0d6d https://go.googlecode.com/hg/ #

Total comments: 1

Patch Set 11 : diff -r 316890203045 https://go.googlecode.com/hg/ #

Patch Set 12 : diff -r 316890203045 https://go.googlecode.com/hg/ #

Patch Set 13 : diff -r 316890203045 https://go.googlecode.com/hg/ #

Total comments: 4
Unified diffs Side-by-side diffs Delta from patch set Stats (+350 lines, -1440 lines) Patch
M src/cmd/5g/cgen.c View 1 2 3 4 5 6 7 2 chunks +21 lines, -0 lines 0 comments Download
M src/cmd/5g/gg.h View 1 2 3 4 5 6 7 1 chunk +0 lines, -1 line 0 comments Download
M src/cmd/5g/ggen.c View 1 2 3 4 5 6 7 1 chunk +0 lines, -392 lines 0 comments Download
M src/cmd/5g/gsubr.c View 1 2 3 4 5 6 7 8 9 10 1 chunk +21 lines, -0 lines 1 comment Download
M src/cmd/6g/cgen.c View 1 2 3 2 chunks +19 lines, -2 lines 0 comments Download
M src/cmd/6g/gg.h View 1 1 chunk +0 lines, -1 line 0 comments Download
M src/cmd/6g/ggen.c View 1 2 3 4 5 6 1 chunk +0 lines, -363 lines 0 comments Download
M src/cmd/6g/gsubr.c View 1 2 3 4 5 6 7 1 chunk +19 lines, -0 lines 0 comments Download
M src/cmd/8g/cgen.c View 1 2 3 4 5 6 2 chunks +19 lines, -2 lines 0 comments Download
M src/cmd/8g/gg.h View 1 2 3 4 5 6 1 chunk +0 lines, -1 line 0 comments Download
M src/cmd/8g/ggen.c View 1 2 3 4 5 6 1 chunk +0 lines, -398 lines 0 comments Download
M src/cmd/8g/gsubr.c View 1 2 3 4 5 6 1 chunk +19 lines, -0 lines 0 comments Download
M src/cmd/gc/builtin.c View 1 1 chunk +0 lines, -3 lines 0 comments Download
src/cmd/gc/gen.c View 1 2 3 4 5 6 1 chunk +60 lines, -0 lines 0 comments Download
M src/cmd/gc/go.h View 1 2 3 4 5 6 2 chunks +2 lines, -0 lines 0 comments Download
M src/cmd/gc/runtime.go View 1 1 chunk +0 lines, -3 lines 0 comments Download
M src/cmd/gc/walk.c View 1 2 3 4 5 6 7 8 9 10 11 12 5 chunks +170 lines, -106 lines 3 comments Download
M src/pkg/runtime/slice.c View 1 1 chunk +0 lines, -143 lines 0 comments Download
M src/pkg/runtime/string.goc View 1 1 chunk +0 lines, -25 lines 0 comments Download

Messages

Total messages: 29
lvd
Hello rsc@golang.org (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://go.googlecode.com/hg/
11 years, 11 months ago (2012-04-16 11:30:40 UTC) #1
lvd
6g and 8g pass all tests for 5g i need to be able to test ...
11 years, 11 months ago (2012-04-16 11:33:02 UTC) #2
ality
This is glorious. Over 1000 lines deleted. Do we have any slice heavy benchmarks to ...
11 years, 11 months ago (2012-04-16 11:46:43 UTC) #3
lvd
On Mon, Apr 16, 2012 at 13:46, Anthony Martin <ality@pbrane.org> wrote: > This is glorious. ...
11 years, 11 months ago (2012-04-16 12:56:18 UTC) #4
rog
what does the code look like after the linker has done with it? On 16 ...
11 years, 11 months ago (2012-04-16 14:30:42 UTC) #5
lvd
On Mon, Apr 16, 2012 at 16:30, roger peppe <rogpeppe@gmail.com> wrote: > what does the ...
11 years, 11 months ago (2012-04-16 14:44:49 UTC) #6
minux1
On 2012/04/16 11:33:02, lvd wrote: > 6g and 8g pass all tests > for 5g ...
11 years, 11 months ago (2012-04-17 18:18:05 UTC) #7
minux1
This should fix the test failure (./all.bash passed on the Linux/ARM builder). I need more ...
11 years, 11 months ago (2012-04-17 18:54:49 UTC) #8
lvd
thank you! i hadn't gotten around to figuring out that missing bit and i didn't ...
11 years, 11 months ago (2012-04-17 20:19:41 UTC) #9
lvd
On Tue, Apr 17, 2012 at 22:19, Luuk van Dijk <lvd@google.com> wrote: > thank you! ...
11 years, 11 months ago (2012-04-17 20:39:30 UTC) #10
lvd
On Tue, Apr 17, 2012 at 22:39, Luuk van Dijk <lvd@google.com> wrote: > http://codereview.appspot.com/**5966075/diff/24001/src/cmd/5g/** >>> ...
11 years, 11 months ago (2012-04-17 20:41:11 UTC) #11
dfc
I hand patched minux's suggested checkref only my 5g system and now all the tests ...
11 years, 11 months ago (2012-04-18 00:56:22 UTC) #12
lvd
On Wed, Apr 18, 2012 at 02:56, Dave Cheney <dave@cheney.net> wrote: > I hand patched ...
11 years, 11 months ago (2012-04-18 05:46:09 UTC) #13
lvd
actually i didn't look close enough, there is a regression in test/benchmark/go1, for JSONDecode on ...
11 years, 11 months ago (2012-04-18 06:19:38 UTC) #14
lvd
actually, on 386 it's terrible too, and even on amd64, there's a 33% slowdown for ...
11 years, 11 months ago (2012-04-18 06:29:48 UTC) #15
lvd
On Wed, Apr 18, 2012 at 08:29, Luuk van Dijk <lvd@google.com> wrote: > actually, on ...
11 years, 11 months ago (2012-04-18 06:42:27 UTC) #16
lvd
On Wed, Apr 18, 2012 at 08:42, Luuk van Dijk <lvd@google.com> wrote: > > > ...
11 years, 11 months ago (2012-04-18 08:01:18 UTC) #17
minux1
beside this small typo, I can confirm that the performance regression on ARM is fixed. ...
11 years, 11 months ago (2012-04-19 17:45:34 UTC) #18
lvd
On 2012/04/19 17:45:34, minux wrote: > beside this small typo, I can confirm that the ...
11 years, 11 months ago (2012-04-19 18:13:46 UTC) #19
dfc
LGTM. Before, core i5 darwin/amd64 odessa(~/go/test) % time ./run 0 known bugs; 0 unexpected bugs ...
11 years, 11 months ago (2012-04-19 21:59:34 UTC) #20
rsc
LGTM I'm sorry it took me so long to get to this review. I think ...
11 years, 10 months ago (2012-06-02 22:47:58 UTC) #21
dfc
Woot. On 03/06/2012, at 8:47, rsc@golang.org wrote: > LGTM > > I'm sorry it took ...
11 years, 10 months ago (2012-06-02 22:49:17 UTC) #22
rsc
*** Submitted as http://code.google.com/p/go/source/detail?r=9acc060bf892 *** cmd/gc: inline slice[arr,str] in the frontend (mostly). R=rsc, ality, rogpeppe, ...
11 years, 10 months ago (2012-06-03 02:51:03 UTC) #23
dfc
Sorry, this doesn't compile under 5c cmd/5g /home/dfc/go/src/cmd/5g/gsubr.c:1146:1: error: unknown type name 'bsdvoid' bsdvoid ^ ...
11 years, 10 months ago (2012-06-03 03:54:25 UTC) #24
rsc
On Sat, Jun 2, 2012 at 11:54 PM, Dave Cheney <dave@cheney.net> wrote: > Sorry, this ...
11 years, 10 months ago (2012-06-03 03:55:43 UTC) #25
dfc
Yup. On Sun, Jun 3, 2012 at 1:55 PM, Russ Cox <rsc@golang.org> wrote: > On ...
11 years, 10 months ago (2012-06-03 03:56:44 UTC) #26
rsc
On Sat, Jun 2, 2012 at 11:56 PM, Dave Cheney <dave@cheney.net> wrote: > Yup. submitted ...
11 years, 10 months ago (2012-06-03 03:58:17 UTC) #27
lvd
what a nice surprise to find coming back from holiday. thanks! On Sun, Jun 3, ...
11 years, 9 months ago (2012-06-13 07:50:07 UTC) #28
lvd
11 years, 6 months ago (2012-10-01 13:46:29 UTC) #29
*** Abandoned ***
Sign in to reply to this message.

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