[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