[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#56884] [PATCH] Make x265 build on ppc64le

From: Thiago Jung Bauermann
Subject: [bug#56884] [PATCH] Make x265 build on ppc64le
Date: Tue, 09 Aug 2022 23:16:35 -0300

Hello Marcel,

Marcel van der Boom <> writes:

> Anything else needed here?
> The patch seems consistent with other packagers I think.

Thank you for the patch! It looks good to me. I only have one
suggestion: IMHO it would be a good idea to have a short comment above
the “(if (target-ppc64le?) …)” line explaining why it's needed, such as
“Enabling AltiVec support causes compilation errors.” or something

Ideally upstream should be notified about this problem so that they are
aware of it, but looking at the x265 website I can't find any bug

> Others are also confused on what is needed for the altivec code
> between different POWER processors.

Sorry, I'm not sure what you mean.

PS: Just for the sake of documenting in the issue tracker why x265 isn't
building on ppc64le — the build fails because of many errors such as:

 error: no matches converting function ‘sad16_altivec’ to type 
‘x265_12bit::pixelcmp_t’ {aka ‘int (*)(const short unsigned int*, long int, 
const short unsigned int*, long int)’}
 4199 |         p.pu[LUMA_ ## W ## x ## H].sad = sad16_altivec<W, H>; \
      |                                          ^~~~~~~~~~~~~~~~~~~


reply via email to

[Prev in Thread] Current Thread [Next in Thread]