RPCS3/llvm-mirror
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm-mirror.git synced 2024-12-14 23:29:51 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Eliminate the concept of a deferred symbol table. The optimization really isn't,

Browse Source 
and it causes obscure bugs to show up in passes.

llvm-svn: 4777
This commit is contained in:
Chris Lattner 2002-11-20 18:07:48 +00:00
parent 505ca2e419
commit 46b6e30fef
2 changed files with 4 additions and 35 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
include/llvm/Function.h
View File

@ -111,22 +111,12 @@ public:
//===--------------------------------------------------------------------===//
// Symbol Table Accessing functions...
/// hasSymbolTable() - Returns true if there is a symbol table allocated to
/// this object AND if there is at least one name in it!
///
bool hasSymbolTable() const;
/// getSymbolTable() - CAUTION: The current symbol table may be null if there
/// are no names (ie, the symbol table is empty)
/// getSymbolTable() - Return the symbol table...
///
inline SymbolTable *getSymbolTable() { return SymTab; }
inline const SymbolTable *getSymbolTable() const { return SymTab; }
/// getSymbolTableSure is guaranteed to not return a null pointer, because if
/// the function does not already have a symtab, one is created. Use this if
/// you intend to put something into the symbol table for the function.
///
SymbolTable *getSymbolTableSure(); // Implemented in Value.cpp
SymbolTable *getSymbolTableSure() { return SymTab; }
bool hasSymbolTable() const { return true; }
//===--------------------------------------------------------------------===//

23
lib/VMCore/Function.cpp
View File

@ -84,7 +84,7 @@ Function::Function(const FunctionType *Ty, bool isInternal,
BasicBlocks.setParent(this);
ArgumentList.setItemParent(this);
ArgumentList.setParent(this);
SymTab = 0;
SymTab = new SymbolTable();
// Create the arguments vector, all arguments start out unnamed.
for (unsigned i = 0, e = Ty->getNumParams(); i != e; ++i) {
@ -137,27 +137,6 @@ const Type *Function::getReturnType() const {
return getFunctionType()->getReturnType();
}
SymbolTable *Function::getSymbolTableSure() {
if (!SymTab) SymTab = new SymbolTable();
return SymTab;
}
// hasSymbolTable() - Returns true if there is a symbol table allocated to
// this object AND if there is at least one name in it!
//
bool Function::hasSymbolTable() const {
if (!SymTab) return false;
for (SymbolTable::const_iterator I = SymTab->begin();
I != SymTab->end(); ++I) {
if (I->second.begin() != I->second.end())
return true; // Found nonempty type plane!
}
return false;
}
// dropAllReferences() - This function causes all the subinstructions to "let
// go" of all references that they are maintaining. This allows one to
// 'delete' a whole class at a time, even though there may be circular