Re: [PATCH v2 2/2] fix building with clang-3.8.0 compiler
From: Bruce Richardson <hidden>
Date: 2016-06-27 08:46:53
From: Bruce Richardson <hidden>
Date: 2016-06-27 08:46:53
On Sun, Jun 26, 2016 at 10:54:12AM -0500, Keith Wiles wrote:
Latest clang compiler 3.8.0 on latest update of Ubuntu creates a few more warnings on -Warray-bounds and extra () around 'if' expressions. Signed-off-by: Keith Wiles <redacted> --- app/test-pmd/Makefile | 3 +++ app/test/Makefile | 3 +++ drivers/net/bonding/Makefile | 4 ++++ drivers/net/fm10k/Makefile | 2 ++ drivers/net/i40e/Makefile | 2 ++ lib/librte_cmdline/Makefile | 6 ++++++ lib/librte_eal/linuxapp/eal/Makefile | 8 ++++++++ 7 files changed, 28 insertions(+)
All the fixes in this patch seem to be just disabling the compiler warnings, which should really be the last resort in cases like this. Can some of the issues be fixed by actually fixing the issues in the code? /Bruce