diff options
| author | Ian Lance Taylor <ian@gcc.gnu.org> | 2017-01-23 18:15:22 +0000 |
|---|---|---|
| committer | Ian Lance Taylor <ian@gcc.gnu.org> | 2017-01-23 18:15:22 +0000 |
| commit | b0147ac2e62c3ed744d55fc5d009c4ef7a8ef651 (patch) | |
| tree | 807c3b6f6779fa8ab24d85323f1593d91af510de /libgo/runtime/runtime.h | |
| parent | a0488df43613f596dffc3f18118a0e3216f2f842 (diff) | |
| download | gcc-b0147ac2e62c3ed744d55fc5d009c4ef7a8ef651.zip gcc-b0147ac2e62c3ed744d55fc5d009c4ef7a8ef651.tar.gz gcc-b0147ac2e62c3ed744d55fc5d009c4ef7a8ef651.tar.bz2 | |
re PR go/79037 (gccgo: Binaries crash with parforsetup: pos is not aligned on m68k)
PR go/79037
compiler, runtime: align gc data for m68k
The current GC requires that the gc data be aligned to at least a 4
byte boundary, because it uses the lower two bits of the address for
flags (see LOOP and PRECISE in runtime/mgc0.c). As the gc data is
stored as a [...]uintptr, that is normally always true. However, on
m68k, that only guarantees 2 byte alignment. Fix it by forcing the
alignment.
The parfor code used by the current GC requires that the parfor data
be aligned to at least an 8 byte boundary. The code in parfor.c
verifies this. This is normally true, as the data uses uint64_t
values, but, again, this must be enforced explicitly on m68k.
Fixes GCC PR 79037.
Reviewed-on: https://go-review.googlesource.com/35478
From-SVN: r244824
Diffstat (limited to 'libgo/runtime/runtime.h')
| -rw-r--r-- | libgo/runtime/runtime.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libgo/runtime/runtime.h b/libgo/runtime/runtime.h index 5fd1155..644fe92 100644 --- a/libgo/runtime/runtime.h +++ b/libgo/runtime/runtime.h @@ -191,7 +191,7 @@ struct ParFor // otherwise parfor may return while other threads are still working ParForThread *thr; // array of thread descriptors // stats - uint64 nsteal; + uint64 nsteal __attribute__((aligned(8))); // force alignment for m68k uint64 nstealcnt; uint64 nprocyield; uint64 nosyield; |