From: Cherry Zhang Date: Thu, 26 Mar 2020 18:43:15 +0000 (-0400) Subject: [dev.link] cmd/link: migrate symbol's Value and Align when converting to sym.Symbols X-Git-Tag: go1.15beta1~679^2~28 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=c7e3d817ab48a6a8a0221d78a54d76bbb4e07e4b;p=gostls13.git [dev.link] cmd/link: migrate symbol's Value and Align when converting to sym.Symbols Currently, in LoadFull we migrate a symbol's Value to sym.Symbol only for external symbols. And symbol's Align is not migrated at all. As we move LoadFull forward, there are already places where we set symbol's Value and Align (e.g. in doelf). Migrate them correctly. Currently I think we only set them on external symbols, but as we move forward I think we'll need to set them on Go symbols as well. Change-Id: I63e97e38fc08b653ba9faefe15697944faf21bed Reviewed-on: https://go-review.googlesource.com/c/go/+/225658 Run-TryBot: Cherry Zhang TryBot-Result: Gobot Gobot Reviewed-by: Jeremy Faller --- diff --git a/src/cmd/link/internal/loader/loader.go b/src/cmd/link/internal/loader/loader.go index 7e8469f501..e342fbe680 100644 --- a/src/cmd/link/internal/loader/loader.go +++ b/src/cmd/link/internal/loader/loader.go @@ -1833,11 +1833,9 @@ func (l *Loader) LoadFull(arch *sys.Arch, syms *sym.Symbols) { s.Version = int16(pp.ver) s.Type = pp.kind s.Size = pp.size - s.Value = l.SymValue(i) if pp.gotype != 0 { s.Gotype = l.Syms[pp.gotype] } - s.Value = l.values[i] if f, ok := l.symFile[i]; ok { s.File = f } else if pp.objidx != 0 { @@ -2326,6 +2324,9 @@ func (l *Loader) CopyAttributes(src Sym, dst Sym) { // migrateAttributes copies over all of the attributes of symbol 'src' to // sym.Symbol 'dst'. func (l *Loader) migrateAttributes(src Sym, dst *sym.Symbol) { + dst.Value = l.SymValue(src) + dst.Align = l.SymAlign(src) + dst.Attr.Set(sym.AttrReachable, l.AttrReachable(src)) dst.Attr.Set(sym.AttrOnList, l.AttrOnList(src)) dst.Attr.Set(sym.AttrLocal, l.AttrLocal(src))