[FFmpeg-devel] llvm failure : don't do undefined shift

Matthieu CASTET castet.matthieu
Fri Aug 21 18:31:39 CEST 2009


M?ns Rullg?rd <mans <at> mansr.com> writes:

> 
> Michael Niedermayer <michaelni <at> gmx.at> writes:
> 
> > that clearly splits the put_bits()
> 
> Splitting it in the rare cases where exactly 32 is needed is probably
> faster than checking every time.  Bear in mind that a lot of calls
> have variable number of bits, so the compiler can't remove the check
> even if we know it's always less than 32.
> 
May be we could add a put_bits_long for this case.
It will do the split without affecting code readability

Matthieu






More information about the ffmpeg-devel mailing list