Skip to content

Conversation

@llvmbot
Copy link
Member

@llvmbot llvmbot commented Jul 15, 2025

Backport 116110e

Requested by: @efriedma-quic

@llvmbot
Copy link
Member Author

llvmbot commented Jul 15, 2025

@Endilll What do you think about merging this PR to the release branch?

@llvmbot llvmbot requested a review from Endilll July 15, 2025 22:46
@llvmbot llvmbot added clang Clang issues not falling into any other category clang:as-a-library libclang and C++ API labels Jul 15, 2025
@llvmbot
Copy link
Member Author

llvmbot commented Jul 15, 2025

@llvm/pr-subscribers-clang

Author: None (llvmbot)

Changes

Backport 116110e

Requested by: @efriedma-quic


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

1 Files Affected:

  • (modified) clang/tools/libclang/libclang.map (+1-1)
diff --git a/clang/tools/libclang/libclang.map b/clang/tools/libclang/libclang.map
index d140a71e771a0..49c472e3833fd 100644
--- a/clang/tools/libclang/libclang.map
+++ b/clang/tools/libclang/libclang.map
@@ -435,12 +435,12 @@ LLVM_20 {
     clang_getTypePrettyPrinted;
     clang_isBeforeInTranslationUnit;
     clang_visitCXXBaseClasses;
-    clang_visitCXXMethods;
 };
 
 LLVM_21 {
   global:
     clang_getFullyQualifiedName;
+    clang_visitCXXMethods;
     clang_Cursor_getGCCAssemblyTemplate;
     clang_Cursor_isGCCAssemblyHasGoto;
     clang_Cursor_getGCCAssemblyNumOutputs;

@github-project-automation github-project-automation bot moved this from Needs Triage to Needs Merge in LLVM Release Status Jul 16, 2025
Happened to spot this while looking at libclang.map for other reasons.
clang_visitCXXMethods was added in LLVM 21, not LLVM 20.

(cherry picked from commit 116110e)
@tru tru merged commit 04d4be5 into llvm:release/21.x Jul 17, 2025
@github-project-automation github-project-automation bot moved this from Needs Merge to Done in LLVM Release Status Jul 17, 2025
@github-actions
Copy link

@efriedma-quic (or anyone else). If you would like to add a note about this fix in the release notes (completely optional). Please reply to this comment with a one or two sentence description of the fix. When you are done, please add the release:note label to this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clang:as-a-library libclang and C++ API clang Clang issues not falling into any other category

Projects

Development

Successfully merging this pull request may close these issues.

4 participants