[PATCH] D48116: [libclang] Allow skipping warnings from all included files

Ivan Donchevskii via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 13 01:53:33 PDT 2018


yvvan added a comment.

Cool! That's actually quite a small change with big outcome!



================
Comment at: include/clang/Basic/Diagnostic.h:216
 
+  // Suppress warnings from all included files.
+  bool SuppressWarningsFromIncludedFiles = false;
----------------
Probably mention that it also suppresses everything with lower priority? 


================
Comment at: lib/Basic/DiagnosticIDs.cpp:484
+      !Diag.getSourceManager().isInMainFile(
+          Diag.getSourceManager().getExpansionLoc(Loc)))
+    return diag::Severity::Ignored;
----------------
{ } around return


Repository:
  rC Clang

https://reviews.llvm.org/D48116





More information about the cfe-commits mailing list