[FFmpeg-devel] [PATCH v2 11/13] Remove the MIPS "generic" core in favor of "*"

Andreas Cadhalpun andreas.cadhalpun at googlemail.com
Fri Nov 20 21:10:15 CET 2015


On 18.11.2015 12:13, Vicente Olivert Riera wrote:
> There is no point to have a "generic" core when we can catch all
> unsupported cores with the "*" option, so remove it.
> 
> Signed-off-by: Vicente Olivert Riera <Vincent.Riera at imgtec.com>
> ---
> Changes v1 -> v2:
>  - Nothing.
> 
>  configure |    4 ----
>  1 files changed, 0 insertions(+), 4 deletions(-)
> 
> diff --git a/configure b/configure
> index fc3d559..7d87e5c 100755
> --- a/configure
> +++ b/configure
> @@ -4154,10 +4154,6 @@ elif enabled mips; then
>                  ;;
>              esac
>          ;;
> -        generic)
> -            disable mips64r6
> -            disable msa
> -        ;;
>          *)
>              disable mipsfpu
>              disable mips32r2
> 

This is a bad idea.
If the cpu is not explicitly set via configure options, it is set to generic.
Removing this generic case here, will force-disable all mips optimizations,
even if they were explicitly enabled with configure options.
That would e.g. break how the Debian ffmpeg package is built for mips64el.

Best regards,
Andreas



More information about the ffmpeg-devel mailing list