diff options
| author | Shenghou Ma <minux.ma@gmail.com> | 2013-05-30 03:03:52 +0800 |
|---|---|---|
| committer | Shenghou Ma <minux.ma@gmail.com> | 2013-05-30 03:03:52 +0800 |
| commit | d3512712a469121491357de0be053966a66a4bcd (patch) | |
| tree | b737d065232cb8d387716ffec36fd583a1bf999c /src/cmd/dist/buildruntime.c | |
| parent | f2cf612ba57b67f207b245e3195e129e36492406 (diff) | |
| download | go-d3512712a469121491357de0be053966a66a4bcd.tar.gz | |
cmd/5a, cmd/dist, runtime: support m/g in the assembler, drop support for R9/R10
to avoid unintentionally clobber R9/R10.
Thanks Lucio for the suggestion.
PS: yes, this could be considered a big change (but not an API change), but
as it turns out even temporarily changes R9/R10 in user code is unsafe and
leads to very hard to diagnose problems later, better to disable using R9/R10
when the user first uses it.
See CL 6300043 and CL 6305100 for two problems caused by misusing R9/R10.
R=golang-dev, khr, rsc
CC=golang-dev
https://codereview.appspot.com/9840043
Diffstat (limited to 'src/cmd/dist/buildruntime.c')
| -rw-r--r-- | src/cmd/dist/buildruntime.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/cmd/dist/buildruntime.c b/src/cmd/dist/buildruntime.c index ee867566f..8f3fc5416 100644 --- a/src/cmd/dist/buildruntime.c +++ b/src/cmd/dist/buildruntime.c @@ -174,8 +174,6 @@ static struct { }, {"arm", "", - "#define g R10\n" - "#define m R9\n" "#define LR R14\n" }, }; |