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

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 8 22:11:27 PDT 2019


MaskRay added a comment.

- clang has `-ferror-limit=` that defaults to 19.
- Since we've got `error-limit=`, it seems reasonable to have a parallel `warning-limit=`.
- 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.


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

https://reviews.llvm.org/D61710





More information about the llvm-commits mailing list