[PATCH] D50213: DebugInfo: Add metadata support for disabling DWARF pub sections

[Pavel Labath via Phabricator via llvm-commits]

labath added a comment.

In https://reviews.llvm.org/D50213#1191664, @dblaikie wrote:

> Actually - potentially we could merge the accelerator table options into this too, to promote them to an IR feature as well. But it looks like they aren't really supported on a per-CU basis, but are across the whole compilation only. (read the spec of debug_names - I see, best to emit the widest entry possible (ie: bad idea to emit one table per CU if we're emitting multiple CUs in a single object file (LTO) - and no need not to emit it if it's been requested, I suppose - though I guess a user might reasonably request no accelerator table for /some/ CUs they expect not to debug much/are OK with the debugger indexing slowly later on if they do)). But Apple's names don't support multiple CUs at all, so they can't be unified there anyway.


I got lost a bit when trying to parse this, but I just wanted to confirm that debug_names tables are capable indexing only a subset of CUs in the LTO case (multiple CUs in one object file), and even without emitting one table per each indexed CU -- you would just list the CUs that you index in the single uber-table. (Not that I have an idea why would anyone want to do that). You are correct in saying that apple tables do not allow this.


Repository:
  rL LLVM

https://reviews.llvm.org/D50213