[PATCH] D18994: [LTO] Switch Module to std::unique_ptr<>

[Davide Italiano via llvm-commits]

davide added inline comments.

================
Comment at: ELF/LTO.h:43
@@ +42,3 @@
+  BitcodeCompiler()
+      : Combined(new llvm::Module("blah", Context)), Mover(*Combined.get()){};
+
----------------
ruiu wrote:
> blah -> ld-temp.o
> 
> *Combined.get() can be *Combined?
> 
> Does Combined owns the newly created llvm::Module?
I think so.


http://reviews.llvm.org/D18994