Skip to content

Commit efa0cff

Browse files
committed
Review feedback
1 parent 0f8e77b commit efa0cff

File tree

1 file changed

+9
-11
lines changed

1 file changed

+9
-11
lines changed

llvm/lib/Transforms/Scalar/LoopFuse.cpp

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -449,15 +449,6 @@ struct FusionCandidateCompare {
449449

450450
using LoopVector = SmallVector<Loop *, 4>;
451451

452-
#ifndef NDEBUG
453-
static void printLoopVector(const LoopVector &LV) {
454-
dbgs() << "****************************\n";
455-
for (const Loop *L : LV)
456-
printLoop(*L, dbgs());
457-
dbgs() << "****************************\n";
458-
}
459-
#endif
460-
461452
// Set of Control Flow Equivalent (CFE) Fusion Candidates, sorted in dominance
462453
// order. Thus, if FC0 comes *before* FC1 in a FusionCandidateSet, then FC0
463454
// dominates FC1 and FC1 post-dominates FC0.
@@ -471,7 +462,14 @@ static void printLoopVector(const LoopVector &LV) {
471462
using FusionCandidateSet = std::set<FusionCandidate, FusionCandidateCompare>;
472463
using FusionCandidateCollection = SmallVector<FusionCandidateSet, 4>;
473464

474-
#if !defined(NDEBUG)
465+
#ifndef NDEBUG
466+
static void printLoopVector(const LoopVector &LV) {
467+
dbgs() << "****************************\n";
468+
for (const Loop *L : LV)
469+
printLoop(*L, dbgs());
470+
dbgs() << "****************************\n";
471+
}
472+
475473
static raw_ostream &operator<<(raw_ostream &OS, const FusionCandidate &FC) {
476474
if (FC.isValid())
477475
OS << FC.Preheader->getName();
@@ -498,7 +496,7 @@ printFusionCandidates(const FusionCandidateCollection &FusionCandidates) {
498496
dbgs() << "****************************\n";
499497
}
500498
}
501-
#endif
499+
#endif // NDEBUG
502500

503501
namespace {
504502

0 commit comments

Comments
 (0)