[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