[PATCH] D96035: [WIP][dsymutil][DWARFlinker] implement separate multi-thread processing for compile units.

Alexey Lapshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 5 14:19:52 PST 2021


avl added a comment.

> That seems quite expensive :s

yep, it requires additional space. Though it allows to have benefits:

- it allows separate parallel processing(this patch has more than 2x performance improvement).
- plain DWARF requires less space than parsed/generated(DWARFDie/DIE).

> "how this implementation works" - the patch in this review? The existing checked-in implementation? (apologies as I'm not especially familiar with the implementation of dsymutil as it is today)

yes, the patch in this review.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96035/new/

https://reviews.llvm.org/D96035



More information about the llvm-commits mailing list