6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-01-18 13:34:04 +00:00
Code Issues Projects Releases Wiki Activity

Fix a compile time regression caused by too small hash tables.

Browse Source 
Measure the worst case number of probes for a miss instead of the less
conservative number of probes required for an insertion.

Lower the limit to < 6 probes worst case.

This doubles the size of the ARM and X86 hash tables, other targets are
unaffected. LiveVariables runs 12% faster with this change.

<rdar://problem/9598545>

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@132999 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Jakob Stoklund Olesen 2011-06-14 16:58:16 +00:00
parent f924dea8dd
commit 952036def9
1 changed files with 16 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
utils/TableGen/RegisterInfoEmitter.cpp
View File

@ -145,7 +145,6 @@ static void generateHashTable(raw_ostream &OS, const char *Name,
HT.assign(HSize, Sentinel);
// Insert all entries.
MaxProbes = 0;
for (unsigned i = 0, e = Data.size(); i != e; ++i) {
UUPair D = Data[i];
unsigned Idx = (D.first * 11 + D.second * 97) & (HSize - 1);
@ -155,10 +154,24 @@ static void generateHashTable(raw_ostream &OS, const char *Name,
ProbeAmt += 1;
}
HT[Idx] = D;
}
// Now measure the max number of probes for any worst case miss.
MaxProbes = 0;
unsigned TotalProbes = 0;
for (unsigned i = 0, e = HSize; i != e; ++i) {
unsigned Idx = i;
unsigned ProbeAmt = 1;
while (HT[Idx] != Sentinel) {
Idx = (Idx + ProbeAmt) & (HSize - 1);
ProbeAmt += 1;
}
TotalProbes += ProbeAmt;
MaxProbes = std::max(MaxProbes, ProbeAmt);
}
OS << "\n // Max number of probes: " << MaxProbes;
} while (MaxProbes >= 8);
OS << "\n // Max number of probes: " << MaxProbes
<< format(", avg %.1f", float(TotalProbes)/HSize);
} while (MaxProbes >= 6);
// Print the hash table.
OS << "\n // Used entries: " << Data.size()