From 4e4e51c5c5a1cedf4d2e8fef59b61eeaddb10647 Mon Sep 17 00:00:00 2001 From: Josh Bleecher Snyder Date: Thu, 13 Apr 2017 05:57:59 -0700 Subject: cmd/internal/obj: generate function DWARF symbols early This removes a concurrent access of ctxt.Data. Updates #15756 Change-Id: Id017e90e47e093cd8825907f3853bb3d3bf8280d Reviewed-on: https://go-review.googlesource.com/40507 Run-TryBot: Josh Bleecher Snyder TryBot-Result: Gobot Gobot Reviewed-by: Matthew Dempsky --- src/cmd/internal/obj/objfile.go | 21 +++++++++++++-------- 1 file changed, 13 insertions(+), 8 deletions(-) (limited to 'src/cmd/internal/obj/objfile.go') diff --git a/src/cmd/internal/obj/objfile.go b/src/cmd/internal/obj/objfile.go index 6858143674..a4a9091bb9 100644 --- a/src/cmd/internal/obj/objfile.go +++ b/src/cmd/internal/obj/objfile.go @@ -555,19 +555,24 @@ func (c dwCtxt) AddSectionOffset(s dwarf.Sym, size int, t interface{}, ofs int64 r.Type = R_DWARFREF } -// makeFuncDebugEntry makes a DWARF Debugging Information Entry -// for TEXT symbol s. -func makeFuncDebugEntry(ctxt *Link, curfn interface{}, s *LSym) { - dsym := ctxt.Lookup(dwarf.InfoPrefix+s.Name, int(s.Version)) +// dwarfSym returns the DWARF symbol for TEXT symbol. +func (ctxt *Link) dwarfSym(s *LSym) *LSym { + if s.Type != STEXT { + ctxt.Diag("dwarfSym of non-TEXT %v", s) + } + return ctxt.Lookup(dwarf.InfoPrefix+s.Name, int(s.Version)) +} + +// populateDWARF fills in the DWARF Debugging Information Entry for TEXT symbol s. +// The DWARF symbol must already have been initialized in InitTextSym. +func (ctxt *Link) populateDWARF(curfn interface{}, s *LSym) { + dsym := ctxt.dwarfSym(s) if dsym.Size != 0 { - return + ctxt.Diag("makeFuncDebugEntry double process %v", s) } - dsym.Type = SDWARFINFO - dsym.Set(AttrDuplicateOK, s.DuplicateOK()) var vars []*dwarf.Var if ctxt.DebugInfo != nil { vars = ctxt.DebugInfo(s, curfn) } dwarf.PutFunc(dwCtxt{ctxt}, dsym, s.Name, s.Version == 0, s, s.Size, vars) - ctxt.Data = append(ctxt.Data, dsym) } -- cgit v1.3