carlosgalvezp wrote:

> The check can recognize side effects.

Ok, that's great! Did you have a chance to run the test on a large codebase 
(e.g. the llvm-project) and see if we get any FPs in this regard?

>  if the check does more than one thing (readability, performance) we should 
> place it under misc category

That's a good point, I agree. Is there be a strong reason _not_ to put it in 
`misc`?



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

Reply via email to