cmd/link/internal/ld: re-enable DWARF tests on solaris/illumos

It looks like these are fixed on current tip after CL 84655
marked them to be skipped.

Fixes #23168

Change-Id: I0020e6da1042f723eb54186ef0fe925df5326230
Reviewed-on: https://go-review.googlesource.com/c/go/+/349250
Trust: Tobias Klauser <tobias.klauser@gmail.com>
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
Reviewed-by: Cherry Mui <cherryyz@google.com>
Reviewed-by: Than McIntosh <thanm@google.com>
TryBot-Result: Go Bot <gobot@golang.org>

diff --git a/src/cmd/link/internal/ld/dwarf_test.go b/src/cmd/link/internal/ld/dwarf_test.go
index 543dd5caac7454ade0419564fdc0bff028e38e15..0aeaa7565c839f73ba678f3ebe4735178fad1283 100644 (file)
--- a/src/cmd/link/internal/ld/dwarf_test.go
+++ b/src/cmd/link/internal/ld/dwarf_test.go
@@ -614,9 +614,6 @@ func TestInlinedRoutineRecords(t *testing.T) {
        if runtime.GOOS == "plan9" {
                t.Skip("skipping on plan9; no DWARF symbol table in executables")
        }
-       if runtime.GOOS == "solaris" || runtime.GOOS == "illumos" {
-               t.Skip("skipping on solaris, illumos, pending resolution of issue #23168")
-       }
 
        t.Parallel()
 
@@ -851,9 +848,6 @@ func TestAbstractOriginSanity(t *testing.T) {
        if runtime.GOOS == "plan9" {
                t.Skip("skipping on plan9; no DWARF symbol table in executables")
        }
-       if runtime.GOOS == "solaris" || runtime.GOOS == "illumos" {
-               t.Skip("skipping on solaris, illumos, pending resolution of issue #23168")
-       }
 
        if wd, err := os.Getwd(); err == nil {
                gopathdir := filepath.Join(wd, "testdata", "httptest")
@@ -869,9 +863,6 @@ func TestAbstractOriginSanityIssue25459(t *testing.T) {
        if runtime.GOOS == "plan9" {
                t.Skip("skipping on plan9; no DWARF symbol table in executables")
        }
-       if runtime.GOOS == "solaris" || runtime.GOOS == "illumos" {
-               t.Skip("skipping on solaris, illumos, pending resolution of issue #23168")
-       }
        if runtime.GOARCH != "amd64" && runtime.GOARCH != "386" {
                t.Skip("skipping on not-amd64 not-386; location lists not supported")
        }
@@ -890,9 +881,6 @@ func TestAbstractOriginSanityIssue26237(t *testing.T) {
        if runtime.GOOS == "plan9" {
                t.Skip("skipping on plan9; no DWARF symbol table in executables")
        }
-       if runtime.GOOS == "solaris" || runtime.GOOS == "illumos" {
-               t.Skip("skipping on solaris, illumos, pending resolution of issue #23168")
-       }
        if wd, err := os.Getwd(); err == nil {
                gopathdir := filepath.Join(wd, "testdata", "issue26237")
                abstractOriginSanity(t, gopathdir, DefaultOpt)