From 4c8c903b59b203cc104d8ae18a10d0437ac3fc20 Mon Sep 17 00:00:00 2001 From: Alexey Bataev Date: Wed, 9 Jul 2014 10:00:31 +0000 Subject: [PATCH] Fixed incompatibility with MSVC builds in range-based for loops in DataRecursiveASTVisitor and RecursiveASTVisitor for OpenMP constructs. llvm-svn: 212601 --- clang/include/clang/AST/DataRecursiveASTVisitor.h | 6 ++++-- clang/include/clang/AST/RecursiveASTVisitor.h | 6 ++++-- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/clang/include/clang/AST/DataRecursiveASTVisitor.h b/clang/include/clang/AST/DataRecursiveASTVisitor.h index 5a0e97443506..e2dcdbe392af 100644 --- a/clang/include/clang/AST/DataRecursiveASTVisitor.h +++ b/clang/include/clang/AST/DataRecursiveASTVisitor.h @@ -2273,8 +2273,9 @@ DEF_TRAVERSE_STMT(AsTypeExpr, {}) template bool RecursiveASTVisitor::TraverseOMPExecutableDirective( OMPExecutableDirective *S) { - for (auto *C : S->clauses()) + for (auto *C : S->clauses()) { TRY_TO(TraverseOMPClause(C)); + } return true; } @@ -2376,8 +2377,9 @@ bool RecursiveASTVisitor::VisitOMPNowaitClause(OMPNowaitClause *) { template template bool RecursiveASTVisitor::VisitOMPClauseList(T *Node) { - for (auto *E : Node->varlists()) + for (auto *E : Node->varlists()) { TRY_TO(TraverseStmt(E)); + } return true; } diff --git a/clang/include/clang/AST/RecursiveASTVisitor.h b/clang/include/clang/AST/RecursiveASTVisitor.h index 5e7d0b8284ce..b4c4fd6ddb9d 100644 --- a/clang/include/clang/AST/RecursiveASTVisitor.h +++ b/clang/include/clang/AST/RecursiveASTVisitor.h @@ -2295,8 +2295,9 @@ DEF_TRAVERSE_STMT(AsTypeExpr, {}) template bool RecursiveASTVisitor::TraverseOMPExecutableDirective( OMPExecutableDirective *S) { - for (auto *C : S->clauses()) + for (auto *C : S->clauses()) { TRY_TO(TraverseOMPClause(C)); + } return true; } @@ -2398,8 +2399,9 @@ bool RecursiveASTVisitor::VisitOMPNowaitClause(OMPNowaitClause *) { template template bool RecursiveASTVisitor::VisitOMPClauseList(T *Node) { - for (auto *E : Node->varlists()) + for (auto *E : Node->varlists()) { TRY_TO(TraverseStmt(E)); + } return true; }