From fa3e9d27f333d1cdb75889c2bff8e71d62f82e11 Mon Sep 17 00:00:00 2001 From: Josh Bleecher Snyder Date: Sun, 23 Apr 2017 05:06:22 -0700 Subject: [PATCH] cmd/compile: compile all functions concurrently CL 40693 added concurrent backend compilation support, and used it for user-provided functions. Autogenerated functions were still compiled serially. This CL brings them into the fold. As of this CL, when requested, no functions are compiled serially. There generally aren't many autogenerated functions. When there are, this CL can help a lot, because autogenerated functions are usually short. Many short functions is the best case scenario for concurrent compilation; see CL 41192. One example of such a package comes from Dave Cheney's benchjuju: github.com/juju/govmomi/vim25/types. It has thousands of autogenerated functions. This CL improves performance on the entire benchmark by around a second on my machine at c=8, or about ~5%. Updates #15756 Change-Id: Ia21e302b2469a9ed743df02244ec7ebde55b32f3 Reviewed-on: https://go-review.googlesource.com/41503 Run-TryBot: Josh Bleecher Snyder TryBot-Result: Gobot Gobot Reviewed-by: Robert Griesemer --- src/cmd/compile/internal/gc/main.go | 7 ------- src/cmd/compile/internal/gc/obj.go | 11 +++++++++++ 2 files changed, 11 insertions(+), 7 deletions(-) diff --git a/src/cmd/compile/internal/gc/main.go b/src/cmd/compile/internal/gc/main.go index 33fbb90be4..e72bdfa2d6 100644 --- a/src/cmd/compile/internal/gc/main.go +++ b/src/cmd/compile/internal/gc/main.go @@ -626,13 +626,6 @@ func Main(archInit func(*Arch)) { compileFunctions() - // We autogenerate and compile some small functions - // such as method wrappers and equality/hash routines - // while exporting code. - // Disable concurrent compilation from here on, - // at least until this convoluted structure has been unwound. - nBackendWorkers = 1 - if nowritebarrierrecCheck != nil { // Write barriers are now known. Check the // call graph. diff --git a/src/cmd/compile/internal/gc/obj.go b/src/cmd/compile/internal/gc/obj.go index a694e61099..6b33f2467b 100644 --- a/src/cmd/compile/internal/gc/obj.go +++ b/src/cmd/compile/internal/gc/obj.go @@ -140,6 +140,17 @@ func dumpLinkerObj(bout *bio.Writer) { dumpimportstrings() dumpbasictypes() + // The first call to dumpsignats can generate functions, + // like method wrappers and hash and equality routines. + compileFunctions() + + // Process any new signats added during compilation. + // No need to loop here; signats from compiling the generated + // functions should not themselves generate new functions. + // If they do, we'll know about it; the sanity check of + // len(compilequeue) in gc.Main will fail. + dumpsignats() + // Dump extra globals. tmp := externdcl -- 2.50.0