Re: [FFmpeg-devel] [PATCH 1/2] configure: permit POWER9 cpu flags - Rémi Denis-Courmont

From: "Rémi Denis-Courmont" <remi@remlab.net>
To: ffmpeg-devel@ffmpeg.org
Subject: Re: [FFmpeg-devel] [PATCH 1/2] configure: permit POWER9 cpu flags
Date: Sat, 13 Jul 2024 11:46:14 +0300
Message-ID: <2274798.xIjfq26nYX@basile.remlab.net> (raw)
In-Reply-To: <20240704012330.2046242-1-gseanmcg@gmail.com>

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/

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".