[FFmpeg-devel] [PATCH v2 11/13] Remove the MIPS "generic" core in favor of "*"
andreas.cadhalpun at googlemail.com
Wed Dec 2 18:14:26 CET 2015
On 02.12.2015 15:54, Vicente Olivert Riera wrote:
> On 20/11/15 20:10, Andreas Cadhalpun wrote:
>> 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.
> Aha, I see.
> But I don't understand why msa is disabled for a generic cpu and not the
> dsp or dspr2.
That was added in commit 8af3ce5, with the following comment:
"MIPS 'generic' case is added, with mips32r2 arch as default (fpu and dsp opt enabled)."
I'm not sure why this combination was chosen for the generic case.
> Either we disable everything, or nothing. But just
> disabling features randomly doesn't make any sense, at least for me.
> What about this?
> # We don't disable anything. Is up to the user to
> # disable the unwanted features.
That would be fine with me.
> # Unknown CPU. Disable everything.
I think it would be good if configure printed a warning in this case.
> disable mipsfpu
> disable mips32r2
More information about the ffmpeg-devel