| Message ID | 20220607202828.1130778-1-fontaine.fabrice@gmail.com |
|---|---|
| State | Accepted |
| Headers | show |
| Series | [1/1] package/jpeg: SIMD needs VSX with little endian | expand |
On 07/06/2022 22:28, Fabrice Fontaine wrote: > SIMD needs VSX with little endian to avoid the following build failure: > > In file included from /nvmedata/autobuild/instance-12/output-1/build/jpeg-turbo-2.1.3/simd/powerpc/jccolor-altivec.c:25: > /nvmedata/autobuild/instance-12/output-1/build/jpeg-turbo-2.1.3/simd/powerpc/jccolext-altivec.c: In function 'jsimd_rgb_ycc_convert_altivec': > /nvmedata/autobuild/instance-12/output-1/build/jpeg-turbo-2.1.3/simd/powerpc/jsimd_altivec.h:93:26: warning: implicit declaration of function 'vec_vsx_ld'; did you mean 'vec_vsl'? [-Wimplicit-function-declaration] > 93 | #define VEC_LD(a, b) vec_vsx_ld(a, b) > | ^~~~~~~~~~ > > Fixes: > - http://autobuild.buildroot.org/results/be6d5ad0cee4ee19eb25e595d44555a1af6e073b > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> Applied to master, thanks. Regards, Arnout > --- > package/jpeg/Config.in | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/package/jpeg/Config.in b/package/jpeg/Config.in > index 0a786060d7..371f89aa49 100644 > --- a/package/jpeg/Config.in > +++ b/package/jpeg/Config.in > @@ -3,7 +3,8 @@ config BR2_PACKAGE_JPEG_SIMD_SUPPORT > bool > default y if BR2_X86_CPU_HAS_MMX > default y if BR2_ARM_CPU_HAS_NEON > - default y if BR2_POWERPC_CPU_HAS_ALTIVEC > + default y if BR2_POWERPC_CPU_HAS_ALTIVEC && !BR2_powerpc64le > + default y if BR2_POWERPC_CPU_HAS_VSX && BR2_powerpc64le > default y if BR2_aarch64 || BR2_aarch64_be > > config BR2_PACKAGE_JPEG
>>>>> "Arnout" == Arnout Vandecappelle <arnout@mind.be> writes: > On 07/06/2022 22:28, Fabrice Fontaine wrote: >> SIMD needs VSX with little endian to avoid the following build failure: >> In file included from >> /nvmedata/autobuild/instance-12/output-1/build/jpeg-turbo-2.1.3/simd/powerpc/jccolor-altivec.c:25: >> /nvmedata/autobuild/instance-12/output-1/build/jpeg-turbo-2.1.3/simd/powerpc/jccolext-altivec.c: In function 'jsimd_rgb_ycc_convert_altivec': >> /nvmedata/autobuild/instance-12/output-1/build/jpeg-turbo-2.1.3/simd/powerpc/jsimd_altivec.h:93:26: warning: implicit declaration of function 'vec_vsx_ld'; did you mean 'vec_vsl'? [-Wimplicit-function-declaration] >> 93 | #define VEC_LD(a, b) vec_vsx_ld(a, b) >> | ^~~~~~~~~~ >> Fixes: >> - http://autobuild.buildroot.org/results/be6d5ad0cee4ee19eb25e595d44555a1af6e073b >> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > Applied to master, thanks. Committed to 2022.05.x and 2022.02.x, thanks.
diff --git a/package/jpeg/Config.in b/package/jpeg/Config.in index 0a786060d7..371f89aa49 100644 --- a/package/jpeg/Config.in +++ b/package/jpeg/Config.in @@ -3,7 +3,8 @@ config BR2_PACKAGE_JPEG_SIMD_SUPPORT bool default y if BR2_X86_CPU_HAS_MMX default y if BR2_ARM_CPU_HAS_NEON - default y if BR2_POWERPC_CPU_HAS_ALTIVEC + default y if BR2_POWERPC_CPU_HAS_ALTIVEC && !BR2_powerpc64le + default y if BR2_POWERPC_CPU_HAS_VSX && BR2_powerpc64le default y if BR2_aarch64 || BR2_aarch64_be config BR2_PACKAGE_JPEG
SIMD needs VSX with little endian to avoid the following build failure: In file included from /nvmedata/autobuild/instance-12/output-1/build/jpeg-turbo-2.1.3/simd/powerpc/jccolor-altivec.c:25: /nvmedata/autobuild/instance-12/output-1/build/jpeg-turbo-2.1.3/simd/powerpc/jccolext-altivec.c: In function 'jsimd_rgb_ycc_convert_altivec': /nvmedata/autobuild/instance-12/output-1/build/jpeg-turbo-2.1.3/simd/powerpc/jsimd_altivec.h:93:26: warning: implicit declaration of function 'vec_vsx_ld'; did you mean 'vec_vsl'? [-Wimplicit-function-declaration] 93 | #define VEC_LD(a, b) vec_vsx_ld(a, b) | ^~~~~~~~~~ Fixes: - http://autobuild.buildroot.org/results/be6d5ad0cee4ee19eb25e595d44555a1af6e073b Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> --- package/jpeg/Config.in | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)