[PATCH] D41310: [COFF] Remove /nopdb option

[Rui Ueyama via llvm-commits]

On Fri, Dec 15, 2017 at 2:05 PM, Shoaib Meenai via Phabricator <
reviews at reviews.llvm.org> wrote:

> smeenai added a comment.
>
> So it looks like clang produces both CodeView and DWARF in the same object
> file if you pass both `-gcodeview` and `-gdwarf`. I haven't verified the
> debug info is sensible, but the object file at least contains both CodeView
> and DWARF debug sections.
>
> With that in mind, I think the following makes sense:
>
> 1. `/debug` by itself produces a PDB (this is unchanged).
> 2. `/debug:dwarf` by itself produces DWARF (this is unchanged).
> 3. `/debug` and `/debug:dwarf` produce both a PDB and DWARF (this is
> different from today, where `/debug:dwarf` always prevents a PDB from being
> created).
> 4. `/nopdb` can still be deleted; just don't specify `/debug` if you don't
> want a PDB (or perhaps we can add `/debug:none` to cancel out `/debug`, but
> then it's confusing as to whether it should also cancel out `/debug:dwarf`
> ...)
>
> Does this sound good to everyone?


Makes sense to me.


>
> Repository:
>   rLLD LLVM Linker
>
> https://reviews.llvm.org/D41310
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171215/4f44a53d/attachment.html>