id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
117	Redundant memory copy in predInterSearch()	chiaming		"In predInterSearch(), the cMvTemp would be copied two times as follows:
{{{
if ( cu.Y().width > 8 && cu.Y().height > 8 && cu.partSize == SIZE_2Nx2N && cu.slice->getSPS()->getSpsNext().getUseAffine() 
  && cu.imv == 0
  )
{
  ::memcpy( cMvHevcTemp, cMvTemp, sizeof( cMvTemp ) );
}
if ( cu.Y().width > 8 && cu.Y().height > 8 && cu.partSize == SIZE_2Nx2N && cu.slice->getSPS()->getSpsNext().getUseAffine() )
{
  ::memcpy( cMvHevcTemp, cMvTemp, sizeof( cMvTemp ) );
}
}}}


Suggest to remove the redundant memory copy and also sync the conditions as doing affine search
{{{
if (cu.Y().width > 8 && cu.Y().height > 8 && cu.partSize == SIZE_2Nx2N && cu.slice->getSPS()->getSpsNext().getUseAffine() && cu.imv == 0
#if JVET_L0646_GBI
  && (gbiIdx == GBI_DEFAULT || m_affineModeSelected || !m_pcEncCfg->getUseGBiFast())
#endif      
  )
{
  ::memcpy( cMvHevcTemp, cMvTemp, sizeof( cMvTemp ) );
}
}}}

"	enhancement	closed	minor		VTM	VTM-3.0rc1	fixed		ksuehring XiangLi fbossen jvet@…