diff options
| author | Than McIntosh <thanm@google.com> | 2023-01-03 13:11:10 -0500 |
|---|---|---|
| committer | Than McIntosh <thanm@google.com> | 2023-01-24 13:28:54 +0000 |
| commit | 733ba921875ea11088f1f447cbca418f651aae5c (patch) | |
| tree | 11d60bc069e31b46c7fe78d60157072bc50c57f4 /test/inline_big.go | |
| parent | 1951857ec07c1d491e1836770a647d3902934a67 (diff) | |
| download | go-733ba921875ea11088f1f447cbca418f651aae5c.tar.xz | |
cmd/compile: flag 'large' functions when -m=2+ in effect
When -m=N (where N > 1) is in effect, include a note in the trace
output if a given function is considered "big" during inlining
analysis, since this causes the inliner to be less aggressive. If a
small change to a large function happens to nudge it over the large
function threshold, it can be confusing for developers, thus it's
probably worth including this info in the remark output.
Change-Id: Id31a1b76371ab1ef9265ba28a377f97b0247d0a7
Reviewed-on: https://go-review.googlesource.com/c/go/+/460317
Reviewed-by: Keith Randall <khr@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Than McIntosh <thanm@google.com>
Reviewed-by: Keith Randall <khr@google.com>
Diffstat (limited to 'test/inline_big.go')
| -rw-r--r-- | test/inline_big.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/inline_big.go b/test/inline_big.go index 83672753f7..7dd1abdb6a 100644 --- a/test/inline_big.go +++ b/test/inline_big.go @@ -20,7 +20,7 @@ func medium(a []int) int { // ERROR "can inline medium with cost .* as:.*" "a do return a[0] + a[1] + a[2] + a[3] + a[4] + a[5] + a[6] + a[7] } -func f(a []int) int { // ERROR "cannot inline f:.*" "a does not escape" +func f(a []int) int { // ERROR "cannot inline f:.*" "a does not escape" "function f considered 'big'" // Add lots of nodes to f's body. We need >5000. // See cmd/compile/internal/gc/inl.go:inlineBigFunction* a[0] = 0 |