diff options
| author | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2019-11-03 19:29:29 -0800 |
|---|---|---|
| committer | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2019-11-03 19:57:33 -0800 |
| commit | 31e14f41a21f9016050a20f07d5da03db2e8c13e (patch) | |
| tree | 414a5b696961e951b4170658023c6d674756b71b /lldb/source/Core/Value.cpp | |
| parent | 858b15cb9cf675b33d5c3bf17b3050d98b73ec3f (diff) | |
| download | bcm5719-llvm-31e14f41a21f9016050a20f07d5da03db2e8c13e.tar.gz bcm5719-llvm-31e14f41a21f9016050a20f07d5da03db2e8c13e.zip | |
clang/Modules: Sink CompilerInstance::KnownModules into ModuleMap
Avoid use-after-frees when FrontendAction::BeginSourceFile is called
twice on the same CompilerInstance by sinking
CompilerInstance::KnownModules into ModuleMap. On the way, rename the
map to CachedModuleLoads. I considered (but rejected) merging this with
ModuleMap::Modules, since that only has top-level modules and this map
includes submodules.
This is an alternative to https://reviews.llvm.org/D58497. Thanks to
nemanjai for the detailed analysis of the problem!
Diffstat (limited to 'lldb/source/Core/Value.cpp')
0 files changed, 0 insertions, 0 deletions
