mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2024-08-17 21:29:20 +00:00
* Revert to old behavior of ignoring a module if it doesn't contain a main
function and no symbols were explicitly marked to be externalized. * Add new -internalize-public-api-list option that can be used if the symbol list is small, and making a new file is annoying. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@6289 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
parent
c7a2c7f0c9
commit
2345d71853
@ -22,16 +22,21 @@ namespace {
|
|||||||
// external.
|
// external.
|
||||||
cl::opt<std::string>
|
cl::opt<std::string>
|
||||||
APIFile("internalize-public-api-file", cl::value_desc("filename"),
|
APIFile("internalize-public-api-file", cl::value_desc("filename"),
|
||||||
cl::desc("A file containing list of globals to not internalize"));
|
cl::desc("A file containing list of symbol names to preserve"));
|
||||||
|
|
||||||
|
// APIList - A list of symbols that should not be marked internal.
|
||||||
|
cl::list<std::string>
|
||||||
|
APIList("internalize-public-api-list", cl::value_desc("list"),
|
||||||
|
cl::desc("A list of symbol names to preserve"));
|
||||||
|
|
||||||
class InternalizePass : public Pass {
|
class InternalizePass : public Pass {
|
||||||
std::set<std::string> ExternalNames;
|
std::set<std::string> ExternalNames;
|
||||||
public:
|
public:
|
||||||
InternalizePass() {
|
InternalizePass() {
|
||||||
if (!APIFile.empty())
|
if (!APIFile.empty()) // If a filename is specified, use it
|
||||||
LoadFile(APIFile.c_str());
|
LoadFile(APIFile.c_str());
|
||||||
else
|
else // Else, if a list is specified, use it.
|
||||||
ExternalNames.insert("main");
|
ExternalNames.insert(APIList.begin(), APIList.end());
|
||||||
}
|
}
|
||||||
|
|
||||||
void LoadFile(const char *Filename) {
|
void LoadFile(const char *Filename) {
|
||||||
@ -39,7 +44,7 @@ namespace {
|
|||||||
std::ifstream In(Filename);
|
std::ifstream In(Filename);
|
||||||
if (!In.good()) {
|
if (!In.good()) {
|
||||||
std::cerr << "WARNING: Internalize couldn't load file '" << Filename
|
std::cerr << "WARNING: Internalize couldn't load file '" << Filename
|
||||||
<< "'!: Not internalizing.\n";
|
<< "'!\n";
|
||||||
return; // Do not internalize anything...
|
return; // Do not internalize anything...
|
||||||
}
|
}
|
||||||
while (In) {
|
while (In) {
|
||||||
@ -51,7 +56,19 @@ namespace {
|
|||||||
}
|
}
|
||||||
|
|
||||||
virtual bool run(Module &M) {
|
virtual bool run(Module &M) {
|
||||||
if (ExternalNames.empty()) return false; // Error loading file...
|
// If no list or file of symbols was specified, check to see if there is a
|
||||||
|
// "main" symbol defined in the module. If so, use it, otherwise do not
|
||||||
|
// internalize the module, it must be a library or something.
|
||||||
|
//
|
||||||
|
if (ExternalNames.empty()) {
|
||||||
|
Function *MainFunc = M.getMainFunction();
|
||||||
|
if (MainFunc == 0 || MainFunc->isExternal())
|
||||||
|
return false; // No main found, must be a library...
|
||||||
|
|
||||||
|
// Preserve main, internalize all else.
|
||||||
|
ExternalNames.insert(MainFunc->getName());
|
||||||
|
}
|
||||||
|
|
||||||
bool Changed = false;
|
bool Changed = false;
|
||||||
|
|
||||||
// Found a main function, mark all functions not named main as internal.
|
// Found a main function, mark all functions not named main as internal.
|
||||||
|
Loading…
Reference in New Issue
Block a user