Commit ac8d97b6 authored by Todd Neal's avatar Todd Neal

cmd/compile: fix inlining of switch issue

The issue was seen when inlining an exported function that contained
a fallthrough statement.

Fixes #15071

Change-Id: I1e8215ad49d57673dba7e8f8bd2ed8ad290dc452
Reviewed-on: https://go-review.googlesource.com/21452Reviewed-by: default avatarDave Cheney <dave@cheney.net>
parent 6a0bb87b
...@@ -189,6 +189,7 @@ var goopnames = []string{ ...@@ -189,6 +189,7 @@ var goopnames = []string{
OSUB: "-", OSUB: "-",
OSWITCH: "switch", OSWITCH: "switch",
OXOR: "^", OXOR: "^",
OXFALL: "fallthrough",
} }
// Fmt "%O": Node opcodes // Fmt "%O": Node opcodes
......
// Copyright 2016 The Go Authors. All rights reserved.
// Use of this source code is governed by a BSD-style
// license that can be found in the LICENSE file.
package exp
func Exported(x int) int {
return inlined(x)
}
func inlined(x int) int {
y := 0
switch {
case x > 0:
y += 5
return 0 + y
case x < 1:
y += 6
fallthrough
default:
y += 7
return 2 + y
}
}
// run
// Copyright 2016 The Go Authors. All rights reserved.
// Use of this source code is governed by a BSD-style
// license that can be found in the LICENSE file.
package main
import "os"
import "./exp"
func main() {
_ = exp.Exported(len(os.Args))
}
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