On Mon, 24 Jan 2022 10:17:37 +1300 Michael Barker <mike...@gmail.com> wrote:
> On Fri, 21 Jan 2022 at 03:16, Thomas Monjalon <tho...@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.