[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