Catch trivially true statements of the form a != 1 || a != 3. Statements like these are likely errors. They are particularly easy to miss when handling enums: enum State { RUNNING, STOPPED, STARTING, ENDING } ... if (state != RUNNING || state !=...

[watsond]

Summary

...ENDING)
...

[clang-tidy] Catch trivially true statements like a != 1 || a != 3

[watsond]

Updating D29857: Catch trivially true statements of the form a != 1 || a != 3. Statements like

these are likely errors. They are particularly easy to miss when handling enums:

enum State {
RUNNING,
STOPPED,
STARTING,
ENDING
}

...
if (state != RUNNING || state !=...

[watsond]

Apologies all. I messed up the formatting. Resubmitting in a second.

[etienneb]

which one is the final version?

https://reviews.llvm.org/D25946

Please close one of them.

clang-tidy/misc/RedundantExpressionCheck.cpp

if (OpcodeLHS == BO_LE && OpcodeRHS == BO_GE &&
       APSInt::compareValues(ValueLHS_plus1, ValueRHS) == 0)
     return true;
 
   // Handle cases where the constants are different: x > 4 || x <= 7.
   if ((OpcodeLHS == BO_GT || OpcodeLHS == BO_GE) &&
       (OpcodeRHS == BO_LT || OpcodeRHS == BO_LE))
     return true;
 
+  // Handle cases where constants are different but both ops are !=, like:
+  // x != 5 || x != 10
+  if (OpcodeLHS == BO_NE || OpcodeLHS == BO_NE)
+    return true;
+
   return false;
 }
 
 static bool rangeSubsumesRange(BinaryOperatorKind OpcodeLHS,
                                const APSInt &ValueLHS,
                                BinaryOperatorKind OpcodeRHS,
                                const APSInt &ValueRHS) {
   int Comparison = APSInt::compareValues(ValueLHS, ValueRHS);

test/clang-tidy/misc-redundant-expression.cpp

int TestRelational(int X, int Y) {
   // CHECK-MESSAGES: :[[@LINE-1]]:14: warning: logical expression is always false
 
   if (X < 10 || X >= 10) return 1;
   // CHECK-MESSAGES: :[[@LINE-1]]:14: warning: logical expression is always true
   if (X <= 10 || X > 10) return 1;
   // CHECK-MESSAGES: :[[@LINE-1]]:15: warning: logical expression is always true
   if (X <= 10 || X >= 11) return 1;
   // CHECK-MESSAGES: :[[@LINE-1]]:15: warning: logical expression is always true
+  if (X != 7 || X != 14) return 1;
+  // CHECK-MESSAGES: :[[@LINE-1]]:14: warning: logical expression is always true
 
   if (X < 7 && X < 6) return 1;
   // CHECK-MESSAGES: :[[@LINE-1]]:9: warning: expression is redundant
   if (X < 7 && X < 7) return 1;
   // CHECK-MESSAGES: :[[@LINE-1]]:13: warning: both side of operator are equivalent
   if (X < 7 && X < 8) return 1;
   // CHECK-MESSAGES: :[[@LINE-1]]:18: warning: expression is redundant