[PATCH] D142800: [Clang][Diagnostic] Add `-Wcomparison-op-parentheses` to warn on chained comparisons
Erich Keane via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Feb 3 07:43:12 PST 2023
erichkeane added a comment.
I'm reasonably happy here. I think the chained-comparisons getting the && suggestion is nice. However in each case, the diagnostic is REALLY poor. It isn't really clear to most folks (barely clear to me!) that `'<' within '>'` really means anything. I'd love if we could spend some time brainstorming some better diagnostic wording that makes it clear what the problem is.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142800/new/
https://reviews.llvm.org/D142800
More information about the cfe-commits
mailing list