Re: __sync_swap* [ rename sync builtins ]

[Graham Stott <graham dot stott at btinternet dot com>]

All

--- On Tue, 21/6/11, Richard Henderson <rth@redhat.com> wrote:

> From: Richard Henderson <rth@redhat.com>
> Subject: Re: __sync_swap*  [ rename sync builtins ]
> To: "Andrew MacLeod" <amacleod@redhat.com>
> Cc: "Aldy Hernandez" <aldyh@redhat.com>, "Jakub Jelinek" <jakub@redhat.com>, "Joseph S. Myers" <joseph@codesourcery.com>, "gcc-patches" <gcc-patches@gcc.gnu.org>
> Date: Tuesday, 21 June, 2011, 19:50
> 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 names.
> > ÂÂÂ * builtins.c: Add sync_ or SYNC__
> to builtin names.
> > ÂÂÂ * sync-builtins.def: Add sync_ or
> SYNC__ to builtin names.
> > ÂÂÂ * omp-low.c: Add sync_ or SYNC__ to
> builtin names.
> > ÂÂÂ * cp/semantics.c: Add sync_ or
> SYNC__ to builtin names.
> > ÂÂÂ * fortran/trans-openmp.c: Add sync_
> or SYNC__ to builtin names.
> > ÂÂÂ * fortran/trans-stmt.c: Add sync_
> or SYNC__ to builtin names.
> > ÂÂÂ * fortran/trans-decl.c: Add sync_
> or SYNC__ to builtin names.
> 
> Ok.
> 
> 
> r~
> 

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.cc:94:18: error: âBUILT_IN_ADD_AND_FETCH_1â was not declared in this scope
/usr/local/src/gcc4.7/gcc/gcc/go/gofrontend/gogo-tree.cc:99:19: error: âBUILT_IN_ADD_AND_FETCH_2â was not declared in this scope
/usr/local/src/gcc4.7/gcc/gcc/go/gofrontend/gogo-tree.cc:104:18: error: âBUILT_IN_ADD_AND_FETCH_4â was not declared in this scope
/usr/local/src/gcc4.7/gcc/gcc/go/gofrontend/gogo-tree.cc:109:18: error: âBUILT_IN_ADD_AND_FETCH_8â was not declared in this scope

Graham