This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Fix GCNSchedStrategy.cpp debug output
ClosedPublic

Authored by rampitec on Feb 6 2017, 3:20 PM.

Reviewers

Commits

Summary

There is typo in the debug output: top and bottom candidates are switched.

LGTM

This revision is now accepted and ready to land.Feb 6 2017, 3:24 PM

This revision was automatically updated to reflect the committed changes.

Path

Size

lib/

Target/

AMDGPU/

4 lines

Show First 20 Lines • Show All 219 Lines • ▼ Show 20 Lines	if (!TopCand.isValid() \|\| TopCand.SU->isScheduled \|\|
assert(TopCand.Reason != NoCand && "failed to find the first candidate");		assert(TopCand.Reason != NoCand && "failed to find the first candidate");
} else {		} else {
DEBUG(traceCandidate(TopCand));		DEBUG(traceCandidate(TopCand));
}		}

// Pick best from BotCand and TopCand.		// Pick best from BotCand and TopCand.
DEBUG(		DEBUG(
dbgs() << "Top Cand: ";		dbgs() << "Top Cand: ";
traceCandidate(BotCand);
dbgs() << "Bot Cand: ";
traceCandidate(TopCand);		traceCandidate(TopCand);
		dbgs() << "Bot Cand: ";
		traceCandidate(BotCand);
);		);
SchedCandidate Cand;		SchedCandidate Cand;
if (TopCand.Reason == BotCand.Reason) {		if (TopCand.Reason == BotCand.Reason) {
Cand = BotCand;		Cand = BotCand;
GenericSchedulerBase::CandReason TopReason = TopCand.Reason;		GenericSchedulerBase::CandReason TopReason = TopCand.Reason;
TopCand.Reason = NoCand;		TopCand.Reason = NoCand;
GenericScheduler::tryCandidate(Cand, TopCand, nullptr);		GenericScheduler::tryCandidate(Cand, TopCand, nullptr);
if (TopCand.Reason != NoCand) {		if (TopCand.Reason != NoCand) {
▲ Show 20 Lines • Show All 75 Lines • Show Last 20 Lines