[PATCH] D124650: [clang-tidy] Simplify boolean expressions by DeMorgan's theorem

Richard via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 28 17:32:17 PDT 2022


LegalizeAdulthood added a comment.

General question:

There is the question of whether or not the replacement should have `()`s around the whole
thing because we don't analyze the context to see if the change in precedence between
`operator!` and `operator&&` or `operator||` makes any difference.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D124650/new/

https://reviews.llvm.org/D124650



More information about the cfe-commits mailing list