On Tue, Oct 17, 2023 at 1:34 PM Tyler Retzlaff
<roret...@linux.microsoft.com> wrote:
>
> On Tue, Oct 17, 2023 at 01:31:25PM +0200, David Marchand wrote:
> > rte_memory_order_* should be used when calling the new stdatomic API.
> > Add a check on __ATOMIC_* tokens.
> >
> > Signed-off-by: David Marchand <david.march...@redhat.com>
> Acked-by: Tyler Retzlaff <roret...@linux.microsoft.com>
>

[snip]

> >               -v RET_ON_FAIL=1 \
> > -             -v MESSAGE='Using __atomic_xxx built-ins, prefer 
> > rte_atomic_xxx' \
> > +             -v MESSAGE='Using __atomic_xxx/__ATOMIC_XXX built-ins, prefer 
> > rte_atomic_xxx' \
>
> Not sure if it should say prefer rte_atomic_xxx/rte_memory_order_xxx ?

Looks ok to me, I did this small change while applying.

Applied, thanks.

-- 
David Marchand

Reply via email to