diff options
author | Lucio De Re <lucio.dere@gmail.com> | 2013-01-04 11:02:49 -0500 |
---|---|---|
committer | Lucio De Re <lucio.dere@gmail.com> | 2013-01-04 11:02:49 -0500 |
commit | 1899f838b3c3e43b10fa58f67b4da8fadc6fa74f (patch) | |
tree | 3449c6ef09714f738ad4902cd9f55ddbf6fc72de | |
parent | 1b3f12e50f991bf4739335c6746b74a7a3607e56 (diff) | |
download | go-1899f838b3c3e43b10fa58f67b4da8fadc6fa74f.tar.gz |
cmd/5g, cmd/5l, cmd/ld: Small adjustments for the Plan 9 native tools
A few USED(xxx) additions and a couple of deletions of variable
initialisations that go unused. One questionable correction,
mirrored in 8l/asm.c, where the result of invocation of a function
shouldn't be used.
R=golang-dev, rsc
CC=golang-dev
https://codereview.appspot.com/6736054
Committer: Russ Cox <rsc@golang.org>
-rw-r--r-- | src/cmd/5g/ggen.c | 1 | ||||
-rw-r--r-- | src/cmd/5g/reg.c | 1 | ||||
-rw-r--r-- | src/cmd/5l/asm.c | 1 | ||||
-rw-r--r-- | src/cmd/ld/pe.c | 1 |
4 files changed, 3 insertions, 1 deletions
diff --git a/src/cmd/5g/ggen.c b/src/cmd/5g/ggen.c index 840d55a95..40ef0bd7b 100644 --- a/src/cmd/5g/ggen.c +++ b/src/cmd/5g/ggen.c @@ -543,6 +543,7 @@ cgen_shift(int op, int bounded, Node *nl, Node *nr, Node *res) Type *tr; uvlong sc; + USED(bounded); if(nl->type->width > 4) fatal("cgen_shift %T", nl->type); diff --git a/src/cmd/5g/reg.c b/src/cmd/5g/reg.c index 14cda5148..5f7ed2e88 100644 --- a/src/cmd/5g/reg.c +++ b/src/cmd/5g/reg.c @@ -207,7 +207,6 @@ regopt(Prog *firstp) return; } - r1 = R; firstr = R; lastr = R; diff --git a/src/cmd/5l/asm.c b/src/cmd/5l/asm.c index f62c927e2..1937d899a 100644 --- a/src/cmd/5l/asm.c +++ b/src/cmd/5l/asm.c @@ -913,6 +913,7 @@ asmb(void) } phsh(pnote, sh); } + USED(resoff); elfphload(&segtext); elfphload(&segdata); diff --git a/src/cmd/ld/pe.c b/src/cmd/ld/pe.c index d263562ae..f2903ba0f 100644 --- a/src/cmd/ld/pe.c +++ b/src/cmd/ld/pe.c @@ -536,6 +536,7 @@ addexcept(IMAGE_SECTION_HEADER *text) uvlong n; Sym *sym; + USED(text); if(thechar != '6') return; |