From 60a57222fe2166519b54641766bccab92fa15606 Mon Sep 17 00:00:00 2001 From: Ian Lance Taylor Date: Fri, 2 Feb 2018 00:33:03 +0000 Subject: [PATCH] reflect: enable allocation tests They were disabled due to the lack of escape analysis. Now that we have escape analysis, unskip these tests. Reviewed-on: https://go-review.googlesource.com/86248 From-SVN: r257324 --- gcc/go/gofrontend/MERGE | 2 +- libgo/go/reflect/all_test.go | 6 ------ 2 files changed, 1 insertion(+), 7 deletions(-) diff --git a/gcc/go/gofrontend/MERGE b/gcc/go/gofrontend/MERGE index 46bdc3c67da..162c4a85b38 100644 --- a/gcc/go/gofrontend/MERGE +++ b/gcc/go/gofrontend/MERGE @@ -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. diff --git a/libgo/go/reflect/all_test.go b/libgo/go/reflect/all_test.go index 86e738555f8..3070aac0fe0 100644 --- a/libgo/go/reflect/all_test.go +++ b/libgo/go/reflect/all_test.go @@ -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)) -- 2.30.2