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

Jan Svoboda via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 15 08:46:09 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:

I don't understand how would this impact explicit module invocations? These can still use relative module map paths, it's just that the compiler will use the absolute canonical path internally (IIUC in an non-observable way).

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


More information about the cfe-commits mailing list