mirror of https://github.com/golang/go.git
go/ssa: remove workaround for lack of vendoring in go/loader
Support for vendoring in go/loader was added in golang.org/cl/18053 Similar workaround was removed from go/loader tests in golang.org/cl/18214 Resolves TODO left by adonovan Change-Id: I418463feb3e2dded191b6f1770dd7b9e44c2deca GitHub-Last-Rev: b556acfce87ab1bac9a73b69202a1afdd6e641cc GitHub-Pull-Request: golang/tools#199 Reviewed-on: https://go-review.googlesource.com/c/tools/+/215458 Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
This commit is contained in:
parent
e919f86751
commit
eb0d8dd85b
|
|
@ -28,18 +28,6 @@ import (
|
|||
"golang.org/x/tools/internal/testenv"
|
||||
)
|
||||
|
||||
// Skip the set of packages that transitively depend on
|
||||
// cmd/internal/objfile, which uses vendoring,
|
||||
// which go/loader does not yet support.
|
||||
// TODO(adonovan): add support for vendoring and delete this.
|
||||
var skip = map[string]bool{
|
||||
"cmd/addr2line": true,
|
||||
"cmd/internal/objfile": true,
|
||||
"cmd/nm": true,
|
||||
"cmd/objdump": true,
|
||||
"cmd/pprof": true,
|
||||
}
|
||||
|
||||
func bytesAllocated() uint64 {
|
||||
runtime.GC()
|
||||
var stats runtime.MemStats
|
||||
|
|
@ -62,9 +50,6 @@ func TestStdlib(t *testing.T) {
|
|||
ctxt.GOPATH = "" // disable GOPATH
|
||||
conf := loader.Config{Build: &ctxt}
|
||||
for _, path := range buildutil.AllPackages(conf.Build) {
|
||||
if skip[path] {
|
||||
continue
|
||||
}
|
||||
conf.ImportWithTests(path)
|
||||
}
|
||||
|
||||
|
|
|
|||
Loading…
Reference in New Issue