[PATCH, v3] ARM: fix inline atomic handling

Phoebe Buckheister llvm at quasiparticle.net
Tue Jul 8 11:21:59 PDT 2014


On Tue, 8 Jul 2014 19:10:39 +0100, Tim Northover wrote:
> Hi Phoebe,
> 
> On 8 July 2014 18:58, Phoebe Buckheister <llvm at quasiparticle.net>
> wrote:
> > Another attempt at fixing atomic handling. This version includes
> > tests that atomics are used on the v7M profile only for widths up
> > to 32 bits, and not at all on the v6M profile.
> 
> Thanks for persisting with this! I think this looks OK now, and from
> what I remember there was a reasonable consensus in the original
> thread that opt-out was the right position. So I say go ahead!
> 
> I've not noticed you committing before, do you have access or would
> you like me to do it for you?

I don't have access, so it'd be much appreciated if you committed. :)

> Cheers.
> 
> Tim.
> 
> (P.S. Very minor point, but we usually try to avoid creating a new
> thread with each update. If you fix anything else (please do!),
> replying to the previous discussion keeps everything together).

Oh. Thanks, I'll keep that in mind for future patches - Linux handles
this a bit differently ;)



More information about the cfe-commits mailing list