2016-02-04 03:35, Wang, Zhihong:
> > Subject: [PATCH 1/3] eal/x86: fix build with clang for old AVX
> >
> > 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")
>
> Hi Thomas,
>
> Thanks for pointing this out!
> My last hasty modification on this is not correct.
>
> The patch below will fix it. All modifications are tested.
> Sorry for all the hassle! :'(
>
> "srcofs" should be calculated based on source address anyway.
OK
Please send a full patch as usual, thanks.
