This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: change a redundant if () to an assert(). NFC
ClosedPublic

Authored by nhaehnle on Apr 12 2016, 4:40 PM.

Details

Summary

I've been carrying this change around with me for a while, because the if ()
managed to confuse me while following the code. All callers ensure that the
assertion holds.

Diff Detail

Event Timeline

nhaehnle updated this revision to Diff 53493.Apr 12 2016, 4:40 PM
nhaehnle retitled this revision from to AMDGPU: change a redundant if () to an assert(). NFC.
nhaehnle updated this object.
nhaehnle added reviewers: arsenm, tstellarAMD.
nhaehnle added a subscriber: llvm-commits.
tstellarAMD accepted this revision.Apr 12 2016, 5:20 PM
tstellarAMD edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Apr 12 2016, 5:20 PM
This revision was automatically updated to reflect the committed changes.