This patch just adds the msvc and icl equivalent ctzll optimisations to
correspond with the recently added gcc variant
0001-lavu-intmath.h-Add-msvc-icl-ctzll-optimisations.patch
Description: Binary data
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.
On 10/15/2015 11:07 AM, Matt Oliver wrote:
> This patch just adds the msvc and icl equivalent ctzll optimisations to
> correspond with the recently added gcc variant
>
>
> 0001-lavu-intmath.h-Add-msvc-icl-ctzll-optimisations.patch
>
>
> From 2b8e7757cdb8595181a01bb18756d60662c41fde Mon Sep 17 0
On 16 October 2015 at 03:47, James Almer wrote:
> On 10/15/2015 11:07 AM, Matt Oliver wrote:
> > This patch just adds the msvc and icl equivalent ctzll optimisations to
> > correspond with the recently added gcc variant
> >
> >
> > 0001-lavu-intmath.h-Add-msvc-icl-ctzll-optimisations.patch
> >
>
On 10/15/2015 7:25 PM, Matt Oliver wrote:
> ICC defines __GNUC__ but ICL does not! ICC (being the linux variant)
> supports all the same functions as gcc so theres no need in this case for
> special handling. However ICL defines _MSC_VER so this is correct.
Ah, good to know then.
>> >
>>> > > +#d
On 16 October 2015 at 09:46, James Almer wrote:
> On 10/15/2015 7:25 PM, Matt Oliver wrote:
> > ICC defines __GNUC__ but ICL does not! ICC (being the linux variant)
> > supports all the same functions as gcc so theres no need in this case for
> > special handling. However ICL defines _MSC_VER so
On 16 October 2015 at 12:25, Matt Oliver wrote:
> On 16 October 2015 at 09:46, James Almer wrote:
>
>> On 10/15/2015 7:25 PM, Matt Oliver wrote:
>> > ICC defines __GNUC__ but ICL does not! ICC (being the linux variant)
>> > supports all the same functions as gcc so theres no need in this case
>>
On 10/16/2015 3:03 AM, Matt Oliver wrote:
> Take 3. Updated the patch based on comments so that it correctly supports
> msvc, icl and icc
LGTM.
How does the assembly ICL generates for the x86_32 version look like?
ICC seems to create a branch for the v == 0 check, so a handwritten
inline assembly
