diff options
| author | Cherry Zhang <cherryyz@google.com> | 2020-04-17 20:41:59 -0400 |
|---|---|---|
| committer | Cherry Zhang <cherryyz@google.com> | 2020-04-18 01:09:57 +0000 |
| commit | b0da26a668fd6d4e351a00ca76695c5a233e84a2 (patch) | |
| tree | 2bb2e9f24e6202a0b39784a5c278a5ab8b41290c /src/os/exec/exec_test.go | |
| parent | 4d9ecde30a77f4a4197b585b42cc2117607a8c40 (diff) | |
| download | go-b0da26a668fd6d4e351a00ca76695c5a233e84a2.tar.xz | |
cmd/link: stop checking reflect.Value.Call in deadcode pass
In the linker's deadcode pass, we need to keep a method live if
it can be reached through reflection. We do this by marking all
exported method live if reflect.Value.Method or
reflect.Type.Method is used. Currently we also check for
reflect.Value.Call, which is unnecessary because in order to call
a method through reflection, the method must be obtained through
reflect.Value.Method or reflect.Type.Method, which we already
check.
Per discussion in https://groups.google.com/d/msg/golang-dev/eG9It63-Bxg/_bnoVy-eAwAJ
Thanks Brad, Russ, and Ian for bringing this up.
Change-Id: I8e9529a224bb898dbf5752674cc9d155db386c14
Reviewed-on: https://go-review.googlesource.com/c/go/+/228792
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Diffstat (limited to 'src/os/exec/exec_test.go')
0 files changed, 0 insertions, 0 deletions