[PATCH] D40242: Do not perform the analysis based warning if all warnings are ignored

[Olivier Goffart via Phabricator via cfe-commits]

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


================
Comment at: lib/Sema/AnalysisBasedWarnings.cpp:2084
+  // Do not do any analysis if we are going to just ignore them.
+  if (Diags.getIgnoreAllWarnings() ||
+      (Diags.getSuppressSystemWarnings() &&
----------------
lebedev.ri wrote:
> I guess you can try commenting-out that entire check and seeing whether any test fails.
Commenting out the entire if does not cause any test to fail.


https://reviews.llvm.org/D40242