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

[Peter Collingbourne via Phabricator via llvm-commits]

pcc created this revision.
Herald added a subscriber: aprantl.

This module will contain nothing but vtable definitions and (soon)
available_externally function definitions, so there is no point in keeping
debug info in the module.


https://reviews.llvm.org/D28913

Files:
  llvm/lib/Transforms/IPO/ThinLTOBitcodeWriter.cpp
  llvm/test/Transforms/ThinLTOBitcodeWriter/split.ll


Index: llvm/test/Transforms/ThinLTOBitcodeWriter/split.ll
===================================================================
--- llvm/test/Transforms/ThinLTOBitcodeWriter/split.ll
+++ llvm/test/Transforms/ThinLTOBitcodeWriter/split.ll
@@ -26,3 +26,10 @@
 
 ; M1: !0 = !{i32 0, !"typeid"}
 !0 = !{i32 0, !"typeid"}
+
+; M0: !llvm.dbg.cu
+; M1-NOT: !llvm.dbg.cu
+!llvm.dbg.cu = !{}
+
+!1 = !{i32 2, !"Debug Info Version", i32 3}
+!llvm.module.flags = !{!1}
Index: llvm/lib/Transforms/IPO/ThinLTOBitcodeWriter.cpp
===================================================================
--- llvm/lib/Transforms/IPO/ThinLTOBitcodeWriter.cpp
+++ llvm/lib/Transforms/IPO/ThinLTOBitcodeWriter.cpp
@@ -19,6 +19,7 @@
 #include "llvm/Analysis/TypeMetadataUtils.h"
 #include "llvm/Bitcode/BitcodeWriter.h"
 #include "llvm/IR/Constants.h"
+#include "llvm/IR/DebugInfo.h"
 #include "llvm/IR/Intrinsics.h"
 #include "llvm/IR/Module.h"
 #include "llvm/IR/PassManager.h"
@@ -261,6 +262,7 @@
 
   ValueToValueMapTy VMap;
   std::unique_ptr<Module> MergedM(CloneModule(&M, VMap, IsInMergedM));
+  StripDebugInfo(*MergedM);
 
   filterModule(&M, [&](const GlobalValue *GV) { return !IsInMergedM(GV); });
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D28913.85015.patch
Type: text/x-patch
Size: 1183 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170119/9bfb77e6/attachment.bin>