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

[David Blaikie via llvm-commits]

On Tue, Apr 24, 2018 at 10:08 AM Alexey Bataev via Phabricator <
reviews at reviews.llvm.org> wrote:

> ABataev added a comment.
>
> 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'm not sure I follow - specifically I'm still not sure where the "no ',
debug'" support is in this patch - did I miss it?

As for whether it's useful to have the "don't emit debug sections except
for the line table" as a DICompileUnit feature - the reason to put it there
would be for consistency with other related variations in debug info
emission, even if it is only used by one target.


>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D46021
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180424/740e8473/attachment.html>