mbox series

[00/11] Fixes for clang 15

Message ID 20221118085313.2118977-1-david.marchand@redhat.com (mailing list archive)
Headers
Series Fixes for clang 15 |

Message

David Marchand Nov. 18, 2022, 8:53 a.m. UTC
  Fedora 37 has just been released with clang 15.
The latter seems more picky wrt unused variable.

Fixes have been tested in GHA with a simple patch I used in my own repo:
https://github.com/david-marchand/dpdk/commit/82cd57ae5490
https://github.com/david-marchand/dpdk/actions/runs/3495454457
  

Comments

David Marchand Nov. 21, 2022, 8:12 a.m. UTC | #1
On Sat, Nov 19, 2022 at 1:13 AM Tyler Retzlaff
<roretzla@linux.microsoft.com> wrote:
>
> On Fri, Nov 18, 2022 at 09:53:02AM +0100, David Marchand wrote:
> > Fedora 37 has just been released with clang 15.
> > The latter seems more picky wrt unused variable.
> >
> > Fixes have been tested in GHA with a simple patch I used in my own repo:
> > https://github.com/david-marchand/dpdk/commit/82cd57ae5490
> > https://github.com/david-marchand/dpdk/actions/runs/3495454457
> >
> > --
>
> Series-Acked-By: Tyler Retzlaff <roretzla@linux.microsoft.com>

Thanks for the ack.
Re-adding dev@.
  
David Marchand Nov. 21, 2022, 10:23 a.m. UTC | #2
On Fri, Nov 18, 2022 at 9:55 AM David Marchand
<david.marchand@redhat.com> wrote:
>
> Fedora 37 has just been released with clang 15.
> The latter seems more picky wrt unused variable.
>
> Fixes have been tested in GHA with a simple patch I used in my own repo:
> https://github.com/david-marchand/dpdk/commit/82cd57ae5490
> https://github.com/david-marchand/dpdk/actions/runs/3495454457
>
> --
> David Marchand
>
> David Marchand (11):
>   service: fix build with clang 15
>   vhost: fix build with clang 15
>   bus/dpaa: fix build with clang 15
>   net/atlantic: fix build with clang 15
>   net/dpaa2: fix build with clang 15
>   net/ice: fix build with clang 15
>   app/testpmd: fix build with clang 15
>   app/testpmd: fix build with clang 15 in flow code
>   test/efd: fix build with clang 15
>   test/member: fix build with clang 15
>   test/event: fix build with clang 15
>
>  app/test-pmd/config.c               | 14 --------------
>  app/test-pmd/noisy_vnf.c            |  2 +-
>  app/test/test_efd_perf.c            |  1 -
>  app/test/test_event_timer_adapter.c |  2 --
>  app/test/test_member.c              |  1 -
>  app/test/test_member_perf.c         |  1 -
>  drivers/bus/dpaa/base/qbman/bman.h  |  4 +---
>  drivers/net/atlantic/atl_rxtx.c     |  5 ++---
>  drivers/net/dpaa2/dpaa2_rxtx.c      |  4 +---
>  drivers/net/ice/ice_ddp_package.c   |  3 ---
>  lib/eal/common/rte_service.c        |  2 --
>  lib/vhost/virtio_net.c              |  2 --
>  12 files changed, 5 insertions(+), 36 deletions(-)

Series applied for rc4, thanks.