vbvictor 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

If we get unsatisfying results with it, we should think about making this check 
less aggressive (via new options) and leave this check disabled for now (But we 
could merge good transformations from it)



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