[PATCH] D68620: DebugInfo: Use base address selection entries for debug_loc

[David Blaikie via Phabricator via llvm-commits]

dblaikie added a comment.

In D68620#2287669 <https://reviews.llvm.org/D68620#2287669>, @Orlando wrote:

> Just to clarify: when I say "elf" here I'm talking about the linked executable file, and "object files" are the pre-link .o files.

Thanks - makes sense.

I guess all of these measurements were done without Split DWARF (shouldn't make things better/worse overall (across .o and .dwo), really, compared to DWARFv5 non-split - but means when only looking at .o files the difference of avoiding more debug_addr entries is more significant because there's fewer remaining .o debug bytes to begin with) and without compression (-gz) enabled?

@probinson - how're these tradeoffs all sounding to you, and did you have further thoughts on what sounds like somewhat of a source of confusion in the previous posts on this thread (the question of whether a base address selection entry for a one-entry list could be beneficial, which I believe it is/showed the numbers/my reasoning there)?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68620