Skip to content

Commit

Permalink
cmd/compile/internal/gc: better inliner diagnostics
Browse files Browse the repository at this point in the history
When debugging inliner with -m -m print cost of complex functions,
instead of simple "function too complex". This helps to understand,
how close to inlining is this particular function.

Change-Id: I6871f69b5b914d23fd0b43a24d7c6fc928f4b716
Reviewed-on: https://go-review.googlesource.com/63330
Run-TryBot: Ilya Tocar <ilya.tocar@intel.com>
Reviewed-by: Matthew Dempsky <mdempsky@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
  • Loading branch information
TocarIP committed Sep 20, 2017
1 parent 8e5ac83 commit 475df0e
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/cmd/compile/internal/gc/inl.go
Expand Up @@ -157,7 +157,7 @@ func caninl(fn *Node) {
return
}
if visitor.budget < 0 {
reason = "function too complex"
reason = fmt.Sprintf("function too complex: cost %d exceeds budget %d", maxBudget-visitor.budget, maxBudget)
return
}

Expand Down Expand Up @@ -297,8 +297,8 @@ func (v *hairyVisitor) visit(n *Node) bool {
v.budget -= 2
}

if v.budget < 0 {
v.reason = "function too complex"
// When debugging, don't stop early, to get full cost of inlining this function
if v.budget < 0 && Debug['m'] < 2 {
return true
}

Expand Down

0 comments on commit 475df0e

Please sign in to comment.