]> Cypherpunks repositories - gostls13.git/commit
Revert "cmd/internal/goobj: add index to symbol name for indexed symbols"
authorCherry Zhang <cherryyz@google.com>
Tue, 2 Jun 2020 18:52:16 +0000 (14:52 -0400)
committerCherry Zhang <cherryyz@google.com>
Wed, 3 Jun 2020 19:20:41 +0000 (19:20 +0000)
commitb7717e46340b7ffe5fd53313f10dfa85a141f77a
treea504e5e228950afa3b83b4acf7576f900c328710
parent9984ef824c635421c225f5bf3d5573a1fbd94dde
Revert "cmd/internal/goobj: add index to symbol name for indexed symbols"

This reverts CL 229246.

For new indexed object files, in CL 229246 we added symbol index
to tools (nm, objdump) output. This affects external tools that
parse those outputs. And the added index doesn't look very nice.
In this release we take it out. For future releases we may
introduce a flag to tools (nm, objdump) and optionally dump the
symbol index.

For refererenced (not defined) indexed symbols, currently the
symbol is still referenced only by index, not by name. The next
CL will make the object file self-contained, so tools can dump
the symbol names properly (as before).

For #38875.

Change-Id: I07375e85a8e826e15c82fa452d11f0eaf8535a00
Reviewed-on: https://go-review.googlesource.com/c/go/+/236167
Reviewed-by: Than McIntosh <thanm@google.com>
Reviewed-by: Jeremy Faller <jeremy@golang.org>
src/cmd/internal/goobj/goobj_test.go
src/cmd/internal/goobj/readnew.go
src/cmd/nm/nm_test.go
src/cmd/objdump/objdump_test.go