James Almer (2018-08-05): > This patch makes the code slightly less readable, IMO. What compiler > doesn't optimize out an integer division by 2 into a right shift?
Any compiler that cannot figure out that x is positive in this convoluted loop. Signed divisions cannot be optimized as shifts. Make everything unsigned and it will be fine. But I do not understand the logic in having i going from 0 to w-1 just to compute x from w-1 to 0. unsigned w, x; for (x = w - 1; x < w; x--) would be better IMHO. Regards, -- Nicolas George
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel