[FFmpeg-devel] [PATCH] Fix compilation of FASTDIV() with llvm-gcc on x86_64

[Måns Rullgård]

Reimar D?ffinger <Reimar.Doeffinger at gmx.de> writes:

> On Mon, Jul 05, 2010 at 06:45:38PM +0100, M?ns Rullg?rd wrote:
>> Reimar D?ffinger <Reimar.Doeffinger at gmx.de> writes:
>> 
>> > On Mon, Jul 05, 2010 at 02:54:09PM +0100, Mans Rullgard wrote:
>> >> LLVM requires matching input and output asm parameters to have the
>> >> same type.  For the FASTDIV macro, an (int) cast on the input fixes
>> >> this trivially.
>> >
>> > Uh, where is a different type than int used with FASTDIV?
>> > I am not at all convinced those uses are really correct.
>> 
>> vorbis_dec.c line 1329.
>
> partition_size being uint32t_fast seems completely pointless to me,
> it is not used for array indexing,

What has array indexing go to do with anything?

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