This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Remove register operand from si_mask_branch
ClosedPublic

Authored by arsenm on Aug 25 2016, 12:14 PM.

Details

Reviewers
tstellarAMD
Summary

It isn't used for anything, and is also misleading since
it could be spilled at the end of the block, so it can't be relied
on. There ends up being a verifier error about using an undefined
register since the spill kills the register.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 69281.Aug 25 2016, 12:14 PM
arsenm retitled this revision from to AMDGPU: Remove register operand from si_mask_branch.
arsenm updated this object.
arsenm added a reviewer: tstellarAMD.
arsenm added a subscriber: llvm-commits.
tstellarAMD accepted this revision.Aug 26 2016, 4:55 PM
tstellarAMD edited edge metadata.
This revision is now accepted and ready to land.Aug 26 2016, 4:55 PM
arsenm closed this revision.Aug 26 2016, 5:51 PM

r279899