aboutsummaryrefslogtreecommitdiff
path: root/src/cmd/compile/internal/base
diff options
context:
space:
mode:
authorAustin Clements <austin@google.com>2023-07-07 17:01:47 -0400
committerAustin Clements <austin@google.com>2023-08-22 19:18:34 +0000
commit56b3b244fdd7a50fcd9ffb20d7cfdbfae3af81f0 (patch)
treee209196818501db68c0a8f798bba7173d5c0bcaf /src/cmd/compile/internal/base
parent83e0704c50c4269d90ec654b6590ae4db554f4eb (diff)
downloadgo-56b3b244fdd7a50fcd9ffb20d7cfdbfae3af81f0.tar.xz
cmd/internal/objabi,cmd/compile: move NoInstrumentPkgs and NoRacePkgs to PkgSpecials
This consolidates the NoInstrumentPkgs and NoRacePkgs lists into the objabi.LookupPkgSpecial mechanism. Change-Id: I411654afdd690fb01c412e7e8b57ddfbe85415e3 Reviewed-on: https://go-review.googlesource.com/c/go/+/521702 TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Matthew Dempsky <mdempsky@google.com> Run-TryBot: Austin Clements <austin@google.com>
Diffstat (limited to 'src/cmd/compile/internal/base')
-rw-r--r--src/cmd/compile/internal/base/base.go41
1 files changed, 0 insertions, 41 deletions
diff --git a/src/cmd/compile/internal/base/base.go b/src/cmd/compile/internal/base/base.go
index 841241efb1..ee3772c5ca 100644
--- a/src/cmd/compile/internal/base/base.go
+++ b/src/cmd/compile/internal/base/base.go
@@ -219,44 +219,3 @@ func AdjustStartingHeap(requestedHeapGoal uint64) {
forEachGC(adjustFunc)
}
-
-func Compiling(pkgs []string) bool {
- if Ctxt.Pkgpath != "" {
- for _, p := range pkgs {
- if Ctxt.Pkgpath == p {
- return true
- }
- }
- }
-
- return false
-}
-
-// Do not instrument the following packages at all,
-// at best instrumentation would cause infinite recursion.
-var NoInstrumentPkgs = []string{
- "runtime/internal/atomic",
- "runtime/internal/math",
- "runtime/internal/sys",
- "runtime/internal/syscall",
- "runtime",
- "runtime/race",
- "runtime/msan",
- "runtime/asan",
- "internal/cpu",
- "internal/abi",
- // We omit bytealg even though it's imported by runtime because it also
- // backs a lot of package bytes. Currently we don't have a way to omit race
- // instrumentation when used from the runtime while keeping race
- // instrumentation when used from user code. Somehow this doesn't seem to
- // cause problems, though we may be skating on thin ice. See #61204.
- //"internal/bytealg",
- "internal/coverage/rtcov",
- "internal/godebugs",
- "internal/goexperiment",
- "internal/goos",
-}
-
-// Don't insert racefuncenter/racefuncexit into the following packages.
-// Memory accesses in the packages are either uninteresting or will cause false positives.
-var NoRacePkgs = []string{"sync", "sync/atomic"}