diff options
author | Jonas Devlieghere <jonas@devlieghere.com> | 2019-12-03 19:41:04 -0800 |
---|---|---|
committer | Jonas Devlieghere <jonas@devlieghere.com> | 2019-12-04 10:20:53 -0800 |
commit | 9b15873c92832b06f41be1e1047ea5033411ee0f (patch) | |
tree | 046344aa8f755de12976265a0f2d245aeb0f6e8d /llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp | |
parent | 95a8e8a2557e487be6133475966255661a9f6567 (diff) | |
download | bcm5719-llvm-9b15873c92832b06f41be1e1047ea5033411ee0f.tar.gz bcm5719-llvm-9b15873c92832b06f41be1e1047ea5033411ee0f.zip |
[dsymutil] Remove recursion from lookForChildDIEsToKeep (2/2) (NFC)
The functions lookForDIEsToKeep and keepDIEAndDependencies are mutually
recursive and can cause a stackoverflow for large projects. While this
has always been the case, it became a bigger issue when we parallelized
dsymutil, because threads get only a fraction of the stack space.
This patch removes the final recursive call from lookForDIEsToKeep. The
call was used to look at the current DIE's parent chain and mark
everything as kept.
This was tested by running dsymutil on clang built in debug (both with
and without modules) and comparing the MD5 hash of the generated dSYM
companion file.
Differential revision: https://reviews.llvm.org/D70994
Diffstat (limited to 'llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp')
0 files changed, 0 insertions, 0 deletions