mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2025-06-22 10:24:26 +00:00
Fix bug: FunctionResolve/2003-05-31-AllInternalDecls.ll
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@6486 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
@ -309,6 +309,23 @@ static bool ProcessGlobalsWithSameName(Module &M,
|
|||||||
}
|
}
|
||||||
|
|
||||||
if (Globals.size() > 1) { // Found a multiply defined global...
|
if (Globals.size() > 1) { // Found a multiply defined global...
|
||||||
|
// If there are no external declarations, and there is at most one
|
||||||
|
// externally visible instance of the global, then there is nothing to do.
|
||||||
|
//
|
||||||
|
bool HasExternal = false;
|
||||||
|
unsigned NumInstancesWithExternalLinkage = 0;
|
||||||
|
|
||||||
|
for (unsigned i = 0, e = Globals.size(); i != e; ++i) {
|
||||||
|
if (Globals[i]->isExternal())
|
||||||
|
HasExternal = true;
|
||||||
|
else if (!Globals[i]->hasInternalLinkage())
|
||||||
|
NumInstancesWithExternalLinkage++;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!HasExternal && NumInstancesWithExternalLinkage <= 1)
|
||||||
|
return false; // Nothing to do? Must have multiple internal definitions.
|
||||||
|
|
||||||
|
|
||||||
// We should find exactly one concrete function definition, which is
|
// We should find exactly one concrete function definition, which is
|
||||||
// probably the implementation. Change all of the function definitions and
|
// probably the implementation. Change all of the function definitions and
|
||||||
// uses to use it instead.
|
// uses to use it instead.
|
||||||
|
Reference in New Issue
Block a user