[PATCH v2] Add pragma to ignore gcc-compat warnings in clang when used with diagnose_if.
Stephen Hemminger
stephen at networkplumber.org
Mon Jan 24 00:53:40 CET 2022
On Mon, 24 Jan 2022 10:17:37 +1300
Michael Barker <mikeb01 at gmail.com> wrote:
> On Fri, 21 Jan 2022 at 03:16, Thomas Monjalon <thomas at monjalon.net> wrote:
>
> > 18/01/2022 00:23, Michael Barker:
> > > When using clang with -Wall the use of diagnose_if kicks up a warning,
> >
> > Please could you copy the warning in the commit log?
> >
>
> I've updated the commit log to be more descriptive (and included the
> associated warning).
>
> > requiring all dpdk includes to be wrapped with the pragma. This change
> > > isolates the ignore just the appropriate location and makes it easier
> > > for users to apply -Wall,-Werror
> >
> > Please could you explain how it is related to -Wgcc-compat?
> >
>
> I'm currently working on some code that makes use of DPDK, which is built
> with '-Wall,-Werror' enabled. When using the clang toolchain the build
> fails as a result of this macro that this patch updates. The workaround
> from my application is to wrap all of the DPDK header includes in pragma to
> disable the warnings (see below). This has the unfortunate side effect of
> disabling this warning across all of the included DPDK headers, which is
> not ideal. Hence the reason to submit the patch which disables the warning
> just in the location where it occurs.
>
Fix the issue please, don't suppress it.
More information about the dev
mailing list