]> Cypherpunks repositories - gostls13.git/commit
misc/cgo/test: fix issue9400 test on android/386
authorElias Naur <elias.naur@gmail.com>
Thu, 2 Jun 2016 13:00:34 +0000 (15:00 +0200)
committerElias Naur <elias.naur@gmail.com>
Thu, 2 Jun 2016 14:01:47 +0000 (14:01 +0000)
commit6c4f8cd0d17c8147319effcffcb608fc42eaf307
tree2431a7310ec6fe79510303e54f4108ad2768bfb0
parent42c51debe824bd9b1fd93b3d50ff7187530754d3
misc/cgo/test: fix issue9400 test on android/386

The test for #9400 relies on an assembler function that manipulates
the stack pointer. Meanwile, it uses a global variable for
synchronization. However, position independent code on 386 use a
function call to fetch the base address for global variables.
That function call in turn overwrites the Go stack.

Fix that by fetching the global variable address once before the
stack register manipulation.

Fixes the android/386 builder.

Change-Id: Ib77bd80affaa12f09d582d09d8b84a73bd021b60
Reviewed-on: https://go-review.googlesource.com/23683
Run-TryBot: Elias Naur <elias.naur@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: David Crawshaw <crawshaw@golang.org>
misc/cgo/test/issue9400/asm_386.s