On Sat, Jan 23, 2016 at 01:40:53AM -0300, James Almer wrote: > On 1/21/2016 11:39 PM, Michael Niedermayer wrote: > > On Thu, Jan 21, 2016 at 10:45:55AM +0000, John Cox wrote: > >> Hi > >> > >> v2 of my hevc residual patch > >> > >> I've fixed the fate regression > >> I've split it into more pieces > >> Now uses ff_clz > >> Some reformating of function headers > >> > >> The patches can also be found on > >> https://github.com/jc-kynesim/rpi-ffmpeg.git on branch > >> test/ff_hevc_cabac_4 from tag ff_hevc_cabac_4_base > >> > >> Note that I will be going on holiday from the end of Friday (UK time) > >> till the 1st Feb and will be unable to edit code or read this list > >> during that period. > > > > applied the first 3 patches > > one #endif was moved to another patch, one typo fixed > > and confirmed that clz is faster > > > > thx > > http://fate.ffmpeg.org/report.cgi?time=20160123040651&slot=x86_64-archlinux-gcc-valgrindundef > The "cabac_functions: Count zeros with ctz if it is fast" commit seems to be > at fault.
fixed this btw is not a regression, just changing the number of writen symbols, the valgrind failure occurs before already it just didnt occur with the number and seed used in fate [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Freedom in capitalist society always remains about the same as it was in ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
