Re: [PATCH] D15125: [OPENMP] 'omp distribute' directive basic support.

2015-12-14 Thread Carlo Bertolli via cfe-commits
carlo.bertolli added a comment. Resubmitted at rev255498 Repository: rL LLVM http://reviews.llvm.org/D15125 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D15125: [OPENMP] 'omp distribute' directive basic support.

2015-12-07 Thread Carlo Bertolli via cfe-commits
carlo.bertolli closed this revision. carlo.bertolli added a comment. This was committed in r255001. Many thanks for all your help. Repository: rL LLVM http://reviews.llvm.org/D15125 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D15125: [OPENMP] 'omp distribute' directive basic support.

2015-12-04 Thread Alexey Bataev via cfe-commits
ABataev added inline comments. Comment at: include/clang-c/Index.h:2263-2267 @@ -2262,3 +2262,7 @@ - CXCursor_LastStmt = CXCursor_OMPTaskLoopDirective, + /** \brief OpenMP distribute directive. + */ + CXCursor_OMPDistributeDirective= 259, + +

Re: [PATCH] D15125: [OPENMP] 'omp distribute' directive basic support.

2015-12-04 Thread Carlo Bertolli via cfe-commits
carlo.bertolli updated this revision to Diff 41980. carlo.bertolli marked 7 inline comments as done. carlo.bertolli added a comment. This updated patch includes taskloop simd changes form the trunk and it addresses the second round of comments. Repository: rL LLVM

Re: [PATCH] D15125: [OPENMP] 'omp distribute' directive basic support.

2015-12-04 Thread Carlo Bertolli via cfe-commits
carlo.bertolli added inline comments. Comment at: lib/Basic/OpenMPKinds.cpp:460 @@ -449,4 +459,3 @@ DKind == OMPD_single || DKind == OMPD_parallel_for || - DKind == OMPD_parallel_for_simd || - DKind == OMPD_parallel_sections; // TODO add next directives.

Re: [PATCH] D15125: [OPENMP] 'omp distribute' directive basic support.

2015-12-03 Thread Carlo Bertolli via cfe-commits
carlo.bertolli updated this revision to Diff 41712. carlo.bertolli marked 15 inline comments as done. carlo.bertolli added a comment. Thanks very much for your kind comments and patience at some trivial errors on my side. This new version of the patch addresses all your concerns and it removes

Re: [PATCH] D15125: [OPENMP] 'omp distribute' directive basic support.

2015-12-03 Thread Carlo Bertolli via cfe-commits
carlo.bertolli added inline comments. Comment at: include/clang/AST/OpenMPClause.h:667 @@ -666,3 +666,3 @@ OpenMPScheduleClauseKind Kind; - /// \brief Start location of the schedule ind in source code. + /// \brief Start location of the schedule kind in source code.

Re: [PATCH] D15125: [OPENMP] 'omp distribute' directive basic support.

2015-12-02 Thread Kelvin Li via cfe-commits
kkwli0 added inline comments. Comment at: include/clang/AST/OpenMPClause.h:708 @@ -707,3 +707,3 @@ public: - /// \brief Build 'schedule' clause with schedule kind \a Kind and chunk size - /// expression \a ChunkSize. + /// \brief Build 'dist_schedule' clause with schedule

Re: [PATCH] D15125: [OPENMP] 'omp distribute' directive basic support.

2015-12-02 Thread Alexey Bataev via cfe-commits
ABataev added inline comments. Comment at: include/clang/AST/OpenMPClause.h:667 @@ -666,3 +666,3 @@ OpenMPScheduleClauseKind Kind; - /// \brief Start location of the schedule ind in source code. + /// \brief Start location of the schedule kind in source code.

[PATCH] D15125: [OPENMP] 'omp distribute' directive basic support.

2015-12-01 Thread Carlo Bertolli via cfe-commits
carlo.bertolli created this revision. carlo.bertolli added reviewers: hfinkel, ABataev, rjmccall. carlo.bertolli added subscribers: sfantao, cfe-commits. carlo.bertolli set the repository for this revision to rL LLVM. This patch includes parsing and semantic analysis for 'omp distribute'