This is an archive of the discontinued LLVM Phabricator instance.

[MCA] Remove SM.hasNext() call in FetchStage::execute.
ClosedPublic

Authored by orodley on Sep 27 2018, 10:31 PM.

Details

Summary

This is redundant, as FetchStage::getNextInstruction already checks this
and returns llvm::ErrorSuccess() as appropriate.

NFC.

Diff Detail

Event Timeline

orodley created this revision.Sep 27 2018, 10:31 PM
andreadb accepted this revision.Sep 28 2018, 1:18 AM

LGTM.

Thanks!

This revision is now accepted and ready to land.Sep 28 2018, 1:18 AM
This revision was automatically updated to reflect the committed changes.