diff options
author | Johannes Doerfert <johannes@jdoerfert.de> | 2020-05-07 16:50:50 -0500 |
---|---|---|
committer | Johannes Doerfert <johannes@jdoerfert.de> | 2020-05-07 17:00:50 -0500 |
commit | edf0391491e3a2076779c5da2c0dfc1829585d64 (patch) | |
tree | 172de56c9c5ba079e6d36d32f2f8d980007b5106 /llvm/lib/CodeGen/MachineBasicBlock.cpp | |
parent | 675334daef991253e917411324d51751007c2fb8 (diff) | |
download | llvm-edf0391491e3a2076779c5da2c0dfc1829585d64.zip llvm-edf0391491e3a2076779c5da2c0dfc1829585d64.tar.gz llvm-edf0391491e3a2076779c5da2c0dfc1829585d64.tar.bz2 |
[Attributor][FIX] Record dependences for assumed dead abstract attributes
In a recent patch we introduced a problem with abstract attributes that
were assumed dead at some point. Since `Attributor::updateAA` was
introduced in 95e0d28b71e42c9b7cd77c96f728311981a021f6, we did not
remember the dependence on the liveness AA when an abstract attribute
was assumed dead and therefore not updated.
Explicit reproducer added in liveness.ll.
---
Single run of the Attributor module and then CGSCC pass (oldPM)
for SPASS/clause.c (~10k LLVM-IR loc):
Before:
```
calls to allocation functions: 509242 (345483/s)
temporary memory allocations: 98666 (66937/s)
peak heap memory consumption: 18.60MB
peak RSS (including heaptrack overhead): 103.29MB
total memory leaked: 269.10KB
```
After:
```
calls to allocation functions: 529332 (355494/s)
temporary memory allocations: 102107 (68574/s)
peak heap memory consumption: 19.40MB
peak RSS (including heaptrack overhead): 102.79MB
total memory leaked: 269.10KB
```
Difference:
```
calls to allocation functions: 20090 (1339333/s)
temporary memory allocations: 3441 (229400/s)
peak heap memory consumption: 801.45KB
peak RSS (including heaptrack overhead): 0B
total memory leaked: 0B
```
Diffstat (limited to 'llvm/lib/CodeGen/MachineBasicBlock.cpp')
0 files changed, 0 insertions, 0 deletions