mbox series

[0/2] fix enable_stdatomic=true build with clang

Message ID 1697482390-32106-1-git-send-email-roretzla@linux.microsoft.com (mailing list archive)
Headers
Series fix enable_stdatomic=true build with clang |

Message

Tyler Retzlaff Oct. 16, 2023, 6:53 p.m. UTC 
  We are temporarily exposed while the CI is enabled that tests
CC=clang && -Denable_stdatomic.

In the meantime two mistakes have been found so fix them to allow
the CI to be enabled.

Tyler Retzlaff (2):
  power: fix use of rte stdatomic
  event/cnxk: remove single use of rte stdatomic

 drivers/event/cnxk/cnxk_tim_worker.c |  2 +-
 lib/power/power_amd_pstate_cpufreq.c | 14 +++++++-------
 2 files changed, 8 insertions(+), 8 deletions(-)

Comments

David Marchand Oct. 16, 2023, 7:07 p.m. UTC | #1 
On Mon, Oct 16, 2023 at 8:53 PM Tyler Retzlaff
<roretzla@linux.microsoft.com> wrote:
>
> We are temporarily exposed while the CI is enabled that tests
> CC=clang && -Denable_stdatomic.
>
> In the meantime two mistakes have been found so fix them to allow
> the CI to be enabled.
>
> Tyler Retzlaff (2):
>   power: fix use of rte stdatomic
>   event/cnxk: remove single use of rte stdatomic

Thank you for the fixes.
Reviewed-by: David Marchand <david.marchand@redhat.com>
David Marchand Oct. 17, 2023, 6:50 a.m. UTC | #2 
On Mon, Oct 16, 2023 at 9:07 PM David Marchand
<david.marchand@redhat.com> wrote:
>
> On Mon, Oct 16, 2023 at 8:53 PM Tyler Retzlaff
> <roretzla@linux.microsoft.com> wrote:
> >
> > We are temporarily exposed while the CI is enabled that tests
> > CC=clang && -Denable_stdatomic.
> >
> > In the meantime two mistakes have been found so fix them to allow
> > the CI to be enabled.
> >
> > Tyler Retzlaff (2):
> >   power: fix use of rte stdatomic
> >   event/cnxk: remove single use of rte stdatomic
>
> Thank you for the fixes.
> Reviewed-by: David Marchand <david.marchand@redhat.com>

Series applied, thanks.