cmd/link: remove superfluous check in TestIssue34788Android386TLSSequence err != nil is already checked in the if condition one line above. Change-Id: If36cdb41016f7be98a65be0a7211d85cd6017f87 Reviewed-on: https://go-review.googlesource.com/c/go/+/250477 Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Cherry Zhang <cherryyz@google.com>
diff --git a/src/cmd/link/link_test.go b/src/cmd/link/link_test.go index 3b5efdf..72ff01c 100644 --- a/src/cmd/link/link_test.go +++ b/src/cmd/link/link_test.go
@@ -455,9 +455,7 @@ cmd := exec.Command(testenv.GoToolPath(t), "tool", "compile", "-o", obj, src) cmd.Env = append(os.Environ(), "GOARCH=386", "GOOS=android") if out, err := cmd.CombinedOutput(); err != nil { - if err != nil { - t.Fatalf("failed to compile blah.go: %v, output: %s\n", err, out) - } + t.Fatalf("failed to compile blah.go: %v, output: %s\n", err, out) } // Run objdump on the resulting object.