[PATCH] D154672: [OPENMP52] Deprecation of 'depend' clause in ordered directive.

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman 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>;
 
----------------
jyu2 wrote:
> 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
I'm not certain; it's never worked properly for me in a .td file (but I use a plugin in Visual Studio). Either it does no formatting, or it formats very oddly because it doesn't understand the syntax.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D154672/new/

https://reviews.llvm.org/D154672