diff mbox series

libgo patch committed: Enable escape analysis tests in reflect package

Message ID CAOyqgcV+hqUbuz3o4kO2WSxibrAZtHFxmOEgEQOTeYe9rUSiCA@mail.gmail.com
State New
Headers show
Series libgo patch committed: Enable escape analysis tests in reflect package | expand

Commit Message

Ian Lance Taylor Feb. 2, 2018, 12:33 a.m. UTC 
This patch by Cherry Zhang enables some tests in the reflect package
that depend on escape analysis being enabled by default.  Bootstrapped
and ran Go testsuite on x86_64-pc-linux-gnu.  Committed to mainline.

Ian
diff mbox series

Patch

Index: gcc/go/gofrontend/MERGE
===================================================================
--- gcc/go/gofrontend/MERGE	(revision 257323)
+++ gcc/go/gofrontend/MERGE	(working copy)
@@ -1,4 +1,4 @@ 
-5e8a91bf239c253d7b5c84bd2c1dd3ecb18980e9
+b332ba2f0d0302eeb01a228c217928296cec56f6
 
 The first line of this file holds the git revision number of the last
 merge done from the gofrontend repository.
Index: libgo/go/reflect/all_test.go
===================================================================
--- libgo/go/reflect/all_test.go	(revision 257312)
+++ libgo/go/reflect/all_test.go	(working copy)
@@ -6162,9 +6162,6 @@  func TestPtrToMethods(t *testing.T) {
 }
 
 func TestMapAlloc(t *testing.T) {
-	if runtime.Compiler == "gccgo" {
-		t.Skip("skipping on gccgo until we have escape analysis")
-	}
 	m := ValueOf(make(map[int]int, 10))
 	k := ValueOf(5)
 	v := ValueOf(7)
@@ -6195,9 +6192,6 @@  func TestMapAlloc(t *testing.T) {
 }
 
 func TestChanAlloc(t *testing.T) {
-	if runtime.Compiler == "gccgo" {
-		t.Skip("skipping on gccgo until we have escape analysis")
-	}
 	// Note: for a chan int, the return Value must be allocated, so we
 	// use a chan *int instead.
 	c := ValueOf(make(chan *int, 1))