Skip to content

Conversation

@augusto2112
Copy link
Contributor

No description provided.

@llvmbot
Copy link
Member

llvmbot commented Aug 15, 2025

@llvm/pr-subscribers-lldb

Author: Augusto Noronha (augusto2112)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/153857.diff

1 Files Affected:

  • (modified) lldb/unittests/Language/CPlusPlus/CPlusPlusLanguageTest.cpp (+3-3)
diff --git a/lldb/unittests/Language/CPlusPlus/CPlusPlusLanguageTest.cpp b/lldb/unittests/Language/CPlusPlus/CPlusPlusLanguageTest.cpp
index cc0275163b344..957fb3f600499 100644
--- a/lldb/unittests/Language/CPlusPlus/CPlusPlusLanguageTest.cpp
+++ b/lldb/unittests/Language/CPlusPlus/CPlusPlusLanguageTest.cpp
@@ -420,10 +420,10 @@ TEST(CPlusPlusLanguage, MatchesCxx) {
 
   EXPECT_TRUE(CPlusPlusLang != nullptr);
 
-  Mangled itaniumSymbol("_ZFoo");
+  Mangled itaniumSymbol("_Z3Foo");
   EXPECT_TRUE(CPlusPlusLang->SymbolNameFitsToLanguage(itaniumSymbol));
-  Mangled itaniumExtensionSymbol("___ZBar");
+  Mangled itaniumExtensionSymbol("___Z3Bar_block_invoke");
   EXPECT_TRUE(CPlusPlusLang->SymbolNameFitsToLanguage(itaniumExtensionSymbol));
-  Mangled msvcSymbol("?Baz");
+  Mangled msvcSymbol("??x@@3AH");
   EXPECT_TRUE(CPlusPlusLang->SymbolNameFitsToLanguage(msvcSymbol));
 }

@augusto2112 augusto2112 enabled auto-merge (squash) August 15, 2025 19:36
@augusto2112 augusto2112 merged commit c61fb5c into llvm:main Aug 15, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants