diff options
| author | Austin Clements <austin@google.com> | 2023-07-07 17:01:47 -0400 |
|---|---|---|
| committer | Austin Clements <austin@google.com> | 2023-08-22 19:18:34 +0000 |
| commit | 56b3b244fdd7a50fcd9ffb20d7cfdbfae3af81f0 (patch) | |
| tree | e209196818501db68c0a8f798bba7173d5c0bcaf /src/cmd/compile/internal/base | |
| parent | 83e0704c50c4269d90ec654b6590ae4db554f4eb (diff) | |
| download | go-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.go | 41 |
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"} |
