maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Drop SrcStructTypesSet. It is redundant. 

At the only point in the code it is used, we haven't added any of the src types
to DstStructTypesSet yet.

llvm-svn: 223057
This commit is contained in:
Rafael Espindola 2014-12-01 18:42:18 +00:00
parent 4af55faa19
commit dd42b533e9
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
llvm/lib/Linker/LinkModules.cpp
View File

@ -793,8 +793,6 @@ void ModuleLinker::computeTypeMapping() {
// it had the same type, it would have been renamed to "%foo.42 = { i32 }".
TypeFinder SrcStructTypes;
SrcStructTypes.run(*SrcM, true);
SmallPtrSet<StructType*, 32> SrcStructTypesSet(SrcStructTypes.begin(),
SrcStructTypes.end());
for (unsigned i = 0, e = SrcStructTypes.size(); i != e; ++i) {
StructType *ST = SrcStructTypes[i];
@ -826,7 +824,7 @@ void ModuleLinker::computeTypeMapping() {
// we prefer to take the '%C' version. So we are then left with both
// '%C.1' and '%C' being used for the same types. This leads to some
// variables using one type and some using the other.
if (!SrcStructTypesSet.count(DST) && TypeMap.DstStructTypesSet.count(DST))
if (TypeMap.DstStructTypesSet.count(DST))
TypeMap.addTypeMapping(DST, ST);
}