================
@@ -44,14 +45,25 @@ void
ClangTidyProfiling::printUserFriendlyTable(llvm::raw_ostream &OS,
void ClangTidyProfiling::printAsJSON(llvm::raw_ostream &OS,
llvm::TimerGroup &TG) {
- assert(Storage && "We should have a filename.");
- OS << "{\n";
- OS << R"("file": ")" << Storage->SourceFilename << "\",\n";
- OS << R"("timestamp": ")" << Storage->Timestamp << "\",\n";
- OS << "\"profile\": {\n";
- TG.printJSONValues(OS, "");
- OS << "\n}\n";
- OS << "}\n";
+ if (!Storage)
+ return;
----------------
zeyi2 wrote:
> IIRC this only happens in release mode, does we get this warning in CI?
https://github.com/llvm/llvm-project/pull/187454#issuecomment-4088570645
In the "edited" history there is a warning for it. But it is also possible that
this is fired because I committed the wrong code (I forgot to include a header
in that commit, somehow code-lint CI still works).
Let me try it again.
https://github.com/llvm/llvm-project/pull/187454
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
