[PATCH] D61710: lld: Add a warning limit, similar to the existing error limit

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 8 22:46:53 PDT 2019


ruiu added a comment.

> - I'm not sure the counterpart of `clang -w` in the linker world will be useful. Most linker warnings are real problems. In contract, many compiler warnings are just style issues and one may argue some are really arbitrary. Some ELF builds have `--fatal-warnings`, the probably just turn off that option if they don't want to see errors.

That's a good point. When the linker prints out a warning message, that usually means there's a real problem in your program, even if your program would work fine. I don't think implementing /ignore:4006 to lld is a bad option. It may actually be a good option, as I can't think of any other warning messages that we want to suppress at the moment.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61710/new/

https://reviews.llvm.org/D61710





More information about the llvm-commits mailing list