]> Cypherpunks.ru repositories - gostls13.git/commit
cmd/compile/internal/pgo: remove most global state
authorMichael Pratt <mpratt@google.com>
Fri, 28 Oct 2022 21:34:43 +0000 (17:34 -0400)
committerMichael Pratt <mpratt@google.com>
Mon, 31 Oct 2022 21:18:33 +0000 (21:18 +0000)
commit204be97d24e4d6bbe2d5b8e85feb456e88fe0f6d
tree5ca8e9649672f4055b88dc14d051d04db642da0c
parent3848b44c754229125195e2f95bb71868d1a36f56
cmd/compile/internal/pgo: remove most global state

Since pgo is a new package, it is reasonably straightforward to
encapsulate its state into a non-global object that we pass around,
which will help keep it isolated.

There are no functional changes in this CL, just packaging up the
globals into a new object.

There are two major pieces of cleanup remaining:

1. reflectdata and noder have separate InlineCalls calls for method
   wrappers. The Profile is not plumbed there yet, but this is not a
   regression as the globals were previously set only right around the
   main inlining pass in gc.Main.

2. pgo.ListOfHotCallSites is still global, as it will require more work
   to clean up. It is effectively a local variable in InlinePackage,
   except that it assumes that InlineCalls is immediately preceded by a
   CanInline call for the same function. This is not necessarily true
   due to the recursive nature of CanInline. This also means that some
   InlineCalls calls may be missing the list of hot callsites right now.

For #55022.

Change-Id: Ic1fe41f73df96861c65f8bfeecff89862b367290
Reviewed-on: https://go-review.googlesource.com/c/go/+/446303
Reviewed-by: Cherry Mui <cherryyz@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Michael Pratt <mpratt@google.com>
src/cmd/compile/internal/gc/main.go
src/cmd/compile/internal/inline/inl.go
src/cmd/compile/internal/noder/reader.go
src/cmd/compile/internal/pgo/graph.go
src/cmd/compile/internal/pgo/irgraph.go
src/cmd/compile/internal/reflectdata/reflect.go