code review 11590043: runtime: cleanup: use ArgsSizeUnknown to mark all functions

runtime: cleanup: use ArgsSizeUnknown to mark all functions
whose argument size is unknown (C vararg functions, and
assembly code without an explicit specification).

We used to use 0 to mean "unknown" and 1 to mean "zero".
Now we use ArgsSizeUnknown (0x80000000) to mean "unknown".

[khr, 2013-07-19 17:33:59]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://khr%40golang.org@code.google.com/p/go/

[rsc, 2013-07-19 17:48:50]

LGTM

Thanks very much.

https://codereview.appspot.com/11590043/diff/13001/src/cmd/5a/a.h
File src/cmd/5a/a.h (right):

https://codereview.appspot.com/11590043/diff/13001/src/cmd/5a/a.h#newcode130
src/cmd/5a/a.h:130: // This value is known to the garbage collector and should
be kept in
Let's put it in runtime/funcdata.h and include that in the .c or .y files that
need it.
Applies throughout.

https://codereview.appspot.com/11590043/diff/13001/src/cmd/6a/a.y
File src/cmd/6a/a.y (right):

https://codereview.appspot.com/11590043/diff/13001/src/cmd/6a/a.y#newcode624
src/cmd/6a/a.y:624: ((long long)ArgsSizeUnknown << 32);
s/long long/vlong/

https://codereview.appspot.com/11590043/diff/13001/src/cmd/6a/a.y#newcode629
src/cmd/6a/a.y:629: ((long long)ArgsSizeUnknown << 32);
same

https://codereview.appspot.com/11590043/diff/13001/src/cmd/8c/sgen.c
File src/cmd/8c/sgen.c (right):

https://codereview.appspot.com/11590043/diff/13001/src/cmd/8c/sgen.c#newcode37
src/cmd/8c/sgen.c:37: a = argsize();
blank line above this one

[khr, 2013-07-19 18:19:21]

*** Submitted as https://code.google.com/p/go/source/detail?r=522153101e0f ***

runtime: cleanup: use ArgsSizeUnknown to mark all functions
whose argument size is unknown (C vararg functions, and
assembly code without an explicit specification).

We used to use 0 to mean "unknown" and 1 to mean "zero".
Now we use ArgsSizeUnknown (0x80000000) to mean "unknown".

R=golang-dev, rsc
CC=golang-dev
https://codereview.appspot.com/11590043