mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2025-01-12 17:32:19 +00:00
[llvm link] Destroy ConstantArrays in LLVMContext if they are not used.
ConstantArrays constructed during linking can cause quadratic memory explosion. An example is the ConstantArrays constructed when linking in GlobalVariables with appending linkage. Releasing all unused constants can cause a 20% LTO compile-time slowdown for a large application. So this commit releases unused ConstantArrays only. rdar://19040716. It reduces memory footprint from 20+G to 6+G. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@226592 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
parent
daa414fd39
commit
0d12d4ebc6
@ -630,6 +630,15 @@ public:
|
||||
named_metadata_end());
|
||||
}
|
||||
|
||||
/// Destroy ConstantArrays in LLVMContext if they are not used.
|
||||
/// ConstantArrays constructed during linking can cause quadratic memory
|
||||
/// explosion. Releasing all unused constants can cause a 20% LTO compile-time
|
||||
/// slowdown for a large application.
|
||||
///
|
||||
/// NOTE: Constants are currently owned by LLVMContext. This can then only
|
||||
/// be called where all uses of the LLVMContext are understood.
|
||||
void dropTriviallyDeadConstantArrays();
|
||||
|
||||
/// @}
|
||||
/// @name Utility functions for printing and dumping Module objects
|
||||
/// @{
|
||||
|
@ -163,6 +163,28 @@ LLVMContextImpl::~LLVMContextImpl() {
|
||||
MDStringCache.clear();
|
||||
}
|
||||
|
||||
void LLVMContextImpl::dropTriviallyDeadConstantArrays() {
|
||||
bool Changed;
|
||||
do {
|
||||
Changed = false;
|
||||
|
||||
for (auto I = ArrayConstants.map_begin(), E = ArrayConstants.map_end();
|
||||
I != E; ) {
|
||||
auto *C = I->first;
|
||||
I++;
|
||||
if (C->use_empty()) {
|
||||
Changed = true;
|
||||
C->destroyConstant();
|
||||
}
|
||||
}
|
||||
|
||||
} while (Changed);
|
||||
}
|
||||
|
||||
void Module::dropTriviallyDeadConstantArrays() {
|
||||
Context.pImpl->dropTriviallyDeadConstantArrays();
|
||||
}
|
||||
|
||||
namespace llvm {
|
||||
/// \brief Make MDOperand transparent for hashing.
|
||||
///
|
||||
|
@ -474,6 +474,9 @@ public:
|
||||
|
||||
LLVMContextImpl(LLVMContext &C);
|
||||
~LLVMContextImpl();
|
||||
|
||||
/// Destroy the ConstantArrays if they are not used.
|
||||
void dropTriviallyDeadConstantArrays();
|
||||
};
|
||||
|
||||
}
|
||||
|
@ -1721,7 +1721,9 @@ void Linker::deleteModule() {
|
||||
bool Linker::linkInModule(Module *Src) {
|
||||
ModuleLinker TheLinker(Composite, IdentifiedStructTypes, Src,
|
||||
DiagnosticHandler);
|
||||
return TheLinker.run();
|
||||
bool RetCode = TheLinker.run();
|
||||
Composite->dropTriviallyDeadConstantArrays();
|
||||
return RetCode;
|
||||
}
|
||||
|
||||
//===----------------------------------------------------------------------===//
|
||||
|
Loading…
x
Reference in New Issue
Block a user