[PATCH] D48899: [dsymutil] Convert recursion in lookForDIEsToKeep into worklist.

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 5 15:06:44 PDT 2018


JDevlieghere updated this revision to Diff 154325.
JDevlieghere marked an inline comment as done.
JDevlieghere added a comment.

- Improve comments.
- Remove redundant continuation at beginning. Wwe always add the continuation if we add the DIE's children to the worklist.
- Use SmallVectors to improve performance. Still this path causes a ~5% performance regression (avg over 5 runs).


https://reviews.llvm.org/D48899

Files:
  llvm/tools/dsymutil/DwarfLinker.cpp
  llvm/tools/dsymutil/DwarfLinker.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D48899.154325.patch
Type: text/x-patch
Size: 7826 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180705/4673021a/attachment.bin>


More information about the llvm-commits mailing list