This is an archive of the discontinued LLVM Phabricator instance.

[LoopUnroll] Fix a debug message. NFC.
ClosedPublic

Authored by haicheng on Sep 7 2016, 9:52 AM.

Reviewers

Commits

Summary

The code says "<=", but the debug message is "<".

haicheng retitled this revision from to [LoopUnroll] Fix a debug message. NFC..

haicheng updated this object.

haicheng set the repository for this revision to rL LLVM.

haicheng added a subscriber: llvm-commits.

LGTM, thanks!

This revision is now accepted and ready to land.Sep 7 2016, 9:59 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

llvm/

trunk/

lib/

Transforms/

Scalar/

2 lines

Show First 20 Lines • Show All 643 Lines • ▼ Show 20 Lines	static bool canUnrollCompletely(Loop *L, unsigned Threshold,
uint64_t RolledDynamicCost) {		uint64_t RolledDynamicCost) {
if (Threshold == NoThreshold) {		if (Threshold == NoThreshold) {
DEBUG(dbgs() << " Can fully unroll, because no threshold is set.\n");		DEBUG(dbgs() << " Can fully unroll, because no threshold is set.\n");
return true;		return true;
}		}

if (UnrolledCost <= Threshold) {		if (UnrolledCost <= Threshold) {
DEBUG(dbgs() << " Can fully unroll, because unrolled cost: "		DEBUG(dbgs() << " Can fully unroll, because unrolled cost: "
<< UnrolledCost << "<" << Threshold << "\n");		<< UnrolledCost << "<=" << Threshold << "\n");
return true;		return true;
}		}

assert(UnrolledCost && "UnrolledCost can't be 0 at this point.");		assert(UnrolledCost && "UnrolledCost can't be 0 at this point.");
assert(RolledDynamicCost >= UnrolledCost &&		assert(RolledDynamicCost >= UnrolledCost &&
"Cannot have a higher unrolled cost than a rolled cost!");		"Cannot have a higher unrolled cost than a rolled cost!");

// Compute the percentage of the dynamic cost in the rolled form that is		// Compute the percentage of the dynamic cost in the rolled form that is
▲ Show 20 Lines • Show All 441 Lines • Show Last 20 Lines