This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG852fe30b687e: [OPENMP52] Deprecation of 'depend'
clause in ordered directive. (authored by jyu2).
Changed prior to commit:
https://reviews.llvm.or
aaron.ballman added inline comments.
Comment at: clang/include/clang/Basic/DiagnosticParseKinds.td:1532
InGroup;
+def warn_omp_depend_in_ordered_deprecated : Warning<"denpend clause for
ordered is deprecated; use doacross instaed">, InGroup;
jyu2 wrote:
> a
jyu2 added inline comments.
Comment at: clang/include/clang/Basic/DiagnosticParseKinds.td:1532
InGroup;
+def warn_omp_depend_in_ordered_deprecated : Warning<"denpend clause for
ordered is deprecated; use doacross instaed">, InGroup;
aaron.ballman wrote:
> A
jyu2 updated this revision to Diff 538162.
jyu2 added a comment.
Thanks @jdoerfert, @ABataev and @aaron.ballman for the code reviews. This is
addressing comments.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154672/new/
https://reviews.llvm.org/
aaron.ballman added inline comments.
Comment at: clang/include/clang/Basic/DiagnosticParseKinds.td:1532
InGroup;
+def warn_omp_depend_in_ordered_deprecated : Warning<"denpend clause for
ordered is deprecated; use doacross instaed">, InGroup;
ABataev wrote:
ABataev added inline comments.
Comment at: clang/include/clang/Basic/DiagnosticParseKinds.td:1532
InGroup;
+def warn_omp_depend_in_ordered_deprecated : Warning<"denpend clause for
ordered is deprecated; use doacross instaed">, InGroup;
denpend->depend
insta
jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.
LG
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154672/new/
https://reviews.llvm.org/D154672
___
jyu2 created this revision.
jyu2 added reviewers: ABataev, mikerice, jdoerfert.
jyu2 added a project: OpenMP.
Herald added subscribers: sunshaoce, guansong, yaxunl.
Herald added a project: All.
jyu2 requested review of this revision.
Herald added subscribers: cfe-commits, jplehr, sstefan1.
Herald a