Skip to content

Commit 4056287

Browse files
authored
[Flang] Clean up LoopVersioning LLVM_DEBUG blocks. NFC (#77818)
Just a little trick to put LLVM_DEBUG blocks into separate { } scopes, so they clang-format better.
1 parent 826fe84 commit 4056287

File tree

1 file changed

+32
-38
lines changed

1 file changed

+32
-38
lines changed

flang/lib/Optimizer/Transforms/LoopVersioning.cpp

Lines changed: 32 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -105,26 +105,24 @@ struct ArgsUsageInLoop {
105105
// Debug dump of the structure members assuming that
106106
// the information has been collected for the given loop.
107107
void dump(fir::DoLoopOp loop) const {
108-
// clang-format off
109-
LLVM_DEBUG(
110-
mlir::OpPrintingFlags printFlags;
111-
printFlags.skipRegions();
112-
llvm::dbgs() << "Arguments usage info for loop:\n";
113-
loop.print(llvm::dbgs(), printFlags);
114-
llvm::dbgs() << "\nUsed args:\n";
115-
for (auto &use : usageInfo) {
116-
mlir::Value v = use.first;
117-
v.print(llvm::dbgs(), printFlags);
118-
llvm::dbgs() << "\n";
119-
}
120-
llvm::dbgs() << "\nCannot transform args:\n";
121-
for (mlir::Value arg : cannotTransform) {
122-
arg.print(llvm::dbgs(), printFlags);
123-
llvm::dbgs() << "\n";
124-
}
125-
llvm::dbgs() << "====\n"
126-
);
127-
// clang-format on
108+
LLVM_DEBUG({
109+
mlir::OpPrintingFlags printFlags;
110+
printFlags.skipRegions();
111+
llvm::dbgs() << "Arguments usage info for loop:\n";
112+
loop.print(llvm::dbgs(), printFlags);
113+
llvm::dbgs() << "\nUsed args:\n";
114+
for (auto &use : usageInfo) {
115+
mlir::Value v = use.first;
116+
v.print(llvm::dbgs(), printFlags);
117+
llvm::dbgs() << "\n";
118+
}
119+
llvm::dbgs() << "\nCannot transform args:\n";
120+
for (mlir::Value arg : cannotTransform) {
121+
arg.print(llvm::dbgs(), printFlags);
122+
llvm::dbgs() << "\n";
123+
}
124+
llvm::dbgs() << "====\n";
125+
});
128126
}
129127

130128
// Erase usageInfo and cannotTransform entries for a set
@@ -344,15 +342,13 @@ void LoopVersioningPass::runOnOperation() {
344342
});
345343

346344
// Dump loops info after initial collection.
347-
// clang-format off
348-
LLVM_DEBUG(
349-
llvm::dbgs() << "Initial usage info:\n";
350-
for (fir::DoLoopOp loop : originalLoops) {
351-
auto &argsInLoop = argsInLoops[loop];
352-
argsInLoop.dump(loop);
353-
}
354-
);
355-
// clang-format on
345+
LLVM_DEBUG({
346+
llvm::dbgs() << "Initial usage info:\n";
347+
for (fir::DoLoopOp loop : originalLoops) {
348+
auto &argsInLoop = argsInLoops[loop];
349+
argsInLoop.dump(loop);
350+
}
351+
});
356352

357353
// Clear argument usage for parent loops if an inner loop
358354
// contains a non-transformable usage.
@@ -382,15 +378,13 @@ void LoopVersioningPass::runOnOperation() {
382378
});
383379
}
384380

385-
// clang-format off
386-
LLVM_DEBUG(
387-
llvm::dbgs() << "Final usage info:\n";
388-
for (fir::DoLoopOp loop : originalLoops) {
389-
auto &argsInLoop = argsInLoops[loop];
390-
argsInLoop.dump(loop);
391-
}
392-
);
393-
// clang-format on
381+
LLVM_DEBUG({
382+
llvm::dbgs() << "Final usage info:\n";
383+
for (fir::DoLoopOp loop : originalLoops) {
384+
auto &argsInLoop = argsInLoops[loop];
385+
argsInLoop.dump(loop);
386+
}
387+
});
394388

395389
// Reduce the collected information to a list of loops
396390
// with attached arguments usage information.

0 commit comments

Comments
 (0)