kadircet added a comment.

In D83224#2133457 <https://reviews.llvm.org/D83224#2133457>, @njames93 wrote:

> Do you think there should be a hidden command line option to disable 
> disabling blacklisted checks, purely to prevent hindering attempts to fix 
> these problematic checks.

I would expect those to be fixed and verified with unittests that are using 
TestTU, so the changes in ClangdServer shouldn't really be relevant to them. 
And if developer is trying to test something locally, i think it is better for 
them to just update the blacklist file(they are going to end up doing that 
anyways and perform possibly multiple compiles), so a flag to change blacklists 
behaviour might not be that meaningful.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83224

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to