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

Issue 9187048: runtime: faster stack switch 1

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 5 months ago by dvyukov
Modified:
13 years, 5 months ago
Reviewers:
dave
Visibility:
Public.

Description

runtime: faster stack switch 1

Patch Set 1 #

Patch Set 2 : diff -r 0d28fd55e721 https://dvyukov%40google.com@code.google.com/p/go/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+176 lines, -51 lines) Patch
M src/cmd/6l/pass.c View 1 2 chunks +8 lines, -0 lines 0 comments Download
M src/cmd/ld/go.c View 1 1 chunk +2 lines, -0 lines 0 comments Download
M src/pkg/runtime/asm_amd64.s View 1 1 chunk +136 lines, -0 lines 0 comments Download
M src/pkg/runtime/panic.c View 1 2 chunks +6 lines, -6 lines 0 comments Download
M src/pkg/runtime/runtime.h View 1 3 chunks +5 lines, -4 lines 0 comments Download
M src/pkg/runtime/stack.c View 1 7 chunks +19 lines, -41 lines 0 comments Download

Messages

Total messages: 2
dave_cheney.net
ping. Will this see the light of day ?
13 years, 5 months ago (2013-05-24 00:05:33 UTC) #1
dvyukov
13 years, 5 months ago (2013-05-24 04:23:24 UTC) #2
On Fri, May 24, 2013 at 4:05 AM,  <dave@cheney.net> wrote:
> ping. Will this see the light of day ?
>
> https://codereview.appspot.com/9187048/


Please ping the continuos stacks design doc discussion.
If they agree that we need to do continuos stacks and Keith is
actively working on it, then I don't wont to put any more effort into
the patch.
If we won't see continuos stacks until at least Go1.3, then it may be
worth trying.
Sign in to reply to this message.

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