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

[Shoaib Meenai via Phabricator via llvm-commits]

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?


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D41310