Re: __sync_swap* [ rename sync builtins ]

2011-06-23 Thread Andrew MacLeod
On 06/23/2011 08:14 PM, Ian Lance Taylor wrote: On Tue, Jun 21, 2011 at 4:03 PM, Andrew MacLeod wrote: On 06/21/2011 06:26 PM, Graham Stott wrote: This looks to have broken the go frontend ah, missed it's .cc file, and I guess it doesn't build by default :-P This ought to fix it, checking i

Re: __sync_swap* [ rename sync builtins ]

2011-06-23 Thread Ian Lance Taylor
On Tue, Jun 21, 2011 at 4:03 PM, Andrew MacLeod wrote: > On 06/21/2011 06:26 PM, Graham Stott wrote: >> This looks to have broken the go frontend > > ah, missed it's .cc file, and I guess it doesn't build by default  :-P > > This ought to fix it, checking in as obvious... Note that the files in g

Re: __sync_swap* [ rename sync builtins ]

2011-06-21 Thread Andrew MacLeod
On 06/21/2011 07:03 PM, Andrew MacLeod wrote: On 06/21/2011 06:26 PM, Graham Stott wrote: All --- On Tue, 21/6/11, Richard Henderson wrote: This looks to have broken the go frontend Graham ah, missed it's .cc file, and I guess it doesn't build by default :-P This ought to fix it, checkin

Re: __sync_swap* [ rename sync builtins ]

2011-06-21 Thread Andrew MacLeod
On 06/21/2011 06:26 PM, Graham Stott wrote: All --- On Tue, 21/6/11, Richard Henderson wrote: This looks to have broken the go frontend gcc/gcc/go/gofrontend/gogo-tree.cc: In member function ‘void Gogo::define_builtin_function_trees()’: /usr/local/src/gcc4.7/gcc/gcc/go/gofrontend/gogo-tree.c

Re: __sync_swap* [ rename sync builtins ]

2011-06-21 Thread Graham Stott
All --- On Tue, 21/6/11, Richard Henderson wrote: > From: Richard Henderson > Subject: Re: __sync_swap* [ rename sync builtins ] > To: "Andrew MacLeod" > Cc: "Aldy Hernandez" , "Jakub Jelinek" , > "Joseph S. Myers" , "gcc-patches&q

Re: __sync_swap* [ rename sync builtins ]

2011-06-21 Thread Richard Henderson
On 06/21/2011 11:46 AM, Andrew MacLeod wrote: > * c-family/c-common.c: Add sync_ or SYNC__ to builtin names. > * c-family/c-omp.c: Add sync_ or SYNC__ to builtin names. > * java/builtins.c: Add sync_ or SYNC__ to builtin names. > * java/expr.c: Add sync_ or SYNC__ to builtin