Re: [PATCH] Re: [v3] updated atomic configury

2011-11-30 Thread Benjamin Kosnik
Thanks Andrew. Here's the updated patch, as checked in. This removes one level of cruft WRT atomic configure-ness (leaving us with the most primordial...). tested x86/linux tested x86/linux x arm-eabi tested x86/linux x cris-elf -benjamin 2011-11-30 Benjamin Kosnik * acinclude.m4 (

Re: [PATCH] Re: [v3] updated atomic configury

2011-11-28 Thread Richard Henderson
On 11/28/2011 11:29 AM, Andrew MacLeod wrote: > c-family > * c-cpp-builtin.c (cpp_atomic_builtins):New. Emit all atomic > predefines in one place. Add LOCK_FREE predefines. > (c_cpp_builtins): Move Legacy HAVE_SYNC predefines to > new func. > > libstdc++-v3 >

Re: [PATCH] Re: [v3] updated atomic configury

2011-11-28 Thread Andrew MacLeod
On 11/28/2011 11:42 AM, Joseph S. Myers wrote: On Mon, 28 Nov 2011, Andrew MacLeod wrote: That would be why cppbuiltin.c, defining __SIZEOF_POINTER__, needs to use the POINTER_SIZE macro instead of directly using ptr_type_node. You may need to do something similar. /* ptr_type_node can't be

Re: [PATCH] Re: [v3] updated atomic configury

2011-11-28 Thread Joseph S. Myers
On Mon, 28 Nov 2011, Andrew MacLeod wrote: > OK, here's a reworked patch. This bootstraps but there is a regression... one > set of PCH tests fail in a very odd way. I've investigated it and it turns out > that for some reason, ptr_type_node doesn't always have it size set if code > isn't being

[PATCH] Re: [v3] updated atomic configury

2011-11-28 Thread Andrew MacLeod
On 11/22/2011 03:48 PM, Joseph S. Myers wrote: On Tue, 22 Nov 2011, Andrew MacLeod wrote: It certainly does have those types, as typedefs defined in for the same types as uint_least16_t and uint_least32_t. And is also required to define ATOMIC_CHAR16_T_LOCK_FREE and ATOMIC_CHAR32_T_LOCK_F