On Sun, Jul 09, 2017 at 10:45:55 -1000, Richard Henderson wrote:
> On 07/08/2017 09:50 PM, Emilio G. Cota wrote:
> >+ /* includes aborted translations because of exceptions */
> >+ atomic_set(&prof->tb_count1, prof->tb_count1 + 1);
>
> Again, atomic_set without atomic_read is pointless.
> Either you're trying to give the compiler extra information, or you aren't.
See my comment to patch 3.
> As always, it won't ever matter in practice because aligned native types
> never tear. This is all about markup for compiler tools.
I do it mostly to avoid undefined behaviour under C11. Pleasing
(some) tools is a nice side effect though.
E.
