code review 5531058: gc: enable inlining by default

gc: enable inlining by default

[rsc, 2012-01-10 04:48:49]

Hello lvd, r (cc: golang-dev@googlegroups.com),

I'd like you to review this change to
https://code.google.com/p/go/

[r, 2012-01-10 04:50:14]

LGTM

http://codereview.appspot.com/5531058/diff/3/src/cmd/gc/lex.c
File src/cmd/gc/lex.c (right):

http://codereview.appspot.com/5531058/diff/3/src/cmd/gc/lex.c#newcode243
src/cmd/gc/lex.c:243: debug['l'] = 1 - debug['l'];
nice

[lvd, 2012-01-10 09:10:04]

can you wait a sec submitting this till we figured out how -m should behave?

On Tue, Jan 10, 2012 at 05:50,  <r@golang.org> wrote:
> LGTM
>
>
> http://codereview.appspot.com/5531058/diff/3/src/cmd/gc/lex.c
> File src/cmd/gc/lex.c (right):
>
> http://codereview.appspot.com/5531058/diff/3/src/cmd/gc/lex.c#newcode243
> src/cmd/gc/lex.c:243: debug['l'] = 1 - debug['l'];
> nice
>
> http://codereview.appspot.com/5531058/

[rsc, 2012-01-10 16:55:04]

On Tue, Jan 10, 2012 at 01:10, Luuk van Dijk <lvd@google.com> wrote:
> can you wait a sec submitting this till we figured out how -m should behave?

yep, i will wait until the fixes CL is in.

[rsc, 2012-01-11 04:08:56]

*** Submitted as f7d0e6a40f06 ***

gc: enable inlining by default

R=lvd, r
CC=golang-dev
http://codereview.appspot.com/5531058