]> Cypherpunks repositories - gostls13.git/commit
[dev.link] cmd/link: add support for setting symbol File property
authorThan McIntosh <thanm@google.com>
Fri, 3 Jan 2020 20:25:28 +0000 (15:25 -0500)
committerThan McIntosh <thanm@google.com>
Thu, 9 Jan 2020 20:35:55 +0000 (20:35 +0000)
commit87f561ab2be1b4fde087c54154c5cdd1814b0f2c
tree5765daee75c303a518ef0ed940ffb924ffb47124
parentce7a0dda31b22a800aca199182901cbbc9a625a1
[dev.link] cmd/link: add support for setting symbol File property

Add loader methods to get/set the "file" symbol property.

In the legacy sym.Symbol struct, there is both a 'unit' field
(pointing to a CompilationUnit with package path, etc) and a 'file'
field. In the case of compiler-generated Go symbols, 'file' is
redundant (stores the package again), however for symbols read out of
a shared library (in the -linkshared case) it is important to be able
to record the file for a symbol.

With the loader, add a side table that can be used to store a file for
a symbol, and add methods for getting/setting file.

Change-Id: Iefceb8e7780f31457b658c099196de6e00be8aaf
Reviewed-on: https://go-review.googlesource.com/c/go/+/213421
Run-TryBot: Than McIntosh <thanm@google.com>
Reviewed-by: Jeremy Faller <jeremy@golang.org>
src/cmd/link/internal/loader/loader.go