diff options
| -rw-r--r-- | include/clang/AST/RecursiveASTVisitor.h | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/include/clang/AST/RecursiveASTVisitor.h b/include/clang/AST/RecursiveASTVisitor.h index 1a61255f04..2e56a486f3 100644 --- a/include/clang/AST/RecursiveASTVisitor.h +++ b/include/clang/AST/RecursiveASTVisitor.h @@ -464,12 +464,19 @@ template<typename Derived> bool RecursiveASTVisitor<Derived>::dataTraverseNode(Stmt *S, bool &EnqueueChildren) { +// The cast for DISPATCH_WALK is needed for older versions of g++, but causes +// problems for MSVC. So we'll skip the cast entirely for MSVC. +#if defined(_MSC_VER) + #define GCC_CAST(CLASS) +#else + #define GCC_CAST(CLASS) (bool (RecursiveASTVisitor::*)(CLASS*)) +#endif + // Dispatch to the corresponding WalkUpFrom* function only if the derived // class didn't override Traverse* (and thus the traversal is trivial). - // The cast here is necessary to work around a bug in old versions of g++. #define DISPATCH_WALK(NAME, CLASS, VAR) \ if (&RecursiveASTVisitor::Traverse##NAME == \ - (bool (RecursiveASTVisitor::*)(CLASS*))&Derived::Traverse##NAME) \ + GCC_CAST(CLASS)&Derived::Traverse##NAME) \ return getDerived().WalkUpFrom##NAME(static_cast<CLASS*>(VAR)); \ EnqueueChildren = false; \ return getDerived().Traverse##NAME(static_cast<CLASS*>(VAR)); @@ -509,6 +516,7 @@ bool RecursiveASTVisitor<Derived>::dataTraverseNode(Stmt *S, } #undef DISPATCH_WALK +#undef GCC_CAST return true; } |