[Lldb-commits] [PATCH] D114178: [lldb] Fix formatted log statement
Keith Smiley via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Nov 18 10:41:05 PST 2021
keith created this revision.
keith added reviewers: kastiglione, aprantl.
keith requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.
Previously this would output literally without replacements
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D114178
Files:
lldb/source/Core/Module.cpp
Index: lldb/source/Core/Module.cpp
===================================================================
--- lldb/source/Core/Module.cpp
+++ lldb/source/Core/Module.cpp
@@ -1627,10 +1627,10 @@
bool Module::MergeArchitecture(const ArchSpec &arch_spec) {
if (!arch_spec.IsValid())
return false;
- LLDB_LOG(GetLogIfAllCategoriesSet(LIBLLDB_LOG_OBJECT | LIBLLDB_LOG_MODULES),
- "module has arch %s, merging/replacing with arch %s",
- m_arch.GetTriple().getTriple().c_str(),
- arch_spec.GetTriple().getTriple().c_str());
+ LLDB_LOGF(GetLogIfAllCategoriesSet(LIBLLDB_LOG_OBJECT | LIBLLDB_LOG_MODULES),
+ "module has arch %s, merging/replacing with arch %s",
+ m_arch.GetTriple().getTriple().c_str(),
+ arch_spec.GetTriple().getTriple().c_str());
if (!m_arch.IsCompatibleMatch(arch_spec)) {
// The new architecture is different, we just need to replace it.
return SetArchitecture(arch_spec);
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D114178.388264.patch
Type: text/x-patch
Size: 970 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20211118/67b07f2f/attachment.bin>
More information about the lldb-commits
mailing list