[dpdk-dev] [PATCH 1/3] eal/x86: fix build with clang for old AVX
Thomas Monjalon
thomas.monjalon at 6wind.com
Wed Feb 3 19:56:37 CET 2016
When configuring RTE_MACHINE to "default", rte_memcpy implementation
is the default one (old AVX).
In this code, clang raises a warning thanks to -Wsometimes-uninitialized:
rte_memcpy.h:838:6: error:
variable 'srcofs' is used uninitialized whenever 'if' condition is false
if (dstofss > 0) {
^~~~~~~~~~~
rte_memcpy.h:849:6: note: uninitialized use occurs here
if (srcofs == 0) {
^~~~~~
It is fixed by initializing srcofs to 0.
Fixes: 1ae817f9f887 ("eal/x86: tune memcpy for platforms without AVX512")
Signed-off-by: Thomas Monjalon <thomas.monjalon at 6wind.com>
---
lib/librte_eal/common/include/arch/x86/rte_memcpy.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/librte_eal/common/include/arch/x86/rte_memcpy.h b/lib/librte_eal/common/include/arch/x86/rte_memcpy.h
index 8e2c53c..5badfbc 100644
--- a/lib/librte_eal/common/include/arch/x86/rte_memcpy.h
+++ b/lib/librte_eal/common/include/arch/x86/rte_memcpy.h
@@ -739,7 +739,7 @@ rte_memcpy(void *dst, const void *src, size_t n)
uintptr_t srcu = (uintptr_t)src;
void *ret = dst;
size_t dstofss;
- size_t srcofs;
+ size_t srcofs = 0;
/**
* Copy less than 16 bytes
--
2.7.0
More information about the dev
mailing list