Skip to content

Commit

Permalink
test: enable fixedbugs/bug193.go for -G compiler option
Browse files Browse the repository at this point in the history
Temporarily disable a questionable test case in fixedbugs/bug193.go
and enable the test as a whole. See the issues below for details.

Updates #45114.
Updates #45117.

Change-Id: I1de6f8d79b592eeeec139cd92b6c9cac56a9a74b
Reviewed-on: https://go-review.googlesource.com/c/go/+/303094
Trust: Robert Griesemer <[email protected]>
Reviewed-by: Emmanuel Odeke <[email protected]>
  • Loading branch information
griesemer committed Mar 23, 2021
1 parent cd26192 commit f4b9183
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
4 changes: 3 additions & 1 deletion test/fixedbugs/bug193.go
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,8 @@ func main() {
ss := 1 << s
y1 := float64(ss)
y2 := float64(1 << s) // ERROR "shift"
y3 := string(1 << s) // ERROR "shift"
// see issues #45114, #45117
// y3 := string(1 << s) // DISABLED "shift"
y3 := 0
_, _, _, _, _ = s, ss, y1, y2, y3
}
1 change: 0 additions & 1 deletion test/run.go
Original file line number Diff line number Diff line change
Expand Up @@ -1939,7 +1939,6 @@ var excluded = map[string]bool{
"writebarrier.go": true, // correct diagnostics, but different lines (probably irgen's fault)

"fixedbugs/bug176.go": true, // types2 reports all errors (pref: types2)
"fixedbugs/bug193.go": true, // types2 bug: shift error not reported (fixed in go/types)
"fixedbugs/bug195.go": true, // types2 reports slightly different (but correct) bugs
"fixedbugs/bug228.go": true, // types2 not run after syntax errors
"fixedbugs/bug231.go": true, // types2 bug? (same error reported twice)
Expand Down

0 comments on commit f4b9183

Please sign in to comment.