Prefer `getCurrentFileOrBufferName` in `FrontendAction::EndSourceFile`

[JohnTitor]

Summary

getCurrentFile here causes an assertion on some condition.
getCurrentFileOrBufferName is preferrable instead.

llvm#55950

[tbaeder]

LGTM but can you try re-running the pre-commit CI?

[tbaeder]

Ping

[JohnTitor]

I've re-triggered it several times but it shows the same failure, maybe they're valid?

[tbaeder]

Those are all libomp tests, I doubt they are related. I think it's fine to just push this.

[JohnTitor]

Alright, let's go ahead then.

clang/lib/Frontend/FrontendAction.cpp

if (DisableFree) {
     llvm::BuryPointer(CI.takeASTConsumer().get());
   } else {
     CI.setSema(nullptr);
     CI.setASTContext(nullptr);
     CI.setASTConsumer(nullptr);
   }
 
   if (CI.getFrontendOpts().ShowStats) {
-    llvm::errs() << "\nSTATISTICS FOR '" << getCurrentFile() << "':\n";
+    llvm::errs() << "\nSTATISTICS FOR '" << getCurrentFileOrBufferName() << "':\n";
     CI.getPreprocessor().PrintStats();
     CI.getPreprocessor().getIdentifierTable().PrintStats();
     CI.getPreprocessor().getHeaderSearchInfo().PrintStats();
     CI.getSourceManager().PrintStats();
     llvm::errs() << "\n";
   }
 
   // Cleanup the output streams, and erase the output files if instructed by the