From: Michael Buch Date: Thu, 25 Aug 2022 22:41:47 +0000 (+0100) Subject: [lldb][ClangExpression] Fix LLDB_LOG incorrect format specifier X-Git-Tag: upstream/17.0.6~35315 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=2b95b50dd69b03be793357cbadae70d413b375a5;p=platform%2Fupstream%2Fllvm.git [lldb][ClangExpression] Fix LLDB_LOG incorrect format specifier Previously this would log: ``` FindExternalLexicalDecls on (ASTContext*)0x00000005CE825200 'Expression ASTContext for ''' in 'weak_ptr' (%sDecl*)ClassTemplateSpecialization FindExternalLexicalDecls on (ASTContext*)0x00000005CE825200 'Expression ASTContext for ''' in '__shared_count' (%sDecl*)CXXRecord ``` Note that the `%s` isn't actually respected. This patch fixes this by providing the format specifiers that `lldb::formatv` supports. Differential Revision: https://reviews.llvm.org/D132709 --- diff --git a/lldb/source/Plugins/ExpressionParser/Clang/ClangASTSource.cpp b/lldb/source/Plugins/ExpressionParser/Clang/ClangASTSource.cpp index 7124292..3187146 100644 --- a/lldb/source/Plugins/ExpressionParser/Clang/ClangASTSource.cpp +++ b/lldb/source/Plugins/ExpressionParser/Clang/ClangASTSource.cpp @@ -405,7 +405,7 @@ void ClangASTSource::FindExternalLexicalDecls( if (const NamedDecl *context_named_decl = dyn_cast(context_decl)) LLDB_LOG(log, "FindExternalLexicalDecls on (ASTContext*){0} '{1}' in " - "'{2}' (%sDecl*){3}", + "'{2}' ({3}Decl*){4}", m_ast_context, m_clang_ast_context->getDisplayName(), context_named_decl->getNameAsString().c_str(), context_decl->getDeclKindName(),