2002-04-28 13:43:27 +08:00
|
|
|
//===-- Internalize.cpp - Mark functions internal -------------------------===//
|
2005-04-22 07:48:37 +08:00
|
|
|
//
|
2003-10-21 03:43:21 +08:00
|
|
|
// The LLVM Compiler Infrastructure
|
|
|
|
//
|
2007-12-30 04:36:04 +08:00
|
|
|
// This file is distributed under the University of Illinois Open Source
|
|
|
|
// License. See LICENSE.TXT for details.
|
2005-04-22 07:48:37 +08:00
|
|
|
//
|
2003-10-21 03:43:21 +08:00
|
|
|
//===----------------------------------------------------------------------===//
|
2002-04-28 13:43:27 +08:00
|
|
|
//
|
2012-10-27 02:47:48 +08:00
|
|
|
// This pass loops over all of the functions and variables in the input module.
|
|
|
|
// If the function or variable is not in the list of external names given to
|
|
|
|
// the pass it is marked as internal.
|
2002-04-28 13:43:27 +08:00
|
|
|
//
|
2013-11-01 04:51:58 +08:00
|
|
|
// This transformation would not be legal in a regular compilation, but it gets
|
|
|
|
// extra information from the linker about what is safe.
|
2013-10-22 01:14:55 +08:00
|
|
|
//
|
2013-11-01 04:51:58 +08:00
|
|
|
// For example: Internalizing a function with external linkage. Only if we are
|
|
|
|
// told it is only used from within this module, it is safe to do it.
|
2013-10-22 01:14:55 +08:00
|
|
|
//
|
2002-04-28 13:43:27 +08:00
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
|
2006-12-20 06:09:18 +08:00
|
|
|
#define DEBUG_TYPE "internalize"
|
2002-07-25 01:12:05 +08:00
|
|
|
#include "llvm/Transforms/IPO.h"
|
2013-07-25 11:23:25 +08:00
|
|
|
#include "llvm/ADT/SmallPtrSet.h"
|
2012-12-04 00:50:05 +08:00
|
|
|
#include "llvm/ADT/Statistic.h"
|
|
|
|
#include "llvm/Analysis/CallGraph.h"
|
2013-01-02 19:36:10 +08:00
|
|
|
#include "llvm/IR/Module.h"
|
2012-12-04 00:50:05 +08:00
|
|
|
#include "llvm/Pass.h"
|
2004-09-02 06:55:40 +08:00
|
|
|
#include "llvm/Support/CommandLine.h"
|
|
|
|
#include "llvm/Support/Debug.h"
|
2009-07-25 08:23:56 +08:00
|
|
|
#include "llvm/Support/raw_ostream.h"
|
2013-10-22 01:14:55 +08:00
|
|
|
#include "llvm/Transforms/Utils/GlobalStatus.h"
|
2013-07-25 11:23:25 +08:00
|
|
|
#include "llvm/Transforms/Utils/ModuleUtils.h"
|
2003-05-23 03:34:49 +08:00
|
|
|
#include <fstream>
|
|
|
|
#include <set>
|
2003-11-22 05:54:22 +08:00
|
|
|
using namespace llvm;
|
2003-11-12 06:41:34 +08:00
|
|
|
|
2009-01-06 05:24:45 +08:00
|
|
|
STATISTIC(NumAliases , "Number of aliases internalized");
|
2006-12-20 06:09:18 +08:00
|
|
|
STATISTIC(NumFunctions, "Number of functions internalized");
|
|
|
|
STATISTIC(NumGlobals , "Number of global vars internalized");
|
|
|
|
|
2008-05-13 08:00:25 +08:00
|
|
|
// APIFile - A file which contains a list of symbols that should not be marked
|
|
|
|
// external.
|
|
|
|
static cl::opt<std::string>
|
|
|
|
APIFile("internalize-public-api-file", cl::value_desc("filename"),
|
|
|
|
cl::desc("A file containing list of symbol names to preserve"));
|
2003-05-23 03:48:00 +08:00
|
|
|
|
2008-05-13 08:00:25 +08:00
|
|
|
// APIList - A list of symbols that should not be marked internal.
|
|
|
|
static cl::list<std::string>
|
|
|
|
APIList("internalize-public-api-list", cl::value_desc("list"),
|
|
|
|
cl::desc("A list of symbol names to preserve"),
|
|
|
|
cl::CommaSeparated);
|
2005-04-22 07:48:37 +08:00
|
|
|
|
2008-05-13 08:00:25 +08:00
|
|
|
namespace {
|
2009-10-25 14:33:48 +08:00
|
|
|
class InternalizePass : public ModulePass {
|
2003-05-23 03:34:49 +08:00
|
|
|
std::set<std::string> ExternalNames;
|
Reapply "LTO: add API to set strategy for -internalize"
Reapply r199191, reverted in r199197 because it carelessly broke
Other/link-opts.ll. The problem was that calling
createInternalizePass("main") would select
createInternalizePass(bool("main")) instead of
createInternalizePass(ArrayRef<const char *>("main")). This commit
fixes the bug.
The original commit message follows.
Add API to LTOCodeGenerator to specify a strategy for the -internalize
pass.
This is a new attempt at Bill's change in r185882, which he reverted in
r188029 due to problems with the gold linker. This puts the onus on the
linker to decide whether (and what) to internalize.
In particular, running internalize before outputting an object file may
change a 'weak' symbol into an internal one, even though that symbol
could be needed by an external object file --- e.g., with arclite.
This patch enables three strategies:
- LTO_INTERNALIZE_FULL: the default (and the old behaviour).
- LTO_INTERNALIZE_NONE: skip -internalize.
- LTO_INTERNALIZE_HIDDEN: only -internalize symbols with hidden
visibility.
LTO_INTERNALIZE_FULL should be used when linking an executable.
Outputting an object file (e.g., via ld -r) is more complicated, and
depends on whether hidden symbols should be internalized. E.g., for
ld -r, LTO_INTERNALIZE_NONE can be used when -keep_private_externs, and
LTO_INTERNALIZE_HIDDEN can be used otherwise. However,
LTO_INTERNALIZE_FULL is inappropriate, since the output object file will
eventually need to link with others.
lto_codegen_set_internalize_strategy() sets the strategy for subsequent
calls to lto_codegen_write_merged_modules() and lto_codegen_compile*().
<rdar://problem/14334895>
llvm-svn: 199244
2014-01-15 02:52:17 +08:00
|
|
|
bool OnlyHidden;
|
2003-05-23 03:34:49 +08:00
|
|
|
public:
|
2007-05-06 21:37:16 +08:00
|
|
|
static char ID; // Pass identification, replacement for typeid
|
Reapply "LTO: add API to set strategy for -internalize"
Reapply r199191, reverted in r199197 because it carelessly broke
Other/link-opts.ll. The problem was that calling
createInternalizePass("main") would select
createInternalizePass(bool("main")) instead of
createInternalizePass(ArrayRef<const char *>("main")). This commit
fixes the bug.
The original commit message follows.
Add API to LTOCodeGenerator to specify a strategy for the -internalize
pass.
This is a new attempt at Bill's change in r185882, which he reverted in
r188029 due to problems with the gold linker. This puts the onus on the
linker to decide whether (and what) to internalize.
In particular, running internalize before outputting an object file may
change a 'weak' symbol into an internal one, even though that symbol
could be needed by an external object file --- e.g., with arclite.
This patch enables three strategies:
- LTO_INTERNALIZE_FULL: the default (and the old behaviour).
- LTO_INTERNALIZE_NONE: skip -internalize.
- LTO_INTERNALIZE_HIDDEN: only -internalize symbols with hidden
visibility.
LTO_INTERNALIZE_FULL should be used when linking an executable.
Outputting an object file (e.g., via ld -r) is more complicated, and
depends on whether hidden symbols should be internalized. E.g., for
ld -r, LTO_INTERNALIZE_NONE can be used when -keep_private_externs, and
LTO_INTERNALIZE_HIDDEN can be used otherwise. However,
LTO_INTERNALIZE_FULL is inappropriate, since the output object file will
eventually need to link with others.
lto_codegen_set_internalize_strategy() sets the strategy for subsequent
calls to lto_codegen_write_merged_modules() and lto_codegen_compile*().
<rdar://problem/14334895>
llvm-svn: 199244
2014-01-15 02:52:17 +08:00
|
|
|
explicit InternalizePass(bool OnlyHidden = false);
|
|
|
|
explicit InternalizePass(ArrayRef<const char *> ExportList, bool OnlyHidden);
|
2006-01-04 03:13:17 +08:00
|
|
|
void LoadFile(const char *Filename);
|
|
|
|
virtual bool runOnModule(Module &M);
|
2008-10-01 06:04:30 +08:00
|
|
|
|
|
|
|
virtual void getAnalysisUsage(AnalysisUsage &AU) const {
|
|
|
|
AU.setPreservesCFG();
|
2013-11-26 12:19:30 +08:00
|
|
|
AU.addPreserved<CallGraphWrapperPass>();
|
2008-10-01 06:04:30 +08:00
|
|
|
}
|
2006-01-04 03:13:17 +08:00
|
|
|
};
|
|
|
|
} // end anonymous namespace
|
2002-11-09 04:34:21 +08:00
|
|
|
|
2008-05-13 08:00:25 +08:00
|
|
|
char InternalizePass::ID = 0;
|
2010-07-22 06:09:45 +08:00
|
|
|
INITIALIZE_PASS(InternalizePass, "internalize",
|
2010-10-08 06:25:06 +08:00
|
|
|
"Internalize Global Symbols", false, false)
|
2008-05-13 08:00:25 +08:00
|
|
|
|
Reapply "LTO: add API to set strategy for -internalize"
Reapply r199191, reverted in r199197 because it carelessly broke
Other/link-opts.ll. The problem was that calling
createInternalizePass("main") would select
createInternalizePass(bool("main")) instead of
createInternalizePass(ArrayRef<const char *>("main")). This commit
fixes the bug.
The original commit message follows.
Add API to LTOCodeGenerator to specify a strategy for the -internalize
pass.
This is a new attempt at Bill's change in r185882, which he reverted in
r188029 due to problems with the gold linker. This puts the onus on the
linker to decide whether (and what) to internalize.
In particular, running internalize before outputting an object file may
change a 'weak' symbol into an internal one, even though that symbol
could be needed by an external object file --- e.g., with arclite.
This patch enables three strategies:
- LTO_INTERNALIZE_FULL: the default (and the old behaviour).
- LTO_INTERNALIZE_NONE: skip -internalize.
- LTO_INTERNALIZE_HIDDEN: only -internalize symbols with hidden
visibility.
LTO_INTERNALIZE_FULL should be used when linking an executable.
Outputting an object file (e.g., via ld -r) is more complicated, and
depends on whether hidden symbols should be internalized. E.g., for
ld -r, LTO_INTERNALIZE_NONE can be used when -keep_private_externs, and
LTO_INTERNALIZE_HIDDEN can be used otherwise. However,
LTO_INTERNALIZE_FULL is inappropriate, since the output object file will
eventually need to link with others.
lto_codegen_set_internalize_strategy() sets the strategy for subsequent
calls to lto_codegen_write_merged_modules() and lto_codegen_compile*().
<rdar://problem/14334895>
llvm-svn: 199244
2014-01-15 02:52:17 +08:00
|
|
|
InternalizePass::InternalizePass(bool OnlyHidden)
|
|
|
|
: ModulePass(ID), OnlyHidden(OnlyHidden) {
|
2010-10-20 01:21:58 +08:00
|
|
|
initializeInternalizePassPass(*PassRegistry::getPassRegistry());
|
2008-05-15 04:01:01 +08:00
|
|
|
if (!APIFile.empty()) // If a filename is specified, use it.
|
2006-01-04 03:13:17 +08:00
|
|
|
LoadFile(APIFile.c_str());
|
2013-09-05 02:53:21 +08:00
|
|
|
ExternalNames.insert(APIList.begin(), APIList.end());
|
2006-01-04 03:13:17 +08:00
|
|
|
}
|
2003-05-23 03:34:49 +08:00
|
|
|
|
Reapply "LTO: add API to set strategy for -internalize"
Reapply r199191, reverted in r199197 because it carelessly broke
Other/link-opts.ll. The problem was that calling
createInternalizePass("main") would select
createInternalizePass(bool("main")) instead of
createInternalizePass(ArrayRef<const char *>("main")). This commit
fixes the bug.
The original commit message follows.
Add API to LTOCodeGenerator to specify a strategy for the -internalize
pass.
This is a new attempt at Bill's change in r185882, which he reverted in
r188029 due to problems with the gold linker. This puts the onus on the
linker to decide whether (and what) to internalize.
In particular, running internalize before outputting an object file may
change a 'weak' symbol into an internal one, even though that symbol
could be needed by an external object file --- e.g., with arclite.
This patch enables three strategies:
- LTO_INTERNALIZE_FULL: the default (and the old behaviour).
- LTO_INTERNALIZE_NONE: skip -internalize.
- LTO_INTERNALIZE_HIDDEN: only -internalize symbols with hidden
visibility.
LTO_INTERNALIZE_FULL should be used when linking an executable.
Outputting an object file (e.g., via ld -r) is more complicated, and
depends on whether hidden symbols should be internalized. E.g., for
ld -r, LTO_INTERNALIZE_NONE can be used when -keep_private_externs, and
LTO_INTERNALIZE_HIDDEN can be used otherwise. However,
LTO_INTERNALIZE_FULL is inappropriate, since the output object file will
eventually need to link with others.
lto_codegen_set_internalize_strategy() sets the strategy for subsequent
calls to lto_codegen_write_merged_modules() and lto_codegen_compile*().
<rdar://problem/14334895>
llvm-svn: 199244
2014-01-15 02:52:17 +08:00
|
|
|
InternalizePass::InternalizePass(ArrayRef<const char *> ExportList,
|
|
|
|
bool OnlyHidden)
|
|
|
|
: ModulePass(ID), OnlyHidden(OnlyHidden) {
|
2010-10-20 01:21:58 +08:00
|
|
|
initializeInternalizePassPass(*PassRegistry::getPassRegistry());
|
2013-09-05 04:08:46 +08:00
|
|
|
for(ArrayRef<const char *>::const_iterator itr = ExportList.begin();
|
|
|
|
itr != ExportList.end(); itr++) {
|
2006-07-21 01:48:05 +08:00
|
|
|
ExternalNames.insert(*itr);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2006-01-04 03:13:17 +08:00
|
|
|
void InternalizePass::LoadFile(const char *Filename) {
|
|
|
|
// Load the APIFile...
|
|
|
|
std::ifstream In(Filename);
|
|
|
|
if (!In.good()) {
|
2009-08-23 15:05:07 +08:00
|
|
|
errs() << "WARNING: Internalize couldn't load file '" << Filename
|
2008-05-15 04:01:01 +08:00
|
|
|
<< "'! Continuing as if it's empty.\n";
|
|
|
|
return; // Just continue as if the file were empty
|
2006-01-04 03:13:17 +08:00
|
|
|
}
|
|
|
|
while (In) {
|
|
|
|
std::string Symbol;
|
|
|
|
In >> Symbol;
|
|
|
|
if (!Symbol.empty())
|
|
|
|
ExternalNames.insert(Symbol);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2013-09-05 02:37:36 +08:00
|
|
|
static bool shouldInternalize(const GlobalValue &GV,
|
Reapply "LTO: add API to set strategy for -internalize"
Reapply r199191, reverted in r199197 because it carelessly broke
Other/link-opts.ll. The problem was that calling
createInternalizePass("main") would select
createInternalizePass(bool("main")) instead of
createInternalizePass(ArrayRef<const char *>("main")). This commit
fixes the bug.
The original commit message follows.
Add API to LTOCodeGenerator to specify a strategy for the -internalize
pass.
This is a new attempt at Bill's change in r185882, which he reverted in
r188029 due to problems with the gold linker. This puts the onus on the
linker to decide whether (and what) to internalize.
In particular, running internalize before outputting an object file may
change a 'weak' symbol into an internal one, even though that symbol
could be needed by an external object file --- e.g., with arclite.
This patch enables three strategies:
- LTO_INTERNALIZE_FULL: the default (and the old behaviour).
- LTO_INTERNALIZE_NONE: skip -internalize.
- LTO_INTERNALIZE_HIDDEN: only -internalize symbols with hidden
visibility.
LTO_INTERNALIZE_FULL should be used when linking an executable.
Outputting an object file (e.g., via ld -r) is more complicated, and
depends on whether hidden symbols should be internalized. E.g., for
ld -r, LTO_INTERNALIZE_NONE can be used when -keep_private_externs, and
LTO_INTERNALIZE_HIDDEN can be used otherwise. However,
LTO_INTERNALIZE_FULL is inappropriate, since the output object file will
eventually need to link with others.
lto_codegen_set_internalize_strategy() sets the strategy for subsequent
calls to lto_codegen_write_merged_modules() and lto_codegen_compile*().
<rdar://problem/14334895>
llvm-svn: 199244
2014-01-15 02:52:17 +08:00
|
|
|
const std::set<std::string> &ExternalNames,
|
|
|
|
bool OnlyHidden) {
|
|
|
|
if (OnlyHidden && !GV.hasHiddenVisibility())
|
|
|
|
return false;
|
|
|
|
|
2013-09-05 02:37:36 +08:00
|
|
|
// Function must be defined here
|
|
|
|
if (GV.isDeclaration())
|
|
|
|
return false;
|
|
|
|
|
|
|
|
// Available externally is really just a "declaration with a body".
|
|
|
|
if (GV.hasAvailableExternallyLinkage())
|
|
|
|
return false;
|
|
|
|
|
2013-12-04 02:05:14 +08:00
|
|
|
// Assume that dllexported symbols are referenced elsewhere
|
2014-01-14 23:22:47 +08:00
|
|
|
if (GV.hasDLLExportStorageClass())
|
2013-12-04 02:05:14 +08:00
|
|
|
return false;
|
|
|
|
|
2013-09-05 02:37:36 +08:00
|
|
|
// Already has internal linkage
|
|
|
|
if (GV.hasLocalLinkage())
|
|
|
|
return false;
|
|
|
|
|
|
|
|
// Marked to keep external?
|
|
|
|
if (ExternalNames.count(GV.getName()))
|
|
|
|
return false;
|
|
|
|
|
2013-11-01 04:51:58 +08:00
|
|
|
return true;
|
2013-09-05 02:37:36 +08:00
|
|
|
}
|
|
|
|
|
2006-01-04 03:13:17 +08:00
|
|
|
bool InternalizePass::runOnModule(Module &M) {
|
2013-11-26 12:19:30 +08:00
|
|
|
CallGraphWrapperPass *CGPass = getAnalysisIfAvailable<CallGraphWrapperPass>();
|
|
|
|
CallGraph *CG = CGPass ? &CGPass->getCallGraph() : 0;
|
2008-10-03 15:36:09 +08:00
|
|
|
CallGraphNode *ExternalNode = CG ? CG->getExternalCallingNode() : 0;
|
2006-01-04 03:13:17 +08:00
|
|
|
bool Changed = false;
|
2009-01-06 04:38:27 +08:00
|
|
|
|
2013-07-25 11:23:25 +08:00
|
|
|
SmallPtrSet<GlobalValue *, 8> Used;
|
|
|
|
collectUsedGlobalVariables(M, Used, false);
|
|
|
|
|
|
|
|
// We must assume that globals in llvm.used have a reference that not even
|
|
|
|
// the linker can see, so we don't internalize them.
|
|
|
|
// For llvm.compiler.used the situation is a bit fuzzy. The assembler and
|
|
|
|
// linker can drop those symbols. If this pass is running as part of LTO,
|
|
|
|
// one might think that it could just drop llvm.compiler.used. The problem
|
|
|
|
// is that even in LTO llvm doesn't see every reference. For example,
|
|
|
|
// we don't see references from function local inline assembly. To be
|
|
|
|
// conservative, we internalize symbols in llvm.compiler.used, but we
|
|
|
|
// keep llvm.compiler.used so that the symbol is not deleted by llvm.
|
|
|
|
for (SmallPtrSet<GlobalValue *, 8>::iterator I = Used.begin(), E = Used.end();
|
|
|
|
I != E; ++I) {
|
|
|
|
GlobalValue *V = *I;
|
|
|
|
ExternalNames.insert(V->getName());
|
|
|
|
}
|
|
|
|
|
2008-05-15 04:01:01 +08:00
|
|
|
// Mark all functions not in the api as internal.
|
2013-09-05 02:37:36 +08:00
|
|
|
for (Module::iterator I = M.begin(), E = M.end(); I != E; ++I) {
|
Reapply "LTO: add API to set strategy for -internalize"
Reapply r199191, reverted in r199197 because it carelessly broke
Other/link-opts.ll. The problem was that calling
createInternalizePass("main") would select
createInternalizePass(bool("main")) instead of
createInternalizePass(ArrayRef<const char *>("main")). This commit
fixes the bug.
The original commit message follows.
Add API to LTOCodeGenerator to specify a strategy for the -internalize
pass.
This is a new attempt at Bill's change in r185882, which he reverted in
r188029 due to problems with the gold linker. This puts the onus on the
linker to decide whether (and what) to internalize.
In particular, running internalize before outputting an object file may
change a 'weak' symbol into an internal one, even though that symbol
could be needed by an external object file --- e.g., with arclite.
This patch enables three strategies:
- LTO_INTERNALIZE_FULL: the default (and the old behaviour).
- LTO_INTERNALIZE_NONE: skip -internalize.
- LTO_INTERNALIZE_HIDDEN: only -internalize symbols with hidden
visibility.
LTO_INTERNALIZE_FULL should be used when linking an executable.
Outputting an object file (e.g., via ld -r) is more complicated, and
depends on whether hidden symbols should be internalized. E.g., for
ld -r, LTO_INTERNALIZE_NONE can be used when -keep_private_externs, and
LTO_INTERNALIZE_HIDDEN can be used otherwise. However,
LTO_INTERNALIZE_FULL is inappropriate, since the output object file will
eventually need to link with others.
lto_codegen_set_internalize_strategy() sets the strategy for subsequent
calls to lto_codegen_write_merged_modules() and lto_codegen_compile*().
<rdar://problem/14334895>
llvm-svn: 199244
2014-01-15 02:52:17 +08:00
|
|
|
if (!shouldInternalize(*I, ExternalNames, OnlyHidden))
|
2013-09-05 02:37:36 +08:00
|
|
|
continue;
|
|
|
|
|
|
|
|
I->setLinkage(GlobalValue::InternalLinkage);
|
|
|
|
|
|
|
|
if (ExternalNode)
|
|
|
|
// Remove a callgraph edge from the external node to this function.
|
|
|
|
ExternalNode->removeOneAbstractEdgeTo((*CG)[I]);
|
|
|
|
|
|
|
|
Changed = true;
|
|
|
|
++NumFunctions;
|
|
|
|
DEBUG(dbgs() << "Internalizing func " << I->getName() << "\n");
|
|
|
|
}
|
2009-01-06 04:38:27 +08:00
|
|
|
|
2006-01-04 03:13:17 +08:00
|
|
|
// Never internalize the llvm.used symbol. It is used to implement
|
|
|
|
// attribute((used)).
|
2009-07-20 14:14:25 +08:00
|
|
|
// FIXME: Shouldn't this just filter on llvm.metadata section??
|
2006-01-04 03:13:17 +08:00
|
|
|
ExternalNames.insert("llvm.used");
|
2009-07-20 14:14:25 +08:00
|
|
|
ExternalNames.insert("llvm.compiler.used");
|
2009-01-06 04:38:27 +08:00
|
|
|
|
2007-01-27 05:22:28 +08:00
|
|
|
// Never internalize anchors used by the machine module info, else the info
|
|
|
|
// won't find them. (see MachineModuleInfo.)
|
2007-06-07 04:51:41 +08:00
|
|
|
ExternalNames.insert("llvm.global_ctors");
|
|
|
|
ExternalNames.insert("llvm.global_dtors");
|
2007-10-04 01:05:40 +08:00
|
|
|
ExternalNames.insert("llvm.global.annotations");
|
2009-01-06 04:38:27 +08:00
|
|
|
|
2012-04-13 09:06:27 +08:00
|
|
|
// Never internalize symbols code-gen inserts.
|
2013-08-13 04:09:37 +08:00
|
|
|
// FIXME: We should probably add this (and the __stack_chk_guard) via some
|
|
|
|
// type of call-back in CodeGen.
|
|
|
|
ExternalNames.insert("__stack_chk_fail");
|
2012-04-13 09:06:27 +08:00
|
|
|
ExternalNames.insert("__stack_chk_guard");
|
|
|
|
|
2008-05-15 04:01:01 +08:00
|
|
|
// Mark all global variables with initializers that are not in the api as
|
|
|
|
// internal as well.
|
2006-01-04 03:13:17 +08:00
|
|
|
for (Module::global_iterator I = M.global_begin(), E = M.global_end();
|
2013-09-05 02:37:36 +08:00
|
|
|
I != E; ++I) {
|
Reapply "LTO: add API to set strategy for -internalize"
Reapply r199191, reverted in r199197 because it carelessly broke
Other/link-opts.ll. The problem was that calling
createInternalizePass("main") would select
createInternalizePass(bool("main")) instead of
createInternalizePass(ArrayRef<const char *>("main")). This commit
fixes the bug.
The original commit message follows.
Add API to LTOCodeGenerator to specify a strategy for the -internalize
pass.
This is a new attempt at Bill's change in r185882, which he reverted in
r188029 due to problems with the gold linker. This puts the onus on the
linker to decide whether (and what) to internalize.
In particular, running internalize before outputting an object file may
change a 'weak' symbol into an internal one, even though that symbol
could be needed by an external object file --- e.g., with arclite.
This patch enables three strategies:
- LTO_INTERNALIZE_FULL: the default (and the old behaviour).
- LTO_INTERNALIZE_NONE: skip -internalize.
- LTO_INTERNALIZE_HIDDEN: only -internalize symbols with hidden
visibility.
LTO_INTERNALIZE_FULL should be used when linking an executable.
Outputting an object file (e.g., via ld -r) is more complicated, and
depends on whether hidden symbols should be internalized. E.g., for
ld -r, LTO_INTERNALIZE_NONE can be used when -keep_private_externs, and
LTO_INTERNALIZE_HIDDEN can be used otherwise. However,
LTO_INTERNALIZE_FULL is inappropriate, since the output object file will
eventually need to link with others.
lto_codegen_set_internalize_strategy() sets the strategy for subsequent
calls to lto_codegen_write_merged_modules() and lto_codegen_compile*().
<rdar://problem/14334895>
llvm-svn: 199244
2014-01-15 02:52:17 +08:00
|
|
|
if (!shouldInternalize(*I, ExternalNames, OnlyHidden))
|
2013-09-05 02:37:36 +08:00
|
|
|
continue;
|
|
|
|
|
|
|
|
I->setLinkage(GlobalValue::InternalLinkage);
|
|
|
|
Changed = true;
|
|
|
|
++NumGlobals;
|
|
|
|
DEBUG(dbgs() << "Internalized gvar " << I->getName() << "\n");
|
|
|
|
}
|
2009-01-06 04:38:27 +08:00
|
|
|
|
2009-01-06 05:24:45 +08:00
|
|
|
// Mark all aliases that are not in the api as internal as well.
|
|
|
|
for (Module::alias_iterator I = M.alias_begin(), E = M.alias_end();
|
2013-09-05 02:37:36 +08:00
|
|
|
I != E; ++I) {
|
Reapply "LTO: add API to set strategy for -internalize"
Reapply r199191, reverted in r199197 because it carelessly broke
Other/link-opts.ll. The problem was that calling
createInternalizePass("main") would select
createInternalizePass(bool("main")) instead of
createInternalizePass(ArrayRef<const char *>("main")). This commit
fixes the bug.
The original commit message follows.
Add API to LTOCodeGenerator to specify a strategy for the -internalize
pass.
This is a new attempt at Bill's change in r185882, which he reverted in
r188029 due to problems with the gold linker. This puts the onus on the
linker to decide whether (and what) to internalize.
In particular, running internalize before outputting an object file may
change a 'weak' symbol into an internal one, even though that symbol
could be needed by an external object file --- e.g., with arclite.
This patch enables three strategies:
- LTO_INTERNALIZE_FULL: the default (and the old behaviour).
- LTO_INTERNALIZE_NONE: skip -internalize.
- LTO_INTERNALIZE_HIDDEN: only -internalize symbols with hidden
visibility.
LTO_INTERNALIZE_FULL should be used when linking an executable.
Outputting an object file (e.g., via ld -r) is more complicated, and
depends on whether hidden symbols should be internalized. E.g., for
ld -r, LTO_INTERNALIZE_NONE can be used when -keep_private_externs, and
LTO_INTERNALIZE_HIDDEN can be used otherwise. However,
LTO_INTERNALIZE_FULL is inappropriate, since the output object file will
eventually need to link with others.
lto_codegen_set_internalize_strategy() sets the strategy for subsequent
calls to lto_codegen_write_merged_modules() and lto_codegen_compile*().
<rdar://problem/14334895>
llvm-svn: 199244
2014-01-15 02:52:17 +08:00
|
|
|
if (!shouldInternalize(*I, ExternalNames, OnlyHidden))
|
2013-09-05 02:37:36 +08:00
|
|
|
continue;
|
|
|
|
|
|
|
|
I->setLinkage(GlobalValue::InternalLinkage);
|
|
|
|
Changed = true;
|
|
|
|
++NumAliases;
|
|
|
|
DEBUG(dbgs() << "Internalized alias " << I->getName() << "\n");
|
|
|
|
}
|
2009-01-06 05:24:45 +08:00
|
|
|
|
2006-01-04 03:13:17 +08:00
|
|
|
return Changed;
|
|
|
|
}
|
2002-07-24 02:06:35 +08:00
|
|
|
|
Reapply "LTO: add API to set strategy for -internalize"
Reapply r199191, reverted in r199197 because it carelessly broke
Other/link-opts.ll. The problem was that calling
createInternalizePass("main") would select
createInternalizePass(bool("main")) instead of
createInternalizePass(ArrayRef<const char *>("main")). This commit
fixes the bug.
The original commit message follows.
Add API to LTOCodeGenerator to specify a strategy for the -internalize
pass.
This is a new attempt at Bill's change in r185882, which he reverted in
r188029 due to problems with the gold linker. This puts the onus on the
linker to decide whether (and what) to internalize.
In particular, running internalize before outputting an object file may
change a 'weak' symbol into an internal one, even though that symbol
could be needed by an external object file --- e.g., with arclite.
This patch enables three strategies:
- LTO_INTERNALIZE_FULL: the default (and the old behaviour).
- LTO_INTERNALIZE_NONE: skip -internalize.
- LTO_INTERNALIZE_HIDDEN: only -internalize symbols with hidden
visibility.
LTO_INTERNALIZE_FULL should be used when linking an executable.
Outputting an object file (e.g., via ld -r) is more complicated, and
depends on whether hidden symbols should be internalized. E.g., for
ld -r, LTO_INTERNALIZE_NONE can be used when -keep_private_externs, and
LTO_INTERNALIZE_HIDDEN can be used otherwise. However,
LTO_INTERNALIZE_FULL is inappropriate, since the output object file will
eventually need to link with others.
lto_codegen_set_internalize_strategy() sets the strategy for subsequent
calls to lto_codegen_write_merged_modules() and lto_codegen_compile*().
<rdar://problem/14334895>
llvm-svn: 199244
2014-01-15 02:52:17 +08:00
|
|
|
ModulePass *llvm::createInternalizePass(bool OnlyHidden) {
|
|
|
|
return new InternalizePass(OnlyHidden);
|
|
|
|
}
|
|
|
|
|
|
|
|
ModulePass *llvm::createInternalizePass(ArrayRef<const char *> ExportList,
|
|
|
|
bool OnlyHidden) {
|
|
|
|
return new InternalizePass(ExportList, OnlyHidden);
|
2002-04-28 13:43:27 +08:00
|
|
|
}
|
2006-07-21 01:48:05 +08:00
|
|
|
|
Reapply "LTO: add API to set strategy for -internalize"
Reapply r199191, reverted in r199197 because it carelessly broke
Other/link-opts.ll. The problem was that calling
createInternalizePass("main") would select
createInternalizePass(bool("main")) instead of
createInternalizePass(ArrayRef<const char *>("main")). This commit
fixes the bug.
The original commit message follows.
Add API to LTOCodeGenerator to specify a strategy for the -internalize
pass.
This is a new attempt at Bill's change in r185882, which he reverted in
r188029 due to problems with the gold linker. This puts the onus on the
linker to decide whether (and what) to internalize.
In particular, running internalize before outputting an object file may
change a 'weak' symbol into an internal one, even though that symbol
could be needed by an external object file --- e.g., with arclite.
This patch enables three strategies:
- LTO_INTERNALIZE_FULL: the default (and the old behaviour).
- LTO_INTERNALIZE_NONE: skip -internalize.
- LTO_INTERNALIZE_HIDDEN: only -internalize symbols with hidden
visibility.
LTO_INTERNALIZE_FULL should be used when linking an executable.
Outputting an object file (e.g., via ld -r) is more complicated, and
depends on whether hidden symbols should be internalized. E.g., for
ld -r, LTO_INTERNALIZE_NONE can be used when -keep_private_externs, and
LTO_INTERNALIZE_HIDDEN can be used otherwise. However,
LTO_INTERNALIZE_FULL is inappropriate, since the output object file will
eventually need to link with others.
lto_codegen_set_internalize_strategy() sets the strategy for subsequent
calls to lto_codegen_write_merged_modules() and lto_codegen_compile*().
<rdar://problem/14334895>
llvm-svn: 199244
2014-01-15 02:52:17 +08:00
|
|
|
ModulePass *llvm::createInternalizePass(const char *SingleExport) {
|
|
|
|
return createInternalizePass(ArrayRef<const char *>(SingleExport));
|
2006-07-21 01:48:05 +08:00
|
|
|
}
|