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

[Peter Collingbourne via Phabricator via llvm-commits]

pcc added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/ThinLTOBitcodeWriter.cpp:265
   std::unique_ptr<Module> MergedM(CloneModule(&M, VMap, IsInMergedM));
+  StripDebugInfo(*MergedM);
 
----------------
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.


https://reviews.llvm.org/D28913