[PATCH] D79285: [clang-tidy] Add diagnostics level to YAML output

Dmitry Polukhin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 15 05:56:36 PDT 2020


DmitryPolukhin marked an inline comment as done.
DmitryPolukhin added inline comments.


================
Comment at: clang/include/clang/Tooling/DiagnosticsYaml.h:109
+  static void enumeration(IO &IO, clang::tooling::Diagnostic::Level &Value) {
+    IO.enumCase(Value, "Warning", clang::tooling::Diagnostic::Warning);
+    IO.enumCase(Value, "Error", clang::tooling::Diagnostic::Error);
----------------
aaron.ballman wrote:
> Do we have to handle notes as well?
clang::tooling::Diagnostic::Level supports only 2 levels error and warning:
https://github.com/llvm/llvm-project/blob/master/clang/include/clang/Tooling/Core/Diagnostic.h#L65


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79285





More information about the cfe-commits mailing list