[clang] [clang][modules] Minor improvements to diagnosing `out of date` errors (PR #136612)

[Jan Svoboda via cfe-commits]

================
@@ -3349,9 +3351,14 @@ ASTReader::ReadControlBlock(ModuleFile &F,
                                       .getModuleCache()
                                       .getInMemoryModuleCache()
                                       .isPCMFinal(F.FileName);
-      if (isDiagnosedResult(Result, Capabilities) || recompilingFinalized)
+      if (isDiagnosedResult(Result, Capabilities) || recompilingFinalized) {
         Diag(diag::note_module_file_imported_by)
             << F.FileName << !F.ModuleName.empty() << F.ModuleName;
+        if (UseFilemap && !F.ModuleName.empty() && !StoredFile.empty() &&
+            StoredFile != ImportedFile)
----------------
jansvoboda11 wrote:

Is the string comparison here intentional? It seems like this should be a `FileEntry` comparison instead.

https://github.com/llvm/llvm-project/pull/136612