[PATCH] D29025: Revert "IRGen: Start using the WriteThinLTOBitcode pass."

Martin Böhme via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 23 06:41:12 PST 2017


mboehme created this revision.
Herald added subscribers: mgorny, mehdi_amini.

This reverts commit r292662.

This change broke internal builds. Will provide a reproducer internally.


https://reviews.llvm.org/D29025

Files:
  lib/CodeGen/BackendUtil.cpp
  test/CMakeLists.txt
  test/CodeGenCXX/type-metadata-thinlto.cpp


Index: test/CodeGenCXX/type-metadata-thinlto.cpp
===================================================================
--- test/CodeGenCXX/type-metadata-thinlto.cpp
+++ /dev/null
@@ -1,11 +0,0 @@
-// RUN: %clang_cc1 -flto=thin -flto-unit -triple x86_64-unknown-linux -fvisibility hidden -emit-llvm-bc -o %t %s
-// RUN: llvm-modextract -o - -n 1 %t | llvm-dis | FileCheck %s
-
-// CHECK: @_ZTV1A = linkonce_odr
-class A {
-  virtual void f() {}
-};
-
-A *f() {
-  return new A;
-}
Index: test/CMakeLists.txt
===================================================================
--- test/CMakeLists.txt
+++ test/CMakeLists.txt
@@ -80,7 +80,6 @@
     llc
     llvm-bcanalyzer
     llvm-dis
-    llvm-modextract
     llvm-nm
     llvm-objdump
     llvm-profdata
Index: lib/CodeGen/BackendUtil.cpp
===================================================================
--- lib/CodeGen/BackendUtil.cpp
+++ lib/CodeGen/BackendUtil.cpp
@@ -689,11 +689,9 @@
     break;
 
   case Backend_EmitBC:
-    if (CodeGenOpts.EmitSummaryIndex)
-      PerModulePasses.add(createWriteThinLTOBitcodePass(*OS));
-    else
-      PerModulePasses.add(
-          createBitcodeWriterPass(*OS, CodeGenOpts.EmitLLVMUseLists));
+    PerModulePasses.add(createBitcodeWriterPass(
+        *OS, CodeGenOpts.EmitLLVMUseLists, CodeGenOpts.EmitSummaryIndex,
+        CodeGenOpts.EmitSummaryIndex));
     break;
 
   case Backend_EmitLL:


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D29025.85375.patch
Type: text/x-patch
Size: 1396 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170123/cca02dd4/attachment.bin>


More information about the cfe-commits mailing list