aaron.ballman accepted this revision.
aaron.ballman added a comment.

LGTM aside from a minor diagnostic wording nit.



================
Comment at: clang-tidy/openmp/UseDefaultNoneCheck.cpp:52
+                                         Clause->getDefaultKind());
+    diag(Clause->getBeginLoc(), "existing 'default' clause is specified here",
+         DiagnosticIDs::Note);
----------------
drop the "is" to make it match how we usually phrase things.


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D57113



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to