Re: [PATCH] test/input: Replace negative architecture test by test for 64-bit

2013-02-12 Thread Michał Masłowski
Geert Uytterhoeven ge...@linux-m68k.org writes: The test for double-aligned members in dix_valuator_alloc() currently depends on if !defined(__i386__) !defined(__sh__). This covers m68k, where it fails. According to the comment, the test should be limited to 64-bit platforms only. Hence

Re: [PATCH] test/input: Replace negative architecture test by test for 64-bit

2013-02-12 Thread Mark Kettenis
From: Geert Uytterhoeven ge...@linux-m68k.org Date: Tue, 12 Feb 2013 11:17:22 +0100 The test for double-aligned members in dix_valuator_alloc() currently depends on if !defined(__i386__) !defined(__sh__). This covers m68k, where it fails. According to the comment, the test should be

Re: [PATCH] test/input: Replace negative architecture test by test for 64-bit

2013-02-12 Thread walter harms
Am 12.02.2013 13:40, schrieb Mark Kettenis: From: Geert Uytterhoeven ge...@linux-m68k.org Date: Tue, 12 Feb 2013 11:17:22 +0100 The test for double-aligned members in dix_valuator_alloc() currently depends on if !defined(__i386__) !defined(__sh__). This covers m68k, where it fails.

Re: [PATCH] test/input: Replace negative architecture test by test for 64-bit

2013-02-12 Thread Geert Uytterhoeven
On Tue, Feb 12, 2013 at 1:40 PM, Mark Kettenis mark.kette...@xs4all.nl wrote: From: Geert Uytterhoeven ge...@linux-m68k.org Date: Tue, 12 Feb 2013 11:17:22 +0100 The test for double-aligned members in dix_valuator_alloc() currently depends on if !defined(__i386__) !defined(__sh__). This

Re: [PATCH] test/input: Replace negative architecture test by test for 64-bit

2013-02-12 Thread Mark Kettenis
Sender: geert.uytterhoe...@gmail.com Date: Tue, 12 Feb 2013 14:03:50 +0100 On Tue, Feb 12, 2013 at 1:40 PM, Mark Kettenis mark.kette...@xs4all.nl wrote: From: Geert Uytterhoeven ge...@linux-m68k.org Date: Tue, 12 Feb 2013 11:17:22 +0100 The test for double-aligned members in

Re: [PATCH] test/input: Replace negative architecture test by test for 64-bit

2013-02-12 Thread Thorsten Glaser
Michał Masłowski dixit: MIPS N32 needs doubles to be 8 byte aligned, while it has 32 bit longs Hrm. Well, GCC has __alignof__ and, I think, so has C11… bye, //mirabilos -- mirabilos│ untested Natureshadow │ tut natürlich Natureshadow │ was auch sonst ... mirabilos│ fijn ☺