Commit 62581ee9 authored by Than McIntosh's avatar Than McIntosh

cmd/link: don't run TestDWARF in c-archive mode on Windows

Test fix: in dwarf_test.go don't try to run the TestDWARF testpoint on
windows with c-archive build mode (linker + debug/pe support for that
build mode on Windows is not fully baked it seems).

Fixes #35512.

Change-Id: I1c87ff3d62e5b98e75062b184d762fb5ed937745
Reviewed-on: https://go-review.googlesource.com/c/go/+/206899
Run-TryBot: Than McIntosh <thanm@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: default avatarBryan C. Mills <bcmills@google.com>
parent 9a9a9005
...@@ -139,13 +139,6 @@ func testDWARF(t *testing.T, buildmode string, expectDWARF bool, env ...string) ...@@ -139,13 +139,6 @@ func testDWARF(t *testing.T, buildmode string, expectDWARF bool, env ...string)
} }
} }
// Until there is a fix for issue 35512, don't try to use
// SeekPC or look at the line table on Windows under
// c-archive build mode.
if buildmode == "c-archive" && runtime.GOOS == "windows" {
t.Skip("avoiding SeekPC until 35512 fixed")
}
// TODO: We'd like to use filepath.Join here. // TODO: We'd like to use filepath.Join here.
// Also related: golang.org/issue/19784. // Also related: golang.org/issue/19784.
wantFile := path.Join(prog, "main.go") wantFile := path.Join(prog, "main.go")
...@@ -175,6 +168,9 @@ func testDWARF(t *testing.T, buildmode string, expectDWARF bool, env ...string) ...@@ -175,6 +168,9 @@ func testDWARF(t *testing.T, buildmode string, expectDWARF bool, env ...string)
func TestDWARF(t *testing.T) { func TestDWARF(t *testing.T) {
testDWARF(t, "", true) testDWARF(t, "", true)
if !testing.Short() { if !testing.Short() {
if runtime.GOOS == "windows" {
t.Skip("skipping Windows/c-archive; see Issue 35512 for more.")
}
t.Run("c-archive", func(t *testing.T) { t.Run("c-archive", func(t *testing.T) {
testDWARF(t, "c-archive", true) testDWARF(t, "c-archive", true)
}) })
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment