[FFmpeg-cvslog] r25146 - trunk/libavcodec/x86/h264dsp_mmx.c

[Måns Rullgård]

Carl Eugen Hoyos <cehoyos at ag.or.at> writes:

> M?ns Rullg?rd <mans <at> mansr.com> writes:
>
>> >> x86: remove hack disabling sse2 h264 loop filter with 32-bit icc
>> >
>> > This broke icc 10.1.
>> >
>> > Please fix
>> 
>> If you want the compiler fixed, you need to contact your Intel
>> representative.  Is 10.1 even supported still?
>
> It is supported.

Then I suppose a minimal effort could be made to not break too badly
when using it.  However, is there any reason for not using version 11?

> You changed a file you do not maintain

Nor do you maintain it.  The change as discussed with the x86 gurus on
IRC.

> without testing 

I tested it with icc 11.1 and found it to work flawlessly.

> and without sending a patch.

As I said, it was discussed with those responsible for x86 support.

> Please revert (then I will fix it) or fix it properly.

I'll see about a proper fix.

-- 
M?ns Rullg?rd
mans at mansr.com