Re: [PATCH 2/2] eal: Fix compilation on C++
From: Thomas Monjalon <hidden>
Date: 2015-08-17 15:51:30
2015-08-17 17:29, Thomas Monjalon:
2015-07-03 21:51, Joongi Kim:quoted
* Forward declaration of enum in C++ requires explicit underlying type definitions. * This fixes the issue at: http://dpdk.org/ml/archives/dev/2015-April/017065.html Signed-off-by: Joongi Kim <redacted>Does the problem appear only with rte_hash_crc.h inclusion? The reported errors were: include/generic/rte_cpuflags.h:50:6: error: use of enum ‘rte_cpu_flag_t’ without previous declaration enum rte_cpu_flag_t; include/generic/rte_cpuflags.h:55:6: error: use of enum ‘cpu_register_t’ without previous declaration enum cpu_register_t;quoted
lib/librte_eal/common/include/arch/x86/rte_cpuflags.h | 4 ++-- lib/librte_eal/common/include/generic/rte_cpuflags.h | 12 ++++++++++--Do we need to update lib/librte_eal/common/include/arch/ppc_64/rte_cpuflags.h and lib/librte_eal/common/include/arch/tile/rte_cpuflags.h as well? Thanks
Applied with same fixes for ppc and tile, thanks