[PATCH] D153268: [DWARFLinkerParallel] Add limited functionality to DWARFLinkerParallel.
Greg Clayton via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 20 13:50:29 PDT 2023
clayborg added a comment.
> a) No common abbreviation table. Each compile unit has its own abbreviation table. Generating common abbreviation table slowdowns parallel execution(This is a resource that is accessed many times from many threads). Abbreviation table does not take a lot of space, so it looks cheap to have separate abbreviations tables. Later, it might be optimized a bit(by removing equal abbreviations tables).
Can't we just let each compile unit generate its own abbrev table and then unique them into a single abbrev table at when we write out the .debug_info data? That way compile units can stay fast in parallel execution, and since we need to emit the compile units serially when generating the .debug_info, we can just have a map that maps CU specific abbrev codes to the global and final abbrev list? Then we end up with just one .debug_abbrev table in the end
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153268/new/
https://reviews.llvm.org/D153268
More information about the llvm-commits
mailing list