[PATCH] D28913: ThinLTOBitcodeWriter: Strip debug info from merged module.

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/ThinLTOBitcodeWriter.cpp:265
   std::unique_ptr<Module> MergedM(CloneModule(&M, VMap, IsInMergedM));
+  StripDebugInfo(*MergedM);
 
----------------
pcc wrote:
> mehdi_amini wrote:
> > Debug info is costly to manipulate, can't we have a flag on CloneModule that just ignores them in the first place?
> Maybe we can do that in a follow up change. For the moment it seems simplest to strip after cloning.
Are you expecting this to be involved? 




https://reviews.llvm.org/D28913