// RUN: %clang_cc1 -analyze -analyzer-store=region -analyzer-constraints=range -fblocks -analyzer-opt-analyze-nested-blocks -analyzer-checker=experimental.deadcode.IdempotentOperations -verify %s // Basic tests extern void test(int i); extern void test_f(float f); unsigned basic() { int x = 10, zero = 0, one = 1; // x op x x = x; // expected-warning {{Assigned value is always the same as the existing value}} test(x - x); // expected-warning {{Both operands to '-' always have the same value}} x -= x; // expected-warning {{Both operands to '-=' always have the same value}} x = 10; // no-warning test(x / x); // expected-warning {{Both operands to '/' always have the same value}} x /= x; // expected-warning {{Both operands to '/=' always have the same value}} x = 10; // no-warning test(x & x); // expected-warning {{Both operands to '&' always have the same value}} x &= x; // expected-warning {{Both operands to '&=' always have the same value}} test(x | x); // expected-warning {{Both operands to '|' always have the same value}} x |= x; // expected-warning {{Both operands to '|=' always have the same value}} // x op 1 test(x * one); // expected-warning {{The right operand to '*' is always 1}} x *= one; // expected-warning {{The right operand to '*=' is always 1}} test(x / one); // expected-warning {{The right operand to '/' is always 1}} x /= one; // expected-warning {{The right operand to '/=' is always 1}} // 1 op x test(one * x); // expected-warning {{The left operand to '*' is always 1}} // x op 0 test(x + zero); // expected-warning {{The right operand to '+' is always 0}} test(x - zero); // expected-warning {{The right operand to '-' is always 0}} test(x * zero); // expected-warning {{The right operand to '*' is always 0}} test(x & zero); // expected-warning {{The right operand to '&' is always 0}} test(x | zero); // expected-warning {{The right operand to '|' is always 0}} test(x ^ zero); // expected-warning {{The right operand to '^' is always 0}} test(x << zero); // expected-warning {{The right operand to '<<' is always 0}} test(x >> zero); // expected-warning {{The right operand to '>>' is always 0}} // 0 op x test(zero + x); // expected-warning {{The left operand to '+' is always 0}} test(zero - x); // expected-warning {{The left operand to '-' is always 0}} test(zero / x); // expected-warning {{The left operand to '/' is always 0}} test(zero * x); // expected-warning {{The left operand to '*' is always 0}} test(zero & x); // expected-warning {{The left operand to '&' is always 0}} test(zero | x); // expected-warning {{The left operand to '|' is always 0}} test(zero ^ x); // expected-warning {{The left operand to '^' is always 0}} test(zero << x); // expected-warning {{The left operand to '<<' is always 0}} test(zero >> x); // expected-warning {{The left operand to '>>' is always 0}} // Overwrite the values so these aren't marked as Pseudoconstants x = 1; zero = 2; one = 3; return x + zero + one; } void floats(float x) { test_f(x * 1.0); // no-warning test_f(x * 1.0F); // no-warning } // Ensure that we don't report false poitives in complex loops void bailout() { int unused = 0, result = 4; result = result; // expected-warning {{Assigned value is always the same as the existing value}} for (unsigned bg = 0; bg < 1024; bg ++) { result = bg * result; // no-warning for (int i = 0; i < 256; i++) { unused *= i; // no-warning } } } // Relaxed liveness - check that we don't kill liveness at assignments typedef unsigned uintptr_t; void kill_at_assign() { short array[2]; uintptr_t x = (uintptr_t) array; short *p = (short *) x; // The following branch should be infeasible. if (!(p = &array[0])) { // expected-warning{{Assigned value is always the same as the existing value}} p = 0; *p = 1; // no-warning } } // False positive tests unsigned false1() { int a = 10; return a * (5 - 2 - 3); // no-warning } enum testenum { enum1 = 0, enum2 }; unsigned false2() { int a = 1234; return enum1 + a; // no-warning } // Self assignments of unused variables are common false positives unsigned false3(int param, int param2) { param = param; // no-warning // if a self assigned variable is used later, then it should be reported still param2 = param2; // expected-warning{{Assigned value is always the same as the existing value}} unsigned nonparam = 5; nonparam = nonparam; // expected-warning{{Assigned value is always the same as the existing value}} return param2 + nonparam; } // Pseudo-constants (vars only read) and constants should not be reported unsigned false4() { // Trivial constant const int height = 1; int c = 42; test(height * c); // no-warning // Pseudo-constant (never changes after decl) int width = height; return width * 10; // no-warning } // Block pseudoconstants void false4a() { // Pseudo-constant __block int a = 1; int b = 10; __block int c = 0; b *= a; // no-warning ^{ // Psuedoconstant block var test(b * c); // no-warning // Non-pseudoconstant block var int d = 0; test(b * d); // expected-warning{{The right operand to '*' is always 0}} d = 5; test(d); }(); test(a + b); } // Static vars are common false positives int false5() { static int test = 0; int a = 56; a *= test; // no-warning test++; return a; } // Non-local storage vars are considered false positives int globalInt = 1; int false6() { int localInt = 23; localInt /= globalInt; return localInt; } // Check that assignments filter out false positives correctly int false7() { int zero = 0; // pseudo-constant int one = 1; int a = 55; a = a; // expected-warning{{Assigned value is always the same as the existing value}} a = enum1 * a; // no-warning int b = 123; b = b; // no-warning return a; } // Check truncations do not flag as self-assignments void false8() { int a = 10000000; a = (short)a; // no-warning test(a); } // This test case previously flagged a warning at 'b == c' because the // analyzer previously allowed 'UnknownVal' as the index for ElementRegions. typedef struct RDar8431728_F { int RDar8431728_A; unsigned char *RDar8431728_B; int RDar8431728_E[6]; } RDar8431728_D; static inline int RDar8431728_C(RDar8431728_D * s, int n, unsigned char **RDar8431728_B_ptr) { int xy, wrap, pred, a, b, c; xy = s->RDar8431728_E[n]; wrap = s->RDar8431728_A; a = s->RDar8431728_B[xy - 1]; b = s->RDar8431728_B[xy - 1 - wrap]; c = s->RDar8431728_B[xy - wrap]; if (b == c) { // no-warning pred = a; } else { pred = c; } *RDar8431728_B_ptr = &s->RDar8431728_B[xy]; return pred; } // <rdar://problem/8601243> - Don't warn on pointer arithmetic. This // is often idiomatic. unsigned rdar8601243_aux(unsigned n); void rdar8601243() { char arr[100]; char *start = arr; start = start + rdar8601243_aux(sizeof(arr) - (arr - start)); // no-warning (void) start; }