This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Fix asserting on 0 dmask for image intrinsics
ClosedPublic

Authored by arsenm on Feb 21 2017, 10:54 AM.

Details

Reviewers
arsenm
mareko
Summary

Fold these to undef during lowering so users get eliminated.

Diff Detail

Event Timeline

arsenm created this revision.Feb 21 2017, 10:54 AM
mareko edited edge metadata.Feb 21 2017, 12:20 PM

LGTM. (other than the fact I don't know why MERGE_VALUES is needed here)

LGTM. (other than the fact I don't know why MERGE_VALUES is needed here)

An output chain is expected, so something needs to be returned for it

arsenm accepted this revision.Mar 21 2017, 9:44 AM

r298387

This revision is now accepted and ready to land.Mar 21 2017, 9:44 AM
arsenm closed this revision.Mar 21 2017, 9:44 AM