On Fri, Sep 23, 2016 at 10:31:12AM +0200, Thomas Monjalon wrote:
> 2016-09-23 14:15, Yuanhan Liu:
> > - _mm_prefetch((const void *)rused, _MM_HINT_T0);
> > + rte_prefetch0(rused);
>
> There are other occurences of _mm_prefetch in other drivers.
> It could deserve a patch.
Indeed. I could make
On Fri, Sep 23, 2016 at 12:05:14PM +0530, Jerin Jacob wrote:
> On Fri, Sep 23, 2016 at 02:15:09PM +0800, Yuanhan Liu wrote:
>
> Hi Yuanhan,
>
> Thanks for this patch.
>
> > Interestingly, clang and gcc has different prototype for _mm_prefetch().
> > For gcc, we have
> >
> >_mm_prefetch (con
Interestingly, clang and gcc has different prototype for _mm_prefetch().
For gcc, we have
_mm_prefetch (const void *__P, enum _mm_hint __I)
While for clang, it's
#define _mm_prefetch(a, sel) (__builtin_prefetch((void *)(a), 0, (sel)))
That how the following error comes with clang:
err
On Fri, Sep 23, 2016 at 02:15:09PM +0800, Yuanhan Liu wrote:
Hi Yuanhan,
Thanks for this patch.
> Interestingly, clang and gcc has different prototype for _mm_prefetch().
> For gcc, we have
>
>_mm_prefetch (const void *__P, enum _mm_hint __I)
>
> While for clang, it's
>
>#define _mm_p
2016-09-23 14:15, Yuanhan Liu:
> - _mm_prefetch((const void *)rused, _MM_HINT_T0);
> + rte_prefetch0(rused);
There are other occurences of _mm_prefetch in other drivers.
It could deserve a patch.
5 matches
Mail list logo