Re: [PATCH 0/5] Convert Sparc to atomic optabs

2011-11-27 Thread Eric Botcazou
> Try top-of-branch > > git://repo.or.cz/gcc/rth.git rth/atomic/sparc > > That's the last iteration I went through with Dave. Thanks, this works on Solaris, both 32-bit and 64-bit. So, modulo the cosmetic changes I suggested, the documentation about the new -m flag and the selection of the de

Re: [PATCH 0/5] Convert Sparc to atomic optabs

2011-11-26 Thread Richard Henderson
On 11/26/2011 01:58 PM, Eric Botcazou wrote: >> The first four patches simply do the conversion, a piece at a time, >> assuming the RMO for all cpus. > > The new form of the membar_v8 insn looks a bit strange since operand 1 is > disregarded. If this is as intended, a comment would be in order.

Re: [PATCH 0/5] Convert Sparc to atomic optabs

2011-11-26 Thread Eric Botcazou
> The first four patches simply do the conversion, a piece at a time, > assuming the RMO for all cpus. The new form of the membar_v8 insn looks a bit strange since operand 1 is disregarded. If this is as intended, a comment would be in order. > The fifth patch adds the ability to explicitly set

Re: [PATCH 0/5] Convert Sparc to atomic optabs

2011-11-17 Thread Eric Botcazou
> Dave and Eric, can you please double-check everything and also test > on Solaris (i.e. RMO). Yes, I will, but note that Solaris is TSO. The only RMO case was Linux AFAIK. -- Eric Botcazou