1
0
mirror of https://github.com/golang/go synced 2024-11-26 09:28:07 -07:00

cmd/compile: fix buglet in inlined info abstract function dwarf-gen

When generating DWARF inlined info records, it's possible to have a
local function whose only callsites are inlined away, meaning that we
emit an abstract function DIE but no regular subprogram DIE. When
emitting DWARF scope info we need to handle this case (specifically
when scoping PCs, check for the case that the func in question has
been entirely deleted).

Fixes #44344.

Change-Id: I9f5bc692f225aa4c5c23f7bd2e50bcf7fe4fc5f3
Reviewed-on: https://go-review.googlesource.com/c/go/+/293309
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
Reviewed-by: Russ Cox <rsc@golang.org>
Trust: Than McIntosh <thanm@google.com>
Run-TryBot: Than McIntosh <thanm@google.com>
This commit is contained in:
Than McIntosh 2021-02-17 15:51:05 -05:00
parent 7764ee5614
commit fce2a94d84
2 changed files with 33 additions and 1 deletions

View File

@ -37,7 +37,9 @@ func assembleScopes(fnsym *obj.LSym, fn *ir.Func, dwarfVars []*dwarf.Var, varSco
} }
scopeVariables(dwarfVars, varScopes, dwarfScopes) scopeVariables(dwarfVars, varScopes, dwarfScopes)
if fnsym.Func().Text != nil {
scopePCs(fnsym, fn.Marks, dwarfScopes) scopePCs(fnsym, fn.Marks, dwarfScopes)
}
return compactScopes(dwarfScopes) return compactScopes(dwarfScopes)
} }

View File

@ -0,0 +1,30 @@
// compile
// Copyright 2021 The Go Authors. All rights reserved.
// Use of this source code is governed by a BSD-style
// license that can be found in the LICENSE file.
// Issue #44344: a crash in DWARF scope generation (trying to
// scope the PCs of a function that was inlined away).
package main
func main() {
pv := []int{3, 4, 5}
if pv[1] != 9 {
pv = append(pv, 9)
}
tryit := func() bool {
lpv := len(pv)
if lpv == 101 {
return false
}
if worst := pv[pv[1]&1]; worst != 101 {
return true
}
return false
}()
if tryit {
println(pv[0])
}
}