diff options
| author | Arpith Chacko Jacob <acjacob@us.ibm.com> | 2016-01-21 19:57:55 +0000 |
|---|---|---|
| committer | Arpith Chacko Jacob <acjacob@us.ibm.com> | 2016-01-21 19:57:55 +0000 |
| commit | 46a04bb5a8f0e73012d67907180b199e064aedee (patch) | |
| tree | c915a5c186dd850459498508147ecf3f097d68d2 /clang/lib/Sema/SemaOpenMP.cpp | |
| parent | ca3326c0d4aecdb37f64b01f03eeacf4d6c258c6 (diff) | |
| download | bcm5719-llvm-46a04bb5a8f0e73012d67907180b199e064aedee.tar.gz bcm5719-llvm-46a04bb5a8f0e73012d67907180b199e064aedee.zip | |
[OpenMP] Check for at least one map clause on target data directive.
Summary:
Adds the following restriction in the OpenMP specifications.
OpenMP [2.10.1, Restrictions, p. 97]
At least one map clause must appear on the directive.
Reviewers: ABataev
Differential Revision: http://reviews.llvm.org/D16341
llvm-svn: 258425
Diffstat (limited to 'clang/lib/Sema/SemaOpenMP.cpp')
| -rw-r--r-- | clang/lib/Sema/SemaOpenMP.cpp | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/clang/lib/Sema/SemaOpenMP.cpp b/clang/lib/Sema/SemaOpenMP.cpp index 3273bd18e16..40c8c18a50d 100644 --- a/clang/lib/Sema/SemaOpenMP.cpp +++ b/clang/lib/Sema/SemaOpenMP.cpp @@ -5618,6 +5618,14 @@ StmtResult Sema::ActOnOpenMPTargetDataDirective(ArrayRef<OMPClause *> Clauses, assert(isa<CapturedStmt>(AStmt) && "Captured statement expected"); + // OpenMP [2.10.1, Restrictions, p. 97] + // At least one map clause must appear on the directive. + if (!HasMapClause(Clauses)) { + Diag(StartLoc, diag::err_omp_no_map_for_directive) << + getOpenMPDirectiveName(OMPD_target_data); + return StmtError(); + } + getCurFunction()->setHasBranchProtectedScope(); return OMPTargetDataDirective::Create(Context, StartLoc, EndLoc, Clauses, |
