Skip to content

Commit

Permalink
test: add a test for gccgo bug golang#32901
Browse files Browse the repository at this point in the history
This CL adds a test for gccgo bug golang#32901: not all the type
descriptors are registered and thus deduplicated with types
created by reflection. It needs a few levels of indirect imports
to trigger this bug.

Updates golang#32901.

Change-Id: Idbd89bedd63fea746769f2687f3f31c9767e5ec0
Reviewed-on: https://go-review.googlesource.com/c/go/+/184718
Reviewed-by: Ian Lance Taylor <[email protected]>
  • Loading branch information
cherrymui committed Jul 3, 2019
1 parent e2fdce9 commit 1c1e517
Show file tree
Hide file tree
Showing 5 changed files with 74 additions and 0 deletions.
15 changes: 15 additions & 0 deletions test/fixedbugs/issue32901.dir/a.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
// Copyright 2019 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.

package a

type T struct { x int }

func F() interface{} {
return [2]T{}
}

func P() interface{} {
return &[2]T{}
}
15 changes: 15 additions & 0 deletions test/fixedbugs/issue32901.dir/b.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
// Copyright 2019 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.

package b

import "./a"

func F() interface{} {
return a.F()
}

func P() interface{} {
return a.P()
}
17 changes: 17 additions & 0 deletions test/fixedbugs/issue32901.dir/c.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
// Copyright 2019 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.

package c

import "./b"

func F() interface{} {
go func(){}() // make it non-inlineable
return b.F()
}

func P() interface{} {
go func(){}() // make it non-inlineable
return b.P()
}
18 changes: 18 additions & 0 deletions test/fixedbugs/issue32901.dir/main.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
// Copyright 2019 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.

package main

import "./c"
import "reflect"

func main() {
x := c.F()
p := c.P()
t := reflect.PtrTo(reflect.TypeOf(x))
tp := reflect.TypeOf(p)
if t != tp {
panic("FAIL")
}
}
9 changes: 9 additions & 0 deletions test/fixedbugs/issue32901.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
// rundir

// Copyright 2019 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 32901: type descriptor equality bug in gccgo.

package ignored

0 comments on commit 1c1e517

Please sign in to comment.