On Tue, Sep 30, 2014 at 03:51:55PM +0800, rongyan wrote: > Hi, > I present 4 patches to fix bugs for POWER8 little endian. > I will send 4 patches in 4 different email. This is the first. > The fate test result after merge these 4 patches can be found on > http://fate.ffmpeg.org/ by search "ibmcrl", also attached here to facilitate > the review: > > > > The passed test cases increased from 1649/2169 to 1675/2174.
This one seems wrong. I assume HAVE_VSX is a feature related define, which means it should be possible to disable it without breaking anything. If the altivec implementation is broken on little-endian, the correct bug-fix is to disable it. Then as a next step can a different optimized implementation be added. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
