This patch by Than McIntosh uncomments the call to trace.Stop in the testing package. It was commented out when the runtime/trace package did not build with gccgo, but now it does and has for some time. The fixes the go test -test.trace option. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline.
Ian
Index: gcc/go/gofrontend/MERGE =================================================================== --- gcc/go/gofrontend/MERGE (revision 263362) +++ gcc/go/gofrontend/MERGE (working copy) @@ -1,4 +1,4 @@ -8997a3afcc746824cb70b48b32d9c86b4814807d +274c88df4d6f9360dcd657b6e069a3b5a1d37a90 The first line of this file holds the git revision number of the last merge done from the gofrontend repository. Index: libgo/go/testing/testing.go =================================================================== --- libgo/go/testing/testing.go (revision 263362) +++ libgo/go/testing/testing.go (working copy) @@ -1159,7 +1159,7 @@ func (m *M) writeProfiles() { m.deps.StopCPUProfile() // flushes profile to disk } if *traceFile != "" { - // trace.Stop() // flushes trace to disk + trace.Stop() // flushes trace to disk } if *memProfile != "" { f, err := os.Create(toOutputDir(*memProfile))