[llvm-commits] [llvm] r154356 - /llvm/trunk/tools/lto/LTOCodeGenerator.cpp

[Bill Wendling]

Author: void
Date: Mon Apr  9 18:16:51 2012
New Revision: 154356

URL: http://llvm.org/viewvc/llvm-project?rev=154356&view=rev
Log:
Revert the 'EnableInitializing' flag. There is debate on whether we should run that pass by default in LTO.

Modified:
    llvm/trunk/tools/lto/LTOCodeGenerator.cpp

Modified: llvm/trunk/tools/lto/LTOCodeGenerator.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/lto/LTOCodeGenerator.cpp?rev=154356&r1=154355&r2=154356&view=diff
==============================================================================
--- llvm/trunk/tools/lto/LTOCodeGenerator.cpp (original)
+++ llvm/trunk/tools/lto/LTOCodeGenerator.cpp Mon Apr  9 18:16:51 2012
@@ -46,9 +46,6 @@
 #include "llvm/ADT/StringExtras.h"
 using namespace llvm;
 
-static cl::opt<bool> EnableInternalizing("enable-internalizing", cl::init(false),
-  cl::desc("Internalize functions during LTO"));
-
 static cl::opt<bool> DisableInline("disable-inlining", cl::init(false),
   cl::desc("Do not run the inliner pass"));
 
@@ -278,14 +275,6 @@
 }
 
 void LTOCodeGenerator::applyScopeRestrictions() {
-  // Internalize only if specifically asked for. Otherwise, global symbols which
-  // exist in the final image, but which are used outside of that image
-  // (e.g. bundling) may be removed. This also happens when a function is used
-  // only in inline asm. LLVM doesn't recognize that as a "use", so it could be
-  // stripped.
-  if (!EnableInternalizing)
-    return;
-
   if (_scopeRestrictionsDone) return;
   Module *mergedModule = _linker.getModule();