From 823962c52186415d61f23a819af298dd45ed3a52 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?R=C3=A9my=20Oudompheng?= Date: Fri, 3 Aug 2012 22:05:51 +0200 Subject: [PATCH] cmd/8g: fix miscompilation due to BADWIDTH. Fixes #3899. R=rsc CC=golang-dev, remy https://golang.org/cl/6453084 --- src/cmd/8g/gsubr.c | 1 + test/fixedbugs/bug450.go | 28 ++++++++++++++++++++++++++++ 2 files changed, 29 insertions(+) create mode 100644 test/fixedbugs/bug450.go diff --git a/src/cmd/8g/gsubr.c b/src/cmd/8g/gsubr.c index 439495c0345..ca54b862792 100644 --- a/src/cmd/8g/gsubr.c +++ b/src/cmd/8g/gsubr.c @@ -1853,6 +1853,7 @@ naddr(Node *n, Addr *a, int canemitcode) a->width = 0; if(n->type != T) { a->etype = simtype[n->type->etype]; + dowidth(n->type); a->width = n->type->width; a->gotype = ngotype(n); } diff --git a/test/fixedbugs/bug450.go b/test/fixedbugs/bug450.go new file mode 100644 index 00000000000..3f13de16ceb --- /dev/null +++ b/test/fixedbugs/bug450.go @@ -0,0 +1,28 @@ +// run + +// Copyright 2012 The Go Authors. All rights reserved. +// Use of this source code is governed by a BSD-style +// license that can be found in the LICENSE file. + +// Issue 3899: 8g incorrectly thinks a variable is +// "set but not used" and elides an assignment, causing +// variables to end up with wrong data. +// +// The reason is a miscalculation of variable width. + +package main + +func bar(f func()) { + f() +} + +func foo() { + f := func() {} + if f == nil { + } + bar(f) +} + +func main() { + foo() +}