mirror of
https://github.com/golang/go.git
synced 2025-12-08 06:10:04 +00:00
cmd/compile: don't treat devel compiler as a released compiler
The compiler has a logic to print different messages on internal compiler error depending on whether this is a released version of Go. It hides the panic stack trace if it is a released version. It does this by checking the version and see if it has a "go" prefix. This includes all the released versions. However, for a non- released build, if there is no explicit version set, cmd/dist now sets the toolchain version as go1.X-devel_XXX, which makes it be treated as a released compiler, and causes the stack trace to be hidden. Change the logic to not match a devel compiler as a released compiler. Cherry-picked from the dev.simd branch. This CL is not necessarily SIMD specific. Apply early to reduce risk. Change-Id: I5d3b2101527212f825b6e4000b36030c4f83870b Reviewed-on: https://go-review.googlesource.com/c/go/+/682975 Reviewed-by: David Chase <drchase@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-on: https://go-review.googlesource.com/c/go/+/708855 Reviewed-by: Junyang Shao <shaojunyang@google.com>
This commit is contained in:
parent
c54dc1418b
commit
ebb72bef44
1 changed files with 1 additions and 1 deletions
|
|
@ -220,7 +220,7 @@ func FatalfAt(pos src.XPos, format string, args ...interface{}) {
|
||||||
fmt.Printf("\n")
|
fmt.Printf("\n")
|
||||||
|
|
||||||
// If this is a released compiler version, ask for a bug report.
|
// If this is a released compiler version, ask for a bug report.
|
||||||
if Debug.Panic == 0 && strings.HasPrefix(buildcfg.Version, "go") {
|
if Debug.Panic == 0 && strings.HasPrefix(buildcfg.Version, "go") && !strings.Contains(buildcfg.Version, "devel") {
|
||||||
fmt.Printf("\n")
|
fmt.Printf("\n")
|
||||||
fmt.Printf("Please file a bug report including a short program that triggers the error.\n")
|
fmt.Printf("Please file a bug report including a short program that triggers the error.\n")
|
||||||
fmt.Printf("https://go.dev/issue/new\n")
|
fmt.Printf("https://go.dev/issue/new\n")
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue