On Wed, Aug 22, 2012 at 10:19 AM, Diego Biurrun <[email protected]> wrote:
> On Wed, Aug 22, 2012 at 10:15:14AM -0700, Daniel Kang wrote:
>> On Wed, Aug 22, 2012 at 9:14 AM, Diego Biurrun <[email protected]> wrote:
>> >
>> > On Tue, Aug 21, 2012 at 10:08:02PM -0500, [email protected] wrote:
>> > > From: Daniel Kang <[email protected]>
>> > >
>> > > The only CPUs that have 3dnow and don't have mmxext are 12 years old.
>> > >
>> > > Moreover, AMD has deprecated 3dnow.
>> > > ---
>> > >  libavcodec/x86/dsputil_mmx.c              |  142 
>> > > +----------------------------
>> > >  libavcodec/x86/dsputil_mmx_avg_template.c |    8 +-
>> > >  libavcodec/x86/h264_qpel_mmx.c            |    4 -
>> > >  3 files changed, 8 insertions(+), 146 deletions(-)
>> >
>> > You have sent this patch before, we have discussed this before and my
>> > last round of questions remains unanswered.
>>
>> What questions? I fixed the SKIP_FOR_3DNOW
>
> See my mail from July 25th.

Size:
The binary is about 418kb smaller.

Why drop this:
As mentioned in the commit message, 3dnow is deprecated by AMD. I see
no reason to support a deprecated instruction set. Also, it's a
maintenance burden for anyone who makes functional changes to the
code.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to