Page MenuHomePhabricator

[AMDGPU] SIFoldOperands: try harder to fold cndmask instructions
ClosedPublic

Authored by foad on Thu, Apr 8, 6:19 AM.

Details

Summary

Look through copies to find more cases where the two values being
selected are identical. The motivation for this is just to be able to
remove the weird special case where tryFoldCndMask was called from
foldInstOperand, part way through folding a move-immediate into its
users, without regressing any lit tests.

Diff Detail

Event Timeline

foad created this revision.Thu, Apr 8, 6:19 AM
foad requested review of this revision.Thu, Apr 8, 6:19 AM
Herald added a project: Restricted Project. · View Herald TranscriptThu, Apr 8, 6:19 AM
foad added a comment.Thu, Apr 8, 6:19 AM

This removes the "Folded" set that @hliao added in D93174.

arsenm accepted this revision.Thu, Apr 8, 6:21 AM
This revision is now accepted and ready to land.Thu, Apr 8, 6:21 AM