EugeneZelenko wrote:

> > > What do others think?
> > 
> > 
> > I agree, I have somewhat mixed feelings about this rule. Also negating 
> > conditions to make them more optimal/short can also lead to worse 
> > readability. I believe this is alleviated by setting `SimplifyDeMorgan` to 
> > `false`.
> 
> I will do some experimentation the config options

I enabled `IgnoreMacros` when running this check over my work code base, since 
a lot of warnings were related to stuff like asserts.

https://github.com/llvm/llvm-project/pull/158706
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to