This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: llvm.amdgcn.interp.mov is a source of divergence
ClosedPublic

Authored by nhaehnle on Dec 8 2016, 6:22 AM.

Reviewers

• tstellarAMD
arsenm

Commits

Summary

While the result is constant across a single primitive, each pixel
shader wave can have pixels from multiple primitives.

nhaehnle retitled this revision from to AMDGPU: llvm.amdgcn.interp.mov is a source of divergence.

nhaehnle updated this object.

nhaehnle added a subscriber: llvm-commits.

Thanks for catching this. LGTM.

This revision is now accepted and ready to land.Dec 8 2016, 7:11 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

llvm/

trunk/

lib/

Target/

AMDGPU/

1 line

Show First 20 Lines • Show All 235 Lines • ▼ Show 20 Lines	static bool isIntrinsicSourceOfDivergence(const TargetIntrinsicInfo *TII,
case Intrinsic::not_intrinsic:		case Intrinsic::not_intrinsic:
// This means we have an intrinsic that isn't defined in		// This means we have an intrinsic that isn't defined in
// IntrinsicsAMDGPU.td		// IntrinsicsAMDGPU.td
break;		break;

case Intrinsic::amdgcn_workitem_id_x:		case Intrinsic::amdgcn_workitem_id_x:
case Intrinsic::amdgcn_workitem_id_y:		case Intrinsic::amdgcn_workitem_id_y:
case Intrinsic::amdgcn_workitem_id_z:		case Intrinsic::amdgcn_workitem_id_z:
		case Intrinsic::amdgcn_interp_mov:
case Intrinsic::amdgcn_interp_p1:		case Intrinsic::amdgcn_interp_p1:
case Intrinsic::amdgcn_interp_p2:		case Intrinsic::amdgcn_interp_p2:
case Intrinsic::amdgcn_mbcnt_hi:		case Intrinsic::amdgcn_mbcnt_hi:
case Intrinsic::amdgcn_mbcnt_lo:		case Intrinsic::amdgcn_mbcnt_lo:
case Intrinsic::r600_read_tidig_x:		case Intrinsic::r600_read_tidig_x:
case Intrinsic::r600_read_tidig_y:		case Intrinsic::r600_read_tidig_y:
case Intrinsic::r600_read_tidig_z:		case Intrinsic::r600_read_tidig_z:
case Intrinsic::amdgcn_image_atomic_swap:		case Intrinsic::amdgcn_image_atomic_swap:
▲ Show 20 Lines • Show All 88 Lines • Show Last 20 Lines