Re: [dpdk-dev] [PATCH v2 2/2] i40e/base: compile fix on clang 3.3

[Bruce Richardson <bruce.richardson@intel.com>]

On Mon, May 18, 2015 at 11:03:29PM +0800, Helin Zhang wrote:
> Below compile error can be found on clang 3.3, which is a warning
> treated as error. Forcedly disabling the warning can fix it.
> 
> Error log:
> lib/librte_pmd_i40e/i40e/i40e_nvm.c:708:20: error: unused variable
> 'i40e_nvm_update_state_str' [-Werror,-Wunused-variable]
> STATIC const char *i40e_nvm_update_state_str[] = {
>                    ^
> 
> Signed-off-by: Helin Zhang <helin.zhang@intel.com>

Acked-by: Bruce Richardson <bruce.richardson@intel.com>

> ---
>  lib/librte_pmd_i40e/Makefile | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/lib/librte_pmd_i40e/Makefile b/lib/librte_pmd_i40e/Makefile
> index 911e4f5..4a5635b 100644
> --- a/lib/librte_pmd_i40e/Makefile
> +++ b/lib/librte_pmd_i40e/Makefile
> @@ -58,6 +58,7 @@ CFLAGS_BASE_DRIVER += -Wno-format
>  CFLAGS_BASE_DRIVER += -Wno-missing-field-initializers
>  CFLAGS_BASE_DRIVER += -Wno-pointer-to-int-cast
>  CFLAGS_BASE_DRIVER += -Wno-format-nonliteral
> +CFLAGS_BASE_DRIVER += -Wno-unused-variable
>  else
>  CFLAGS_BASE_DRIVER  = -Wno-sign-compare
>  CFLAGS_BASE_DRIVER += -Wno-unused-value
> -- 
> 1.8.1.4
>