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

> (this diff isn't great news for the idea of treating new checks as slow by 
> default :-()

Well, FWIW, i think that was the trade-off we made. Most of the checks are 
fast, but when a slow one sneaks in, it's really hard to notice. So by being 
more explicit about the checks we enable, we make sure quality stays 
"acceptable" at the cost of missing new checks until they're explicitly turned 
on.

---

While at it, I wonder if the list of disabled checks are useful to mention in 
release notes going forward. I think we should at least create bugs in 
llvm-project issue tracker so that the authors are aware. WDYT?

https://github.com/llvm/llvm-project/pull/69723
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to