This patch to the sync/atomic package disables a couple of tests on systems with an inappropriate pointer size. The 32-bit test was clobbering 4 bytes on the heap. The 64-bit test only worked on little-endian systems. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r d75f0efa5fc2 libgo/go/sync/atomic/atomic_test.go --- a/libgo/go/sync/atomic/atomic_test.go Fri Feb 10 23:04:22 2012 -0800 +++ b/libgo/go/sync/atomic/atomic_test.go Sat Feb 11 21:54:35 2012 -0800 @@ -636,6 +636,7 @@ // 64-bit system; clear uintptr tests hammer32[2].f = nil hammer32[5].f = nil + hammer32[6].f = nil } } @@ -760,6 +761,7 @@ // 32-bit system; clear uintptr tests hammer64[2].f = nil hammer64[5].f = nil + hammer64[6].f = nil } }
