[llvm] [LoopInfo][NewPM] Print function name in LoopPrinterPass (PR #76527)

Björn Pettersson via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 2 09:48:29 PST 2024


================
@@ -969,6 +969,8 @@ LoopInfo LoopAnalysis::run(Function &F, FunctionAnalysisManager &AM) {
 
 PreservedAnalyses LoopPrinterPass::run(Function &F,
                                        FunctionAnalysisManager &AM) {
+  OS << "Printing analysis 'Natural Loop Information' for function '"
----------------
bjope wrote:

I re-used the wording from legacy PM to make it backwards compatible. But I guess we aren't obliged to do that when it comes to debug printouts like these.

So the user that complained in https://discourse.llvm.org/t/need-usage-help-w-new-pass-manager-for-opt-analysis-natural-loop-information/75874 probably have to update the script parsing the output if we change the wording. But since we do not talk much about "natural loop information" elsewhere (for new PM) I agree that it could be confusing if we do it in these printouts only.

I'll make an update according to your suggestion!



https://github.com/llvm/llvm-project/pull/76527


More information about the llvm-commits mailing list