Skip to content
This repository has been archived by the owner on May 2, 2018. It is now read-only.

Commit

Permalink
test/fixedbugs: add s390x case to issue11656
Browse files Browse the repository at this point in the history
An instruction consisting of all 0s causes an illegal instruction
signal on s390x. Since 0s are the default in this test this CL just
makes it explicit.

Change-Id: Id6e060eed1a588f4b10a4e4861709fcd19b434ac
Reviewed-on: https://go-review.googlesource.com/20962
Reviewed-by: Minux Ma <[email protected]>
Run-TryBot: Brad Fitzpatrick <[email protected]>
TryBot-Result: Gobot Gobot <[email protected]>
  • Loading branch information
mundaym authored and bradfitz committed Mar 21, 2016
1 parent 992320a commit 1a82946
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions test/fixedbugs/issue11656.go
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,8 @@ func f(n int) {
binary.BigEndian.PutUint32(ill, 0x00000034) // trap
case "mips64le":
binary.LittleEndian.PutUint32(ill, 0x00000034) // trap
case "s390x":
binary.BigEndian.PutUint32(ill, 0) // undefined instruction
default:
// Just leave it as 0 and hope for the best.
}
Expand Down

0 comments on commit 1a82946

Please sign in to comment.