================ @@ -124,10 +164,34 @@ struct ModuleFile { llvm::sys::fs::remove(ModuleFilePath); } + StringRef getModuleName() const { return ModuleName; } + + StringRef getModuleFilePath() const { return ModuleFilePath; } + +private: std::string ModuleName; std::string ModuleFilePath; + + // The required module files. We need to share the ownership for required + // module files. + ReusablePrerequisiteModules RequiredModuleFiles; ---------------- kadircet wrote:
why? since `ModuleFile`s are kept alive by `ReusablePrerequisiteModules`, all of their dependencies should be alive as well, no? https://github.com/llvm/llvm-project/pull/106683 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits