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

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 23 21:07:43 PST 2016


mehdi_amini added inline comments.


================
Comment at: llvm/include/llvm/LTO/LTO.h:104
   friend LTO;
-  InputFile() = default;
+  InputFile(MemoryBufferRef MBRef) : MBRef(MBRef) {}
 
----------------
Why don't you take all the members here?
It seems strange to see the initialization and then `File->Mod = ...`


https://reviews.llvm.org/D27077





More information about the llvm-commits mailing list