[clang] [clang-tools-extra] [clang-tidy] EndSourceFile() for preprocessor before diagnostic client (PR #145784)

[Dave Bartolomeo via cfe-commits]

================
@@ -360,6 +360,12 @@ getFixIt(const tooling::Diagnostic &Diagnostic, bool 
AnyFix) {
 
 void ClangTidyDiagnosticConsumer::HandleDiagnostic(
     DiagnosticsEngine::Level DiagLevel, const Diagnostic &Info) {
+  assert(InSourceFile ||
+         Info.getLocation()
+             .isInvalid()); // A diagnostic should not be reported outside of a
+                            // BeginSourceFile()/EndSourceFile() pair if it has
+                            // a source location.
----------------
dbartol wrote:

Fixed

https://github.com/llvm/llvm-project/pull/145784
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits