Re: svn commit: r228495 - head/sys/sys

2011-12-15 Thread Ed Schouten
Hello Bruce, After reading through your email, I think it's best to fix things using the following patch. Essentially it does the following: - It makes sure __alignof() is always present, by defining it as a macro for GCC 2.95. - All the C1X macros can now be implemented using the ones we

Re: svn commit: r228495 - head/sys/sys

2011-12-15 Thread Bruce Evans
On Thu, 15 Dec 2011, Ed Schouten wrote: Hello Bruce, After reading through your email, I think it's best to fix things using the following patch. Essentially it does the following: - It makes sure __alignof() is always present, by defining it as a macro for GCC 2.95. I tested this with a

Re: svn commit: r228495 - head/sys/sys

2011-12-15 Thread Ed Schouten
Hello Bruce, * Bruce Evans b...@optusnet.com.au, 20111215 15:12: The 2 bugs are: 1. Even __offsetof() is broken in the fallback case (that's all cases where it uses a struct like the above and doesn't use a gccish builtin). This is because __offsetof() uses size_t, but size_t is not

Re: svn commit: r228495 - head/sys/sys

2011-12-15 Thread David Schultz
On Thu, Dec 15, 2011, Ed Schouten wrote: Hello Bruce, * Bruce Evans b...@optusnet.com.au, 20111215 15:12: - recently broken for KR since it now uses signed instead of __signed - broken for longer for KR and C90 since it uses long long. In working versions, the __int64_t

svn commit: r228495 - head/sys/sys

2011-12-14 Thread Ed Schouten
Author: ed Date: Wed Dec 14 09:09:37 2011 New Revision: 228495 URL: http://svn.freebsd.org/changeset/base/228495 Log: Slightly alter the C1X definitions in in cdefs.h: - Add _Alignas(). Unfortunately this macro is only partially functional. The C1X standard will allow both an integer

Re: svn commit: r228495 - head/sys/sys

2011-12-14 Thread Bruce Evans
On Wed, 14 Dec 2011, Ed Schouten wrote: Log: Slightly alter the C1X definitions in in cdefs.h: - Add _Alignas(). Unfortunately this macro is only partially functional. The C1X standard will allow both an integer and a type name to be passed to this macro, while this macro only allows