[FFmpeg-devel] [PATCH 1/2] configure: permit POWER9 cpu flags

Rémi Denis-Courmont remi at remlab.net
Sat Jul 13 11:46:14 EEST 2024


Le torstaina 4. heinäkuuta 2024, 4.23.29 EEST Sean McGovern a écrit :
> ---
>  configure | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configure b/configure
> index b28221f258..bbda7a02cb 100755
> --- a/configure
> +++ b/configure
> @@ -5493,7 +5493,7 @@ elif enabled ppc; then
>              cpuflags="-mcpu=$cpu"
>              disable vsx
>          ;;
> -        power[7-8]*)
> +        power[7-9]*)
>              cpuflags="-mcpu=$cpu"
>          ;;
>          cell)

IMO, the CPU should just be passed as, well, CPU by default - like AArch64 
already does. PPC does not have that many CPU types. But generally speaking, 
keeping up with all CPU model names that GCC and Clang seems like a very vain 
and fruitless exercise.

Of course we can keep special case matching for *existing* special values that 
FFmpeg has historically accepted, but lets maybe stop adding new ones?

-- 
レミ・デニ-クールモン
http://www.remlab.net/





More information about the ffmpeg-devel mailing list