diff options
author | Zhang Qing Shan <zhangqingshan.zll@bytedance.com> | 2021-04-12 14:55:03 +0800 |
---|---|---|
committer | Zhang Qing Shan <zhangqingshan.zll@bytedance.com> | 2021-04-12 14:55:03 +0800 |
commit | d69c236e1d6b4705d3a0041f39bfe864638eb24e (patch) | |
tree | d922da0d56cf1d5d82b97bde71d883f858be0657 /llvm/tools/llvm-objcopy/llvm-objcopy.cpp | |
parent | 2dd22da965ffc1623421cb8163947d7e679ce187 (diff) | |
download | llvm-d69c236e1d6b4705d3a0041f39bfe864638eb24e.zip llvm-d69c236e1d6b4705d3a0041f39bfe864638eb24e.tar.gz llvm-d69c236e1d6b4705d3a0041f39bfe864638eb24e.tar.bz2 |
[NFC][Debug] Fix unnecessary deep-copy for vector to save compiling time
We saw some big compiling time impact after enabling the debug entry value
feature for X86 platform(D73534). Compiling time goes from 900s->1600s with
our testcase. It is caused by allocating/freeing the memory busily.
'using FwdRegWorklist = MapVector<unsigned, SmallVector<FwdRegParamInfo, 2>>;'
The value for this map is vector, and we miss the reference when access the
element. The same happens for `auto CalleesMap = MF->getCallSitesInfo();` which is a DenseMap.
Reviewed by: djtodoro, flychen50
Differential Revision: https://reviews.llvm.org/D100162
Diffstat (limited to 'llvm/tools/llvm-objcopy/llvm-objcopy.cpp')
0 files changed, 0 insertions, 0 deletions