[FFmpeg-devel] [PATCH v2] lsws/ppc/yuv2rgb_altivec: Fix build in non-VSX environments with Clang

[Ramiro Polla]

Hi Brad,

On Wed, Mar 5, 2025 at 2:24 AM Brad Smith
<brad-at-comstyle.com at ffmpeg.org> wrote:
> On 2023-08-23 4:52 p.m., Michael Niedermayer wrote:
> > On Fri, Aug 18, 2023 at 10:14:04PM -0400, Brad Smith wrote:
> >> lsws/ppc/yuv2rgb_altivec: Fix build in non-VSX environments with Clang
> >>
> >> Add a check for the existence of the vec_xl() function. Clang provides
> >> the function even with VSX not enabled.
> >>
> >> v2: test for function if AltiVec is enabled instead of with AltiVec and without VSX
> >> ---
> >>   configure                        | 8 ++++++++
> >>   libswscale/ppc/yuv2rgb_altivec.c | 4 ++--
> >>   2 files changed, 10 insertions(+), 2 deletions(-)
> > Has this been tested on an affected platform ?
> > I mean the function is provided but does it also work ?
>
> This has been in the FreeBSD / OpenBSD ports for years. So to a certain
> extent yes. Anyway, I didn't have access to my test VM for quite some
> time but I do now and ran the various tests with FATE and did not see
> any issues.

Build doesn't seem to fail anymore, but that's probably because of the
#if 0 from b9eaf6e05c2ca16d94869e0263236dbdac752400.

Could you please send a rebased patch?

Ramiro