[PATCH] D27077: LTO: Port the new LTO API to ModuleSymbolTable.

[Peter Collingbourne via Phabricator via llvm-commits]

pcc added inline comments.


================
Comment at: llvm/include/llvm/LTO/LTO.h:84
+  ModuleSymbolTable SymTab;
+  std::unique_ptr<Module> Mod;
+  MemoryBufferRef MBRef;
----------------
mehdi_amini wrote:
> It seems there is not much that still requires us to keep `Mod` here, what is fundamentally the remaining issue? 
The major ones are:
- We need something to own the module
- We need to be able to query the source file name
- We need it to be able to extract `used` bits in `add*LTO`
Essentially we need it to access information that will later be stored in the bitcode symbol table. But until that's ready there's no harm in accessing the module directly.


https://reviews.llvm.org/D27077