[llvm] [ThinLTO] Don't always print ModulesToCompile debugging information (PR #106769)

Nick Sarnie via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 30 10:41:13 PDT 2024


https://github.com/sarnex created https://github.com/llvm/llvm-project/pull/106769

Nothing went wrong in this case, we just succesfully matched a module by identifier. No need to print to std::error like we would for something that should be user-visible.

>From 780119aa15900bfac2e1ed87bf69c983e26b5105 Mon Sep 17 00:00:00 2001
From: "Sarnie, Nick" <nick.sarnie at intel.com>
Date: Fri, 30 Aug 2024 10:31:51 -0700
Subject: [PATCH] [ThinLTO] Don't always print ModulesToCompile debugging
 information

Nothing went wrong in this case, we just succesfully matched a module by
identifier. No need to print to std::error like we would for something
that should be user-visible.

Signed-off-by: Sarnie, Nick <nick.sarnie at intel.com>
---
 llvm/lib/LTO/LTO.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/llvm/lib/LTO/LTO.cpp b/llvm/lib/LTO/LTO.cpp
index 09dfec03cb0c34..647d4ba7dafdc2 100644
--- a/llvm/lib/LTO/LTO.cpp
+++ b/llvm/lib/LTO/LTO.cpp
@@ -1078,8 +1078,8 @@ Error LTO::addThinLTO(BitcodeModule BM, ArrayRef<InputFile::Symbol> Syms,
     for (const std::string &Name : Conf.ThinLTOModulesToCompile) {
       if (BM.getModuleIdentifier().contains(Name)) {
         ThinLTO.ModulesToCompile->insert({BM.getModuleIdentifier(), BM});
-        llvm::errs() << "[ThinLTO] Selecting " << BM.getModuleIdentifier()
-                     << " to compile\n";
+        LLVM_DEBUG(dbgs() << "[ThinLTO] Selecting " << BM.getModuleIdentifier()
+                          << " to compile\n");
       }
     }
   }



More information about the llvm-commits mailing list