[PATCH] D85981: [clang][Modules] Use File Names Instead of inodes As Loaded Module Keys
Adrian Prantl via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Aug 15 18:36:24 PDT 2020
aprantl added inline comments.
================
Comment at: clang/include/clang/Serialization/ModuleManager.h:62
- /// All loaded modules, indexed by name.
- llvm::DenseMap<const FileEntry *, ModuleFile *> Modules;
+ /// All loaded modules, indexed by file name.
+ llvm::StringMap<ModuleFile *> Modules;
----------------
Is it literally the file name, or something like the absolute realpath? And just because I'm curious: Is this the name of the .pcm or of the module map file?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85981/new/
https://reviews.llvm.org/D85981
More information about the cfe-commits
mailing list