[clang] [clang][deps] Load module map file from PCM (PR #66389)

Jan Svoboda via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 14 10:24:36 PDT 2023


================
@@ -1307,6 +1307,9 @@ void ModuleMap::setInferredModuleAllowedBy(Module *M,
 
 std::error_code
 ModuleMap::canonicalizeModuleMapPath(SmallVectorImpl<char> &Path) {
+  FileManager &FM = SourceMgr.getFileManager();
+  FM.makeAbsolutePath(Path);
----------------
jansvoboda11 wrote:

Are you concerned because this would make the second hash in implicitly-built PCM path depend on CWD? The other place this now gets used is in the `ASTWriter`, but there it gets passed through its `AddPath()`, which makes it relative if needed/requested.

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


More information about the cfe-commits mailing list