[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 18:29:17 PDT 2019


ruiu added a comment.

I'm not confident that this is what the user wants. This might be useful, but at the same time I can imagine that people want to see an entire log rather than the first part of it.

Does clang or other tools have a similar feature?



================
Comment at: lld/COFF/Driver.cpp:68-69
+  errorHandler().WarningLimitExceededMsg =
+      "too many warnings emitted, stopping now"
+      " (use /warninglimit:0 to see all warnings)";
   errorHandler().ExitEarly = CanExitEarly;
----------------
I don't think "stopping" is a correct word to explain the feature. It stops showing more warnings, but the process continues.


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

https://reviews.llvm.org/D61710





More information about the llvm-commits mailing list