This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Remove unnecessary check for constant operands
ClosedPublic

Authored by arsenm on Jul 7 2017, 2:05 PM.

Details

Reviewers
rampitec
Summary

An instruction that has an immediate operand can't reach
this point. This is only called for a freshly shrunk instruction,
which prevously couldn't have had a literal constant operand.
This was also not conservative enough since it woudl also have
had to filter other constant-like inputs like frame indexes.

Diff Detail

Event Timeline

arsenm created this revision.Jul 7 2017, 2:05 PM
This revision is now accepted and ready to land.Jul 8 2017, 12:17 AM
arsenm closed this revision.Jul 10 2017, 12:34 PM

r307574