r211347 - [OPENMP] Initial support for 'ordered' clause.
Samuel F Antao
sfantao at us.ibm.com
Fri Jul 18 11:21:40 PDT 2014
Hi Alexey,
I took a close look into this commit and it looks good for me.
I only have a minor suggestion related to the testcase: it would be good to
add a case with the ordered clause being passed twice in order to test the
diagnostics, given that this check is done in code specific to that clause.
Regards,
Samuel
> Date: Fri, 20 Jun 2014 09:44:06 -0000
> From: Alexey Bataev <a.bataev at hotmail.com>
> To: cfe-commits at cs.uiuc.edu
> Subject: r211347 - [OPENMP] Initial support for 'ordered' clause.
> Message-ID: <20140620094407.347DD2A6C026 at llvm.org>
> Content-Type: text/plain; charset="utf-8"
>
> Author: abataev
> Date: Fri Jun 20 04:44:06 2014
> New Revision: 211347
>
> URL: http://llvm.org/viewvc/llvm-project?rev=211347&view=rev
> Log:
> [OPENMP] Initial support for 'ordered' clause.
>
> Modified:
> cfe/trunk/include/clang/AST/DataRecursiveASTVisitor.h
> cfe/trunk/include/clang/AST/OpenMPClause.h
> cfe/trunk/include/clang/AST/RecursiveASTVisitor.h
> cfe/trunk/include/clang/Basic/OpenMPKinds.def
> cfe/trunk/include/clang/Parse/Parser.h
> cfe/trunk/include/clang/Sema/Sema.h
> cfe/trunk/lib/AST/StmtPrinter.cpp
> cfe/trunk/lib/AST/StmtProfile.cpp
> cfe/trunk/lib/Basic/OpenMPKinds.cpp
> cfe/trunk/lib/Parse/ParseOpenMP.cpp
> cfe/trunk/lib/Sema/SemaOpenMP.cpp
> cfe/trunk/lib/Sema/TreeTransform.h
> cfe/trunk/lib/Serialization/ASTReaderStmt.cpp
> cfe/trunk/lib/Serialization/ASTWriterStmt.cpp
> cfe/trunk/test/OpenMP/for_ast_print.cpp
> cfe/trunk/tools/libclang/CIndex.cpp
>
> Modified: cfe/trunk/include/clang/AST/DataRecursiveASTVisitor.h
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/
> AST/DataRecursiveASTVisitor.h?rev=211347&r1=211346&r2=211347&view=diff
>
==============================================================================
> --- cfe/trunk/include/clang/AST/DataRecursiveASTVisitor.h (original)
> +++ cfe/trunk/include/clang/AST/DataRecursiveASTVisitor.h Fri Jun 20
> 04:44:06 2014
> @@ -2351,6 +2351,12 @@ RecursiveASTVisitor<Derived>::VisitOMPSc
> }
>
> template <typename Derived>
> +bool
> +RecursiveASTVisitor<Derived>::VisitOMPOrderedClause(OMPOrderedClause *)
{
> + return true;
> +}
> +
> +template <typename Derived>
> template <typename T>
> void RecursiveASTVisitor<Derived>::VisitOMPClauseList(T *Node) {
> for (auto *I : Node->varlists())
>
> Modified: cfe/trunk/include/clang/AST/OpenMPClause.h
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/
> AST/OpenMPClause.h?rev=211347&r1=211346&r2=211347&view=diff
>
==============================================================================
> --- cfe/trunk/include/clang/AST/OpenMPClause.h (original)
> +++ cfe/trunk/include/clang/AST/OpenMPClause.h Fri Jun 20 04:44:06 2014
> @@ -600,6 +600,35 @@ public:
> StmtRange children() { return StmtRange(&ChunkSize, &ChunkSize + 1); }
> };
>
> +/// \brief This represents 'ordered' clause in the '#pragma omp
> ...' directive.
> +///
> +/// \code
> +/// #pragma omp for ordered
> +/// \endcode
> +/// In this example directive '#pragma omp for' has 'ordered' clause.
> +///
> +class OMPOrderedClause : public OMPClause {
> +public:
> + /// \brief Build 'ordered' clause.
> + ///
> + /// \param StartLoc Starting location of the clause.
> + /// \param EndLoc Ending location of the clause.
> + ///
> + OMPOrderedClause(SourceLocation StartLoc, SourceLocation EndLoc)
> + : OMPClause(OMPC_ordered, StartLoc, EndLoc) {}
> +
> + /// \brief Build an empty clause.
> + ///
> + OMPOrderedClause()
> + : OMPClause(OMPC_ordered, SourceLocation(), SourceLocation()) {}
> +
> + static bool classof(const OMPClause *T) {
> + return T->getClauseKind() == OMPC_ordered;
> + }
> +
> + StmtRange children() { return StmtRange(); }
> +};
> +
> /// \brief This represents clause 'private' in the '#pragma omp
> ...' directives.
> ///
> /// \code
>
> Modified: cfe/trunk/include/clang/AST/RecursiveASTVisitor.h
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/
> AST/RecursiveASTVisitor.h?rev=211347&r1=211346&r2=211347&view=diff
>
==============================================================================
> --- cfe/trunk/include/clang/AST/RecursiveASTVisitor.h (original)
> +++ cfe/trunk/include/clang/AST/RecursiveASTVisitor.h Fri Jun 20 04:44:06
2014
> @@ -2372,6 +2372,12 @@ RecursiveASTVisitor<Derived>::VisitOMPSc
> }
>
> template <typename Derived>
> +bool
> +RecursiveASTVisitor<Derived>::VisitOMPOrderedClause(OMPOrderedClause *)
{
> + return true;
> +}
> +
> +template <typename Derived>
> template <typename T>
> void RecursiveASTVisitor<Derived>::VisitOMPClauseList(T *Node) {
> for (auto *I : Node->varlists())
>
> Modified: cfe/trunk/include/clang/Basic/OpenMPKinds.def
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/
> Basic/OpenMPKinds.def?rev=211347&r1=211346&r2=211347&view=diff
>
==============================================================================
> --- cfe/trunk/include/clang/Basic/OpenMPKinds.def (original)
> +++ cfe/trunk/include/clang/Basic/OpenMPKinds.def Fri Jun 20 04:44:06
2014
> @@ -60,6 +60,7 @@ OPENMP_CLAUSE(aligned, OMPAlignedClause)
> OPENMP_CLAUSE(copyin, OMPCopyinClause)
> OPENMP_CLAUSE(proc_bind, OMPProcBindClause)
> OPENMP_CLAUSE(schedule, OMPScheduleClause)
> +OPENMP_CLAUSE(ordered, OMPOrderedClause)
>
> // Clauses allowed for OpenMP directive 'parallel'.
> OPENMP_PARALLEL_CLAUSE(if)
> @@ -87,6 +88,7 @@ OPENMP_FOR_CLAUSE(firstprivate)
> OPENMP_FOR_CLAUSE(reduction)
> OPENMP_FOR_CLAUSE(collapse)
> OPENMP_FOR_CLAUSE(schedule)
> +OPENMP_FOR_CLAUSE(ordered)
>
> // Static attributes for 'default' clause.
> OPENMP_DEFAULT_KIND(none)
>
> Modified: cfe/trunk/include/clang/Parse/Parser.h
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/
> Parse/Parser.h?rev=211347&r1=211346&r2=211347&view=diff
>
==============================================================================
> --- cfe/trunk/include/clang/Parse/Parser.h (original)
> +++ cfe/trunk/include/clang/Parse/Parser.h Fri Jun 20 04:44:06 2014
> @@ -2351,6 +2351,11 @@ private:
> /// \param Kind Kind of current clause.
> ///
> OMPClause *ParseOpenMPSingleExprWithArgClause(OpenMPClauseKind Kind);
> + /// \brief Parses clause without any additional arguments.
> + ///
> + /// \param Kind Kind of current clause.
> + ///
> + OMPClause *ParseOpenMPClause(OpenMPClauseKind Kind);
> /// \brief Parses clause with the list of variables of a kind \a Kind.
> ///
> /// \param Kind Kind of current clause.
>
> Modified: cfe/trunk/include/clang/Sema/Sema.h
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/
> Sema/Sema.h?rev=211347&r1=211346&r2=211347&view=diff
>
==============================================================================
> --- cfe/trunk/include/clang/Sema/Sema.h (original)
> +++ cfe/trunk/include/clang/Sema/Sema.h Fri Jun 20 04:44:06 2014
> @@ -7381,6 +7381,12 @@ public:
> SourceLocation CommaLoc,
> SourceLocation EndLoc);
>
> + OMPClause *ActOnOpenMPClause(OpenMPClauseKind Kind,
> SourceLocation StartLoc,
> + SourceLocation EndLoc);
> + /// \brief Called on well-formed 'ordered' clause.
> + OMPClause *ActOnOpenMPOrderedClause(SourceLocation StartLoc,
> + SourceLocation EndLoc);
> +
> OMPClause *
> ActOnOpenMPVarListClause(OpenMPClauseKind Kind, ArrayRef<Expr *> Vars,
> Expr *TailExpr, SourceLocation StartLoc,
>
> Modified: cfe/trunk/lib/AST/StmtPrinter.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/
> StmtPrinter.cpp?rev=211347&r1=211346&r2=211347&view=diff
>
==============================================================================
> --- cfe/trunk/lib/AST/StmtPrinter.cpp (original)
> +++ cfe/trunk/lib/AST/StmtPrinter.cpp Fri Jun 20 04:44:06 2014
> @@ -638,6 +638,10 @@ void OMPClausePrinter::VisitOMPScheduleC
> OS << ")";
> }
>
> +void OMPClausePrinter::VisitOMPOrderedClause(OMPOrderedClause *) {
> + OS << "ordered";
> +}
> +
> template<typename T>
> void OMPClausePrinter::VisitOMPClauseList(T *Node, char StartSym) {
> for (typename T::varlist_iterator I = Node->varlist_begin(),
>
> Modified: cfe/trunk/lib/AST/StmtProfile.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/
> StmtProfile.cpp?rev=211347&r1=211346&r2=211347&view=diff
>
==============================================================================
> --- cfe/trunk/lib/AST/StmtProfile.cpp (original)
> +++ cfe/trunk/lib/AST/StmtProfile.cpp Fri Jun 20 04:44:06 2014
> @@ -293,6 +293,8 @@ void OMPClauseProfiler::VisitOMPSchedule
> Profiler->VisitStmt(C->getChunkSize());
> }
>
> +void OMPClauseProfiler::VisitOMPOrderedClause(const OMPOrderedClause *)
{}
> +
> template<typename T>
> void OMPClauseProfiler::VisitOMPClauseList(T *Node) {
> for (auto *I : Node->varlists())
>
> Modified: cfe/trunk/lib/Basic/OpenMPKinds.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Basic/
> OpenMPKinds.cpp?rev=211347&r1=211346&r2=211347&view=diff
>
==============================================================================
> --- cfe/trunk/lib/Basic/OpenMPKinds.cpp (original)
> +++ cfe/trunk/lib/Basic/OpenMPKinds.cpp Fri Jun 20 04:44:06 2014
> @@ -95,6 +95,7 @@ unsigned clang::getOpenMPSimpleClauseTyp
> case OMPC_linear:
> case OMPC_aligned:
> case OMPC_copyin:
> + case OMPC_ordered:
> break;
> }
> llvm_unreachable("Invalid OpenMP simple clause kind");
> @@ -147,6 +148,7 @@ const char *clang::getOpenMPSimpleClause
> case OMPC_linear:
> case OMPC_aligned:
> case OMPC_copyin:
> + case OMPC_ordered:
> break;
> }
> llvm_unreachable("Invalid OpenMP simple clause kind");
>
> Modified: cfe/trunk/lib/Parse/ParseOpenMP.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Parse/
> ParseOpenMP.cpp?rev=211347&r1=211346&r2=211347&view=diff
>
==============================================================================
> --- cfe/trunk/lib/Parse/ParseOpenMP.cpp (original)
> +++ cfe/trunk/lib/Parse/ParseOpenMP.cpp Fri Jun 20 04:44:06 2014
> @@ -317,6 +317,16 @@ OMPClause *Parser::ParseOpenMPClause(Ope
>
> Clause = ParseOpenMPSingleExprWithArgClause(CKind);
> break;
> + case OMPC_ordered:
> + // OpenMP [2.7.1, Restrictions, p. 9]
> + // Only one ordered clause can appear on a loop directive.
> + if (!FirstClause) {
> + Diag(Tok, diag::err_omp_more_one_clause) <<
> getOpenMPDirectiveName(DKind)
> + << getOpenMPClauseName
(CKind);
> + }
> +
> + Clause = ParseOpenMPClause(CKind);
> + break;
> case OMPC_private:
> case OMPC_firstprivate:
> case OMPC_lastprivate:
> @@ -409,6 +419,19 @@ OMPClause *Parser::ParseOpenMPSimpleClau
> Tok.getLocation());
> }
>
> +/// \brief Parsing of OpenMP clauses like 'ordered'.
> +///
> +/// ordered-clause:
> +/// 'ordered'
> +///
> +OMPClause *Parser::ParseOpenMPClause(OpenMPClauseKind Kind) {
> + SourceLocation Loc = Tok.getLocation();
> + ConsumeAnyToken();
> +
> + return Actions.ActOnOpenMPClause(Kind, Loc, Tok.getLocation());
> +}
> +
> +
> /// \brief Parsing of OpenMP clauses with single expressions and
> some additional
> /// argument like 'schedule' or 'dist_schedule'.
> ///
>
> Modified: cfe/trunk/lib/Sema/SemaOpenMP.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/
> SemaOpenMP.cpp?rev=211347&r1=211346&r2=211347&view=diff
>
==============================================================================
> --- cfe/trunk/lib/Sema/SemaOpenMP.cpp (original)
> +++ cfe/trunk/lib/Sema/SemaOpenMP.cpp Fri Jun 20 04:44:06 2014
> @@ -1522,6 +1522,7 @@ OMPClause *Sema::ActOnOpenMPSingleExprCl
> case OMPC_linear:
> case OMPC_aligned:
> case OMPC_copyin:
> + case OMPC_ordered:
> case OMPC_threadprivate:
> case OMPC_unknown:
> llvm_unreachable("Clause is not allowed.");
> @@ -1699,6 +1700,7 @@ OMPClause *Sema::ActOnOpenMPSimpleClause
> case OMPC_linear:
> case OMPC_aligned:
> case OMPC_copyin:
> + case OMPC_ordered:
> case OMPC_threadprivate:
> case OMPC_unknown:
> llvm_unreachable("Clause is not allowed.");
> @@ -1807,6 +1809,7 @@ OMPClause *Sema::ActOnOpenMPSingleExprWi
> case OMPC_linear:
> case OMPC_aligned:
> case OMPC_copyin:
> + case OMPC_ordered:
> case OMPC_threadprivate:
> case OMPC_unknown:
> llvm_unreachable("Clause is not allowed.");
> @@ -1870,6 +1873,41 @@ OMPClause *Sema::ActOnOpenMPScheduleClau
> EndLoc, Kind, ValExpr);
> }
>
> +OMPClause *Sema::ActOnOpenMPClause(OpenMPClauseKind Kind,
> + SourceLocation StartLoc,
> + SourceLocation EndLoc) {
> + OMPClause *Res = nullptr;
> + switch (Kind) {
> + case OMPC_ordered:
> + Res = ActOnOpenMPOrderedClause(StartLoc, EndLoc);
> + break;
> + case OMPC_if:
> + case OMPC_num_threads:
> + case OMPC_safelen:
> + case OMPC_collapse:
> + case OMPC_schedule:
> + case OMPC_private:
> + case OMPC_firstprivate:
> + case OMPC_lastprivate:
> + case OMPC_shared:
> + case OMPC_reduction:
> + case OMPC_linear:
> + case OMPC_aligned:
> + case OMPC_copyin:
> + case OMPC_default:
> + case OMPC_proc_bind:
> + case OMPC_threadprivate:
> + case OMPC_unknown:
> + llvm_unreachable("Clause is not allowed.");
> + }
> + return Res;
> +}
> +
> +OMPClause *Sema::ActOnOpenMPOrderedClause(SourceLocation StartLoc,
> + SourceLocation EndLoc) {
> + return new (Context) OMPOrderedClause(StartLoc, EndLoc);
> +}
> +
> OMPClause *Sema::ActOnOpenMPVarListClause(
> OpenMPClauseKind Kind, ArrayRef<Expr *> VarList, Expr *TailExpr,
> SourceLocation StartLoc, SourceLocation LParenLoc,
> SourceLocation ColonLoc,
> @@ -1911,6 +1949,7 @@ OMPClause *Sema::ActOnOpenMPVarListClaus
> case OMPC_default:
> case OMPC_proc_bind:
> case OMPC_schedule:
> + case OMPC_ordered:
> case OMPC_threadprivate:
> case OMPC_unknown:
> llvm_unreachable("Clause is not allowed.");
>
> Modified: cfe/trunk/lib/Sema/TreeTransform.h
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/
> TreeTransform.h?rev=211347&r1=211346&r2=211347&view=diff
>
==============================================================================
> --- cfe/trunk/lib/Sema/TreeTransform.h (original)
> +++ cfe/trunk/lib/Sema/TreeTransform.h Fri Jun 20 04:44:06 2014
> @@ -6500,6 +6500,13 @@ TreeTransform<Derived>::TransformOMPSche
>
> template <typename Derived>
> OMPClause *
> +TreeTransform<Derived>::TransformOMPOrderedClause(OMPOrderedClause *C) {
> + // No need to rebuild this clause, no template-dependent parameters.
> + return C;
> +}
> +
> +template <typename Derived>
> +OMPClause *
> TreeTransform<Derived>::TransformOMPPrivateClause(OMPPrivateClause *C) {
> llvm::SmallVector<Expr *, 16> Vars;
> Vars.reserve(C->varlist_size());
>
> Modified: cfe/trunk/lib/Serialization/ASTReaderStmt.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/
> Serialization/ASTReaderStmt.cpp?rev=211347&r1=211346&r2=211347&view=diff
>
==============================================================================
> --- cfe/trunk/lib/Serialization/ASTReaderStmt.cpp (original)
> +++ cfe/trunk/lib/Serialization/ASTReaderStmt.cpp Fri Jun 20 04:44:06
2014
> @@ -1694,6 +1694,9 @@ OMPClause *OMPClauseReader::readClause()
> case OMPC_schedule:
> C = new (Context) OMPScheduleClause();
> break;
> + case OMPC_ordered:
> + C = new (Context) OMPOrderedClause();
> + break;
> case OMPC_private:
> C = OMPPrivateClause::CreateEmpty(Context, Record[Idx++]);
> break;
> @@ -1769,6 +1772,8 @@ void OMPClauseReader::VisitOMPScheduleCl
> C->setCommaLoc(Reader->ReadSourceLocation(Record, Idx));
> }
>
> +void OMPClauseReader::VisitOMPOrderedClause(OMPOrderedClause *) {}
> +
> void OMPClauseReader::VisitOMPPrivateClause(OMPPrivateClause *C) {
> C->setLParenLoc(Reader->ReadSourceLocation(Record, Idx));
> unsigned NumVars = C->varlist_size();
>
> Modified: cfe/trunk/lib/Serialization/ASTWriterStmt.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/
> Serialization/ASTWriterStmt.cpp?rev=211347&r1=211346&r2=211347&view=diff
>
==============================================================================
> --- cfe/trunk/lib/Serialization/ASTWriterStmt.cpp (original)
> +++ cfe/trunk/lib/Serialization/ASTWriterStmt.cpp Fri Jun 20 04:44:06
2014
> @@ -1715,6 +1715,8 @@ void OMPClauseWriter::VisitOMPScheduleCl
> Writer->Writer.AddSourceLocation(C->getCommaLoc(), Record);
> }
>
> +void OMPClauseWriter::VisitOMPOrderedClause(OMPOrderedClause *) {}
> +
> void OMPClauseWriter::VisitOMPPrivateClause(OMPPrivateClause *C) {
> Record.push_back(C->varlist_size());
> Writer->Writer.AddSourceLocation(C->getLParenLoc(), Record);
>
> Modified: cfe/trunk/test/OpenMP/for_ast_print.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/OpenMP/
> for_ast_print.cpp?rev=211347&r1=211346&r2=211347&view=diff
>
==============================================================================
> --- cfe/trunk/test/OpenMP/for_ast_print.cpp (original)
> +++ cfe/trunk/test/OpenMP/for_ast_print.cpp Fri Jun 20 04:44:06 2014
> @@ -20,12 +20,12 @@ T tmain(T argc) {
> // CHECK-NEXT: for (int i = 0; i < 2; ++i)
> // CHECK-NEXT: a = 2;
> #pragma omp parallel
> -#pragma omp for private(argc, b), firstprivate(c, d), lastprivate
> (d, f) collapse(N) schedule(static, N)
> +#pragma omp for private(argc, b), firstprivate(c, d), lastprivate
> (d, f) collapse(N) schedule(static, N) ordered
> for (int i = 0; i < 10; ++i)
> for (int j = 0; j < 10; ++j)
> foo();
> // CHECK-NEXT: #pragma omp parallel
> - // CHECK-NEXT: #pragma omp for private(argc,b) firstprivate(c,d)
> lastprivate(d,f) collapse(N) schedule(static, N)
> + // CHECK-NEXT: #pragma omp for private(argc,b) firstprivate(c,d)
> lastprivate(d,f) collapse(N) schedule(static, N) ordered
> // CHECK-NEXT: for (int i = 0; i < 10; ++i)
> // CHECK-NEXT: for (int j = 0; j < 10; ++j)
> // CHECK-NEXT: foo();
> @@ -43,12 +43,12 @@ int main(int argc, char **argv) {
> // CHECK-NEXT: for (int i = 0; i < 2; ++i)
> // CHECK-NEXT: a = 2;
> #pragma omp parallel
> -#pragma omp for private(argc, b), firstprivate(argv, c),
> lastprivate(d, f) collapse(2) schedule(auto)
> +#pragma omp for private(argc, b), firstprivate(argv, c),
> lastprivate(d, f) collapse(2) schedule(auto) ordered
> for (int i = 0; i < 10; ++i)
> for (int j = 0; j < 10; ++j)
> foo();
> // CHECK-NEXT: #pragma omp parallel
> - // CHECK-NEXT: #pragma omp for private(argc,b) firstprivate
> (argv,c) lastprivate(d,f) collapse(2) schedule(auto)
> + // CHECK-NEXT: #pragma omp for private(argc,b) firstprivate
> (argv,c) lastprivate(d,f) collapse(2) schedule(auto) ordered
> // CHECK-NEXT: for (int i = 0; i < 10; ++i)
> // CHECK-NEXT: for (int j = 0; j < 10; ++j)
> // CHECK-NEXT: foo();
>
> Modified: cfe/trunk/tools/libclang/CIndex.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/libclang/
> CIndex.cpp?rev=211347&r1=211346&r2=211347&view=diff
>
==============================================================================
> --- cfe/trunk/tools/libclang/CIndex.cpp (original)
> +++ cfe/trunk/tools/libclang/CIndex.cpp Fri Jun 20 04:44:06 2014
> @@ -1949,6 +1949,8 @@ void OMPClauseEnqueue::VisitOMPScheduleC
> Visitor->AddStmt(C->getChunkSize());
> }
>
> +void OMPClauseEnqueue::VisitOMPOrderedClause(const OMPOrderedClause *)
{}
> +
> template<typename T>
> void OMPClauseEnqueue::VisitOMPClauseList(T *Node) {
> for (const auto *I : Node->varlists())
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140718/f3fb502a/attachment.html>
More information about the cfe-commits
mailing list