Skip to content

Commit

Permalink
[Clang][ASTMatchers] Add visibility macros to variables declared by m…
Browse files Browse the repository at this point in the history
…acros

This will fix missing symbols for ASTMatchersTests on windows when building with
CLANG_LINK_CLANG and explicit visibility macros are used.
This is part of the work to enable LLVM_BUILD_LLVM_DYLIB and
LLVM\Clang plugins on window.
  • Loading branch information
fsfod authored and vgvassilev committed Oct 14, 2024
1 parent 39ac8b2 commit bf2a9fe
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions clang/include/clang/ASTMatchers/ASTMatchersMacros.h
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,8 @@
#ifndef LLVM_CLANG_ASTMATCHERS_ASTMATCHERSMACROS_H
#define LLVM_CLANG_ASTMATCHERS_ASTMATCHERSMACROS_H

#include "clang/Support/Compiler.h"

/// AST_MATCHER_FUNCTION(ReturnType, DefineMatcher) { ... }
/// defines a zero parameter function named DefineMatcher() that returns a
/// ReturnType object.
Expand Down Expand Up @@ -367,7 +369,7 @@
static QualType (T::*value())() const { return &T::FunctionName; } \
}; \
} \
extern const ::clang::ast_matchers::internal:: \
CLANG_ABI extern const ::clang::ast_matchers::internal:: \
TypeTraversePolymorphicMatcher< \
QualType, \
::clang::ast_matchers::internal::TypeMatcher##MatcherName##Getter, \
Expand Down Expand Up @@ -407,7 +409,7 @@
static TypeLoc (T::*value())() const { return &T::FunctionName##Loc; } \
}; \
} \
extern const ::clang::ast_matchers::internal:: \
CLANG_ABI extern const ::clang::ast_matchers::internal:: \
TypeTraversePolymorphicMatcher< \
TypeLoc, \
::clang::ast_matchers::internal:: \
Expand Down

0 comments on commit bf2a9fe

Please sign in to comment.