[PATCH] D152947: [DebugInfo] Change DWARFDebugAbbrev initialization

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 14 15:52:02 PDT 2023


dblaikie added a comment.

FWIW - I'd actually be inclined to consider going in the opposite direction, to parsing the abbrevs even more lazily... I'd been looking into the performance of `llvm-symbolizer` without `.debug_aranges` (only CU ranges) and abbrev parsing ends up taking a not insignificant chunk of the time - not a deal breaker, but a bit of work that could be avoided if we lazily read abbrevs (in most cases we'd only need to read the first apprev - likely where the CU abbrev description is)

I'd prototyped a really hacked up patch that might give the general idea: https://pastebin.com/md943qvf


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152947



More information about the llvm-commits mailing list