Re: [lttng-dev] [PATCH 02/11] urcu/uatomic: Use atomic builtins if configured

Mathieu Desnoyers via lttng-dev <[email protected]> Thu, 29 Jun 2023 14:33:45 -0400

Newsgroups	org.lttng.lists.lttng-dev
Message-ID	<[email protected]>

On 6/29/23 13:27, Olivier Dion wrote:
> On Thu, 29 Jun 2023, Olivier Dion <[email protected]> wrote:
> 
>>    [0] https://godbolt.org/z/3nW14M3v1
>>    [1] https://godbolt.org/z/TcTeMeKbW
> 
> Sorry.  That was:
> 
>      [0] https://godbolt.org/z/ETcxnz4TW

Change

(volatile __typeof__(ptr))(ptr);

for:

(volatile __typeof__(*(ptr)) *)(ptr);

and:

void love_iso(int *x)
{
      __atomic_store_n(cast_volatile(&x), 1,
                       __ATOMIC_RELAXED);
}

for

void love_iso(int *x)
{
      __atomic_store_n(cast_volatile(x), 1,
                       __ATOMIC_RELAXED);
}

Thanks,

Mathieu


>      [1] https://godbolt.org/z/jMjh8YoM4
> -- 
Mathieu Desnoyers
EfficiOS Inc.
https://www.efficios.com

_______________________________________________
lttng-dev mailing list
[email protected]
https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev