[PATCH] D46021: [DEBUGINFO] Disable emission of the dwarf sections, but allow directives.

[Eric Christopher via Phabricator via llvm-commits]

echristo added a comment.

In https://reviews.llvm.org/D46021#1077031, @ABataev wrote:

> In https://reviews.llvm.org/D46021#1077023, @dblaikie wrote:
>
> > Sorry, I'm a bit confused.
> >
> > 1. I'd expect /this/ behavior (line table directives, but no debug_info, etc) to be controlled via a DICompileUnit emission kind (a value between "NoDebug" and "LineTablesOnly" as Paul mentioned).
> > 2. I'd expect the "no ', debug'" flag (which I don't see in this change? Is it missing? is it elsewhere?) to be controlled via an MCOption - because it must be global.
> >
> >   Now, arguably, if we have the global "no ', debug'" flag we maybe might as well use it to implement this behavior too since we don't have any other use for it? So maybe (2) invalidates (1).
>
>
> I thought, that it is not very useful to have an option that, actually, controls only single target and I decided instead to add the option that just disables emission of the DWARF sections. In this case we don't need the special debug info mode, we can use this option.


I agree with Dave here. We already have some existing functionality that handles, for example, "minimal line tables" and so extending it to be "location directives only" seems like the direction we should go.


Repository:
  rL LLVM

https://reviews.llvm.org/D46021