[PATCH] D154672: [OPENMP52] Deprecation of 'depend' clause in ordered directive.
Jennifer Yu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 7 08:53:59 PDT 2023
jyu2 added inline comments.
================
Comment at: clang/include/clang/Basic/DiagnosticParseKinds.td:1532
InGroup<OpenMPClauses>;
+def warn_omp_depend_in_ordered_deprecated : Warning<"denpend clause for ordered is deprecated; use doacross instaed">, InGroup<Deprecated>;
----------------
aaron.ballman wrote:
> ABataev wrote:
> > denpend->depend
> > instaed->instead
> Also, wrap to 80 columns and put syntax elements in single quotes. e.g., 'depend' clause for 'ordered'
Thanks Alexey and Aaron for the review!!! Just a question, do you know why git-clang-format does not format .td file now. It was working before.
Thanks.
Jennifer
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154672/new/
https://reviews.llvm.org/D154672
More information about the cfe-commits
mailing list