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

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 27 14:26:43 PST 2017


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:
> 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? 
> 
> 
We would need to thread the flag through to practically everywhere we create IR when cloning a module. Not that involved but certainly more than a one liner. (And the perf improvement associated with adding the flag hasn't been measured.)

I've started looking at the bug Ivan found. In the meantime, it would be nice if we could land this change to unbreak our bot.


https://reviews.llvm.org/D28913





More information about the llvm-commits mailing list