[Buildroot] [PATCH v3,1/1] package/liquid-dsp: fix x86 builds

Peter Korsgaard peter at korsgaard.com
Sat Jan 13 13:27:02 UTC 2024


>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice at gmail.com> writes:

 > HAVE_{MMX,SSE2,...} are not defined if ax_cv_have_{i}_cpu_ext is not set
 > resulting in the following build failure raised since bump to version
 > 1.5.0 in commit c2aaa0fbe2fdf6e599c68169aa7b2e55237190c7 and
 > https://github.com/jgaeddert/liquid-dsp/commit/02c4e8b99b11d9f1615b8840f243b08a85b86563:

 > src/dotprod/src/dotprod_cccf.sse.c: In function 'dotprod_cccf_execute_sse':
 > src/dotprod/src/dotprod_cccf.sse.c:258:5: error: unknown type name '__m128'; did you mean '__int128'?
 >   258 |     __m128 v;   // input vector
 >       |     ^~~~~~
 >       |     __int128

 > or

 > src/dotprod/src/dotprod_cccf.mmx.c: In function 'dotprod_cccf_execute_mmx':
 > src/dotprod/src/dotprod_cccf.mmx.c:262:5: error: unknown type name '__m128'; did you mean '__int128'?
 >   262 |     __m128 v;   // input vector
 >       |     ^~~~~~
 >       |     __int128

 > While at it, add AVX2 support

 > Fixes:
 >  - http://autobuild.buildroot.org/results/738ce9d3dc74ec165391f21256c955e5524f1632
 >  - http://autobuild.buildroot.org/results/a2d150c724ab6787aeabaf31f65116f802e8584e

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice at gmail.com>
 > ---
 > Changes v2 -> v3 (after review of Yann E. Morin):
 >  - Keep current list format

 > Changes v1 -> v2:
 >  - Keep old options (mmx, etc.) as they seem to be still used somewhere
 >    in the code

Committed to 2023.11.x, thanks.

-- 
Bye, Peter Korsgaard



More information about the buildroot mailing list