6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-01-30 19:35:54 +00:00
Code Issues Projects Releases Wiki Activity

The code that cleans up multiple, isomorphic types has a subtle error that

Browse Source 
manifests itself when building LLVM with LTO.
<rdar://problem/10913281>


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@151576 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Bill Wendling 2012-02-27 22:34:19 +00:00
parent 284a6bbae1
commit c68d127b2c
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
lib/Linker/LinkModules.cpp
View File

@ -580,7 +580,11 @@ void ModuleLinker::computeTypeMapping() {
if (GlobalValue *DGV = getLinkedToGlobal(I)) if (GlobalValue *DGV = getLinkedToGlobal(I))
TypeMap.addTypeMapping(DGV->getType(), I->getType()); TypeMap.addTypeMapping(DGV->getType(), I->getType());
} }
#if 0
// FIXME: This doesn't play well with LTO. We cannot compile LLVM with this
// enabled. <rdar://problem/10913281>.
// Incorporate types by name, scanning all the types in the source module. // Incorporate types by name, scanning all the types in the source module.
// At this point, the destination module may have a type "%foo = { i32 }" for // At this point, the destination module may have a type "%foo = { i32 }" for
// example. When the source module got loaded into the same LLVMContext, if // example. When the source module got loaded into the same LLVMContext, if
@ -610,8 +614,8 @@ void ModuleLinker::computeTypeMapping() {
if (!SrcStructTypesSet.count(DST)) if (!SrcStructTypesSet.count(DST))
TypeMap.addTypeMapping(DST, ST); TypeMap.addTypeMapping(DST, ST);
} }
#endif
// Don't bother incorporating aliases, they aren't generally typed well. // Don't bother incorporating aliases, they aren't generally typed well.
// Now that we have discovered all of the type equivalences, get a body for // Now that we have discovered all of the type equivalences, get a body for