Details

Reviewers

njames93
carlosgalvezp
LegalizeAdulthood

Summary

Check that flags chained comparison expressions,
such as a < b < c or a == b == c, which may have
unintended behavior due to implicit operator
associativity.

Diff Detail

Repository: rG LLVM Github Monorepo

Event Timeline

PiotrZSL created this revision.Feb 20 2023, 1:19 PM

Herald added a reviewer: njames93. · View Herald TranscriptFeb 20 2023, 1:19 PM

Herald added a project: Restricted Project. · View Herald Transcript

Herald added subscribers: carlosgalvezp, xazax.hun. · View Herald Transcript

Note that looks like there is currently in implementation new warning -Wcomparison-op-parentheses under D142800, that warning currently handle only chained comparison by using BinaryOperator, not by using CXXOperatorCallExpr.
Scope of "-Wcomparison-op-parentheses" and whatever that warning will be delivered in LLVM 17 will determinate future of this check.
This check is feature-complete.

Harbormaster completed remote builds in B214841: Diff 498945.Feb 20 2023, 1:38 PM

PiotrZSL published this revision for review.Feb 20 2023, 1:43 PM

Herald added a project: Restricted Project. · View Herald TranscriptFeb 20 2023, 1:43 PM

Herald added a subscriber: cfe-commits. · View Herald Transcript

PiotrZSL added a reviewer: carlosgalvezp.Feb 21 2023, 2:45 PM

Ping, Rebase

Harbormaster completed remote builds in B218060: Diff 503312.Mar 8 2023, 5:05 AM

Rebase due to broken baseline

Harbormaster completed remote builds in B218096: Diff 503362.Mar 8 2023, 7:47 AM

PiotrZSL added a reviewer: LegalizeAdulthood.Mar 14 2023, 12:33 PM

LegalizeAdulthood resigned from this revision.Mar 29 2023, 8:05 AM

Ping, Rebase