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

Shoaib Meenai via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 15 13:48:26 PST 2017


smeenai added a comment.

In https://reviews.llvm.org/D41310#957239, @ruiu wrote:

> Conceptually, an executable that has both DAWRF and PDB debug info is valid, no?


That's true in theory, but the current implementation assumes they're mutually exclusive (e.g. PDBs aren't generated for `/debug:dwarf`).

I don't know how much utility there is for DWARF + PDB. I also though clang could only generate one or the other (i.e. if you specify `-gcodeview` you'll only get CodeView and not DWARF), though I may be mistaken there.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D41310





More information about the llvm-commits mailing list