[clang-tidy] give dummy path when create ClangTidyContext

[HerrCai0907]

#121323 changed the way the absolute path is computed. Empty file name will cause absolute path ignore current folder.
This patch add "dummy" file name to avoid this issue
Fixed: #134502

[llvmbot]

@llvm/pr-subscribers-clang-tidy

@llvm/pr-subscribers-clang-tools-extra

Author: Congcong Cai (HerrCai0907)

Changes

#121323 changed the way the absolute path is computed. Empty file name will cause absolute path ignore current folder.
This patch add "dummy" file name to avoid this issue
Fixed: #134502

---

Full diff: https://github.com/llvm/llvm-project/pull/134670.diff

1 Files Affected:

• (modified) clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.cpp (+2-2)

diff --git a/clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.cpp b/clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.cpp
index 71e852545203e..f0c9881a48838 100644
--- a/clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.cpp
+++ b/clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.cpp
@@ -166,8 +166,8 @@ ClangTidyContext::ClangTidyContext(
       AllowEnablingAnalyzerAlphaCheckers(AllowEnablingAnalyzerAlphaCheckers),
       EnableModuleHeadersParsing(EnableModuleHeadersParsing) {
   // Before the first translation unit we can get errors related to command-line
-  // parsing, use empty string for the file name in this case.
-  setCurrentFile("");
+  // parsing, use dummy string for the file name in this case.
+  setCurrentFile("dummy");
 }
 
 ClangTidyContext::~ClangTidyContext() = default;

[carlosgalvezp]

This should work, thanks! I still feel the whole framework needs a bit of refactoring, the logic and data flow is a bit hidden and it's hard to spot these dependencies that break because things are done in the wrong order.