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

Ben Langmuir via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 14 15:26:39 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);
----------------
benlangmuir wrote:

This would also impact explicit module invocations.  I didn't realize the impact on path hash; good point that also seems like a concern.  I didn't consider ASTWriter would make it relative, so at least that case wouldn't be an issue.

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


More information about the cfe-commits mailing list