id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
70	ROUNDING_CONTROL_BIPRED encoding time issue	rpanchal@…		"I noticed that in TComRdCost.cpp, all the functions for SAD,SSE,HAD where bRound is being passed as parameter, xClip() is being used which  affects the encoding time. xClip() is redundant step.
(255+255+0)>>1 = 255 and (255+255+1)>>1 = 255.

So xClip should be removed from all these functions.

For e.g.
So in TMuC0.7.2, in TComRdCost.cpp, line 471 can be changed
from 
pred = xClip( (piCur[n] + piRef[n] + bRound) >> 1 );
to
pred =        (piCur[n] + piRef[n] + bRound) >> 1  ;

Results should still be bitexact."	defect	closed	minor		HM		fixed		martak@… peisongc@… bossen@… ken@… rpanchal@… hurumi@… fbossen ksuehring davidf jct-vc@…