mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-10-20 19:42:54 +02:00
55c3504da4
This is part 3 of a 3-part series to address a compile-time explosion
issue in LiveDebugValues.
---
Start encoding register locations within VarLoc IDs, and take advantage
of this encoding to speed up transferRegisterDef.
There is no fundamental algorithmic change: this patch simply swaps out
SparseBitVector in favor of CoalescingBitVector. That changes iteration
order (hence the test updates), but otherwise this patch is NFCI.
The only interesting change is in transferRegisterDef. Instead of doing:
```
KillSet = {}
for (ID : OpenRanges.getVarLocs())
if (DeadRegs.count(ID))
KillSet.add(ID)
```
We now do:
```
KillSet = {}
for (Reg : DeadRegs)
for (ID : intervalsReservedForReg(Reg, OpenRanges.getVarLocs()))
KillSet.add(ID)
```
By not visiting each open location every time we visit an instruction,
this eliminates some potentially quadratic behavior. The new
implementation basically does a constant amount of work per instruction
because the interval map lookups are very fast.
For a file in WebKit, this brings the time spent in LiveDebugValues down
from ~2.5 minutes to 4 seconds, reducing compile time spent in that pass
from 28% of the total to just over 1%.
Before:
```
2.49 min 27.8% 0 s LiveDebugValues::process
2.41 min 27.0% 5.40 s LiveDebugValues::transferRegisterDef
1.51 min 16.9% 1.51 min LiveDebugValues::VarLoc::isDescribedByReg() const
32.73 s 6.1% 8.70 s llvm::SparseBitVector<128u>::SparseBitVectorIterator::operator++()
```
After:
```
4.53 s 1.1% 0 s LiveDebugValues::process
3.00 s 0.7% 107.00 ms LiveDebugValues::transferRegisterCopy
892.00 ms 0.2% 406.00 ms LiveDebugValues::transferSpillOrRestoreInst
404.00 ms 0.1% 32.00 ms LiveDebugValues::transferRegisterDef
110.00 ms 0.0% 2.00 ms LiveDebugValues::getUsedRegs
57.00 ms 0.0% 1.00 ms std::__1::vector<>::push_back
40.00 ms 0.0% 1.00 ms llvm::CoalescingBitVector<>::find(unsigned long long)
```
FWIW, I tried the same approach using SparseBitVector, but got bad
results. To do that, I had to extend SparseBitVector to support 64-bit
indices and expose its lower bound operation. The problem with this is
that the performance is very hard to predict: SparseBitVector's lower
bound operation falls back to O(n) linear scans in a std::list if you're
not /very/ careful about managing iteration order. When I profiled this
the performance looked worse than the baseline.
You can see the full CoalescingBitVector-based implementation here:
https://github.com/vedantk/llvm-project/commits/try-coalescing
You can see the full SparseBitVector-based implementation here:
https://github.com/vedantk/llvm-project/commits/try-sparsebitvec-find
Depends on D74984 and D74985.
Differential Revision: https://reviews.llvm.org/D74986
|
||
|---|---|---|
| .. | ||
| AArch64 | ||
| AMDGPU | ||
| ARM | ||
| COFF | ||
| Generic | ||
| Inputs | ||
| Lanai | ||
| Mips | ||
| MIR | ||
| MSP430 | ||
| NVPTX | ||
| PDB | ||
| PowerPC | ||
| RISCV | ||
| Sparc | ||
| SystemZ | ||
| WebAssembly | ||
| X86 | ||
| arange-overlap.test | ||
| arm-relocs.test | ||
| check-debugify-preserves-analyses.ll | ||
| cross-cu-inlining.test | ||
| cross-cu-scope.ll | ||
| debug-frame-cie-pointer-reloc.test | ||
| debugify-bogus-dbg-value.ll | ||
| debugify-each.ll | ||
| debugify-export.ll | ||
| debugify-report-missing-locs-only.ll | ||
| debugify.ll | ||
| debuglineinfo-macho.test | ||
| debuglineinfo-path.ll | ||
| debuglineinfo.test | ||
| debugmacinfo-dwo.test | ||
| debugmacinfo.test | ||
| dwarfdump-64-bit-dwarf.test | ||
| dwarfdump-accel.test | ||
| dwarfdump-bad-lookup-address.test | ||
| dwarfdump-debug-frame-simple.test | ||
| dwarfdump-decompression-corrupt.test | ||
| dwarfdump-decompression-error.test | ||
| dwarfdump-dump-flags.test | ||
| dwarfdump-dump-gdbindex.test | ||
| dwarfdump-dwp.test | ||
| dwarfdump-implicit-const.test | ||
| dwarfdump-invalid.test | ||
| dwarfdump-macho-relocs.test | ||
| dwarfdump-macho-universal.test | ||
| dwarfdump-objc.test | ||
| dwarfdump-pubnames.test | ||
| dwarfdump-ranges.test | ||
| dwarfdump-type-units.test | ||
| dwarfdump-zlib.test | ||
| dwo.ll | ||
| gcc-local-mem-func.test | ||
| high-pc-constant.test | ||
| invalid-relocations.test | ||
| macro_link.ll | ||
| member-pointers.o | ||
| missing-abstract-variable.o | ||
| omit-empty.ll | ||
| pr34186.ll | ||
| pr34672.ll | ||
| pr37964.ll | ||
| precomp.test | ||
| salvage-cast-debug-info.ll | ||
| skeletoncu.ll | ||
| strip-DIGlobalVariable.ll | ||
| strip-loop-metadata.ll | ||
| strip-module-flags.ll | ||
| symbolize-64bit-address.test | ||
| symbolize-absolute-path-in-debug-line.s | ||
| symbolize-build-id-error.test | ||
| symbolize-build-id.test | ||
| symbolize-demangling.s | ||
| symbolize-directory.s | ||
| symbolize-gnu-debuglink-fallback.test | ||
| symbolize-gnu-debuglink-no-realpath.test | ||
| symbolize-gnu-debuglink.test | ||
| symbolize-inlined.test | ||
| symbolize-macho-universal-unknown-arch.test | ||
| symbolize-macho-universal.test | ||
| symbolize-missing-file.test | ||
| symbolize-no-debug-str.test | ||
| symbolize-paths.s | ||
| symbolize-shared-abbrev.s | ||
| symbolize-split-dwarf-empty.test | ||
| symbolize-split-dwarf-no-skel-address.test | ||
| symbolize-stripped-sections.test | ||
| symbolize-stripped.test | ||
| symbolize-tu.test | ||
| symbolize-zlib.test | ||
| symbolize.test | ||
| typeunit-header.test | ||
| unrolled-loop-remainder.ll | ||