Re: [FFmpeg-devel] [PATCHv2] avutil/intmath: use de Bruijn based ff_ctz

2015-10-14 Thread Ronald S. Bultje
Hi, On Wed, Oct 14, 2015 at 1:26 PM, Ganesh Ajjanagadde wrote: > On Wed, Oct 14, 2015 at 1:13 PM, Ronald S. Bultje > wrote: > > Hi, > > > > On Wed, Oct 14, 2015 at 10:26 AM, Ganesh Ajjanagadde > > wrote: > >> > >> It has already been demonstrated that the de Bruijn method has benefits > >> ove

Re: [FFmpeg-devel] [PATCHv2] avutil/intmath: use de Bruijn based ff_ctz

2015-10-14 Thread Ganesh Ajjanagadde
On Wed, Oct 14, 2015 at 1:13 PM, Ronald S. Bultje wrote: > Hi, > > On Wed, Oct 14, 2015 at 10:26 AM, Ganesh Ajjanagadde > wrote: >> >> It has already been demonstrated that the de Bruijn method has benefits >> over the current implementation: commit >> 971d12b7f9d7be3ca8eb98e6c04ed521f83cbd3c. >>

Re: [FFmpeg-devel] [PATCHv2] avutil/intmath: use de Bruijn based ff_ctz

2015-10-14 Thread Ronald S. Bultje
Hi, On Wed, Oct 14, 2015 at 10:26 AM, Ganesh Ajjanagadde wrote: > It has already been demonstrated that the de Bruijn method has benefits > over the current implementation: commit > 971d12b7f9d7be3ca8eb98e6c04ed521f83cbd3c. > That commit implemented it for long long, this extends it to the int >

[FFmpeg-devel] [PATCHv2] avutil/intmath: use de Bruijn based ff_ctz

2015-10-14 Thread Ganesh Ajjanagadde
It has already been demonstrated that the de Bruijn method has benefits over the current implementation: commit 971d12b7f9d7be3ca8eb98e6c04ed521f83cbd3c. That commit implemented it for long long, this extends it to the int version. Tested with FATE. Signed-off-by: Ganesh Ajjanagadde --- libavu