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

[Zachary Turner via Phabricator via llvm-commits]

zturner added a comment.

In https://reviews.llvm.org/D41310#957241, @smeenai wrote:

> @zturner I do agree that `/debug` and `/debug:dwarf` is a bit confusing. We have to keep `/debug` for link.exe compatibility, so perhaps `/debug:dwarf` could be renamed to something else? Any suggestions?


Sorry, I didn't mean the linker shouldn't support more than one option, I just meant it shouldn't supporting using them **at the same time**.  This agrees with what you were saying about `/debug` and `/debug:dwarf` not making sense when used together, but actually goes one step further.  Do any two `/DEBUG` option variants make sense together?


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D41310