[lld] [LLD] [COFF] Add /debug: options nodwarf and nosymtab (PR #75180)

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 13 14:52:19 PST 2023


Martin =?utf-8?q?Storsjö?= <martin at martin.st>,
Martin =?utf-8?q?Storsjö?= <martin at martin.st>,
Martin =?utf-8?q?Storsjö?= <martin at martin.st>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/75180 at github.com>


https://github.com/MaskRay approved this pull request.

For a non-list option, I think it is very uncommon for a second occurrence of an option to override partial effects done by the first occurrence. Actually, I cannot think of an example off my head.

ELF linkers have an example of `-z defs`, `-z undefs`, `--allow-shlib-undefined`, `--unresolved-symbols` where `--unresolved-symbols` overrides both `diagRegular/diagShlib` while other options may override one of diagRegular/diagShlib.

That said, if this `/debug:` is useful, I think it is fine to be a special case...

https://github.com/llvm/llvm-project/pull/75180


More information about the llvm-commits mailing list