From c88893a0da8fd01f20b4e254f706f054a72fc435 Mon Sep 17 00:00:00 2001 From: Ian Lance Taylor Date: Tue, 6 Feb 2018 15:18:50 +0000 Subject: [PATCH] runtime: correct runtime structfield type to match reflect The offset field in structfield has changed to offsetAnon, and now requires a shift to get the actual offset value. Fixes golang/go#23391 Reviewed-on: https://go-review.googlesource.com/92275 From-SVN: r257413 --- gcc/go/gofrontend/MERGE | 2 +- libgo/go/runtime/cgocall.go | 2 +- libgo/go/runtime/type.go | 18 +++++++++++++----- 3 files changed, 15 insertions(+), 7 deletions(-) diff --git a/gcc/go/gofrontend/MERGE b/gcc/go/gofrontend/MERGE index ba8a8e0b48b..91bebe5d232 100644 --- a/gcc/go/gofrontend/MERGE +++ b/gcc/go/gofrontend/MERGE @@ -1,4 +1,4 @@ -c02c71187c9794b50444e2858c582e66a3442ee8 +1927b40e59e7c2067ecb03384b331d1be3cb5eea 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/runtime/cgocall.go b/libgo/go/runtime/cgocall.go index 9d161202dfa..d5794bc3341 100644 --- a/libgo/go/runtime/cgocall.go +++ b/libgo/go/runtime/cgocall.go @@ -189,7 +189,7 @@ func cgoCheckArg(t *_type, p unsafe.Pointer, indir, top bool, msg string) { return } for _, f := range st.fields { - cgoCheckArg(f.typ, add(p, f.offset), true, top, msg) + cgoCheckArg(f.typ, add(p, f.offset()), true, top, msg) } case kindPtr, kindUnsafePointer: if indir { diff --git a/libgo/go/runtime/type.go b/libgo/go/runtime/type.go index 6788f2425e1..0ec0da41179 100644 --- a/libgo/go/runtime/type.go +++ b/libgo/go/runtime/type.go @@ -113,11 +113,19 @@ type ptrtype struct { } type structfield struct { - name *string // nil for embedded fields - pkgPath *string // nil for exported Names; otherwise import path - typ *_type // type of field - tag *string // nil if no tag - offset uintptr // byte offset of field within struct + name *string // nil for embedded fields + pkgPath *string // nil for exported Names; otherwise import path + typ *_type // type of field + tag *string // nil if no tag + offsetAnon uintptr // byte offset of field<<1 | isAnonymous +} + +func (f *structfield) offset() uintptr { + return f.offsetAnon >> 1 +} + +func (f *structfield) anon() bool { + return f.offsetAnon&1 != 0 } type structtype struct { -- 2.30.2