[clang-tools-extra] [clang-tidy][NFC] Enable 'readability-simplify-boolean-expr' check (PR #158706)
via cfe-commits
cfe-commits at lists.llvm.org
Tue Sep 16 07:45:52 PDT 2025
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
More information about the cfe-commits
mailing list