[release-branch.go1.9] cmd/compile: omit ICE diagnostics after normal error messages

After we detect errors, the AST is in a precarious state and more
likely to trip useless ICE failures. Instead let the user fix any
existing errors and see if the ICE persists.  This makes Fatalf more
consistent with how panics are handled by hidePanic.

While here, also fix detection for release versions: release version
strings begin with "go" ("go1.8", "go1.9.1", etc), not "release".

Fixes #22252.

Change-Id: I1c400af62fb49dd979b96e1bf0fb295a81c8b336
Reviewed-on: https://go-review.googlesource.com/70850
Run-TryBot: Matthew Dempsky <mdempsky@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>
Reviewed-on: https://go-review.googlesource.com/70985
Run-TryBot: Russ Cox <rsc@golang.org>
Reviewed-by: Matthew Dempsky <mdempsky@google.com>

diff --git a/src/cmd/compile/internal/gc/subr.go b/src/cmd/compile/internal/gc/subr.go
index d79789c4febca2f02f429fe3fad93c4dc85aa683..047acee05f7d12ac840a55a1c8c08077e23b212b 100644 (file)
--- a/src/cmd/compile/internal/gc/subr.go
+++ b/src/cmd/compile/internal/gc/subr.go
@@ -166,20 +166,22 @@ func Warnl(line src.XPos, fmt_ string, args ...interface{}) {
 func Fatalf(fmt_ string, args ...interface{}) {
        flusherrors()
 
-       fmt.Printf("%v: internal compiler error: ", linestr(lineno))
-       fmt.Printf(fmt_, args...)
-       fmt.Printf("\n")
-
-       // If this is a released compiler version, ask for a bug report.
-       if strings.HasPrefix(objabi.Version, "release") {
+       if Debug_panic != 0 || nsavederrors+nerrors == 0 {
+               fmt.Printf("%v: internal compiler error: ", linestr(lineno))
+               fmt.Printf(fmt_, args...)
                fmt.Printf("\n")
-               fmt.Printf("Please file a bug report including a short program that triggers the error.\n")
-               fmt.Printf("https://golang.org/issue/new\n")
-       } else {
-               // Not a release; dump a stack trace, too.
-               fmt.Println()
-               os.Stdout.Write(debug.Stack())
-               fmt.Println()
+
+               // If this is a released compiler version, ask for a bug report.
+               if strings.HasPrefix(objabi.Version, "go") {
+                       fmt.Printf("\n")
+                       fmt.Printf("Please file a bug report including a short program that triggers the error.\n")
+                       fmt.Printf("https://golang.org/issue/new\n")
+               } else {
+                       // Not a release; dump a stack trace, too.
+                       fmt.Println()
+                       os.Stdout.Write(debug.Stack())
+                       fmt.Println()
+               }
        }
 
        hcrash()