[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