xref: /aosp_15_r20/external/clang/test/Sema/warn-unsequenced.c (revision 67e74705e28f6214e480b399dd47ea732279e315)
1*67e74705SXin Li // RUN: %clang_cc1 -fsyntax-only -verify -std=c11 -Wno-unused %s
2*67e74705SXin Li 
3*67e74705SXin Li int f(int, int);
4*67e74705SXin Li 
5*67e74705SXin Li typedef struct A {
6*67e74705SXin Li   int x, y;
7*67e74705SXin Li } A;
8*67e74705SXin Li 
test()9*67e74705SXin Li void test() {
10*67e74705SXin Li   int a;
11*67e74705SXin Li   int xs[10];
12*67e74705SXin Li   a + ++a; // expected-warning {{unsequenced modification and access to 'a'}}
13*67e74705SXin Li   a = ++a; // expected-warning {{multiple unsequenced modifications to 'a'}}
14*67e74705SXin Li   a + a++; // expected-warning {{unsequenced modification and access to 'a'}}
15*67e74705SXin Li   a = a++; // expected-warning {{multiple unsequenced modifications to 'a'}}
16*67e74705SXin Li   (a++, a++); // ok
17*67e74705SXin Li   ++a + ++a; // expected-warning {{multiple unsequenced modifications}}
18*67e74705SXin Li   a++ + a++; // expected-warning {{multiple unsequenced modifications}}
19*67e74705SXin Li   a = xs[++a]; // expected-warning {{multiple unsequenced modifications}}
20*67e74705SXin Li   a = xs[a++]; // expected-warning {{multiple unsequenced modifications}}
21*67e74705SXin Li   a = (++a, ++a); // expected-warning {{multiple unsequenced modifications}}
22*67e74705SXin Li   a = (a++, ++a); // expected-warning {{multiple unsequenced modifications}}
23*67e74705SXin Li   a = (a++, a++); // expected-warning {{multiple unsequenced modifications}}
24*67e74705SXin Li   f(a, a); // ok
25*67e74705SXin Li   f(a = 0, a); // expected-warning {{unsequenced modification and access}}
26*67e74705SXin Li   f(a, a += 0); // expected-warning {{unsequenced modification and access}}
27*67e74705SXin Li   f(a = 0, a = 0); // expected-warning {{multiple unsequenced modifications}}
28*67e74705SXin Li   a = f(++a, 0); // ok
29*67e74705SXin Li   a = f(a++, 0); // ok
30*67e74705SXin Li   a = f(++a, a++); // expected-warning {{multiple unsequenced modifications}}
31*67e74705SXin Li 
32*67e74705SXin Li   ++a + f(++a, 0); // expected-warning {{multiple unsequenced modifications}}
33*67e74705SXin Li   f(++a, 0) + ++a; // expected-warning {{multiple unsequenced modifications}}
34*67e74705SXin Li   a++ + f(a++, 0); // expected-warning {{multiple unsequenced modifications}}
35*67e74705SXin Li   f(a++, 0) + a++; // expected-warning {{multiple unsequenced modifications}}
36*67e74705SXin Li 
37*67e74705SXin Li   a = ++a; // expected-warning {{multiple unsequenced modifications}}
38*67e74705SXin Li   a += ++a; // expected-warning {{unsequenced modification and access}}
39*67e74705SXin Li 
40*67e74705SXin Li   A agg1 = { a++, a++ }; // expected-warning {{multiple unsequenced modifications}}
41*67e74705SXin Li   A agg2 = { a++ + a, a++ }; // expected-warning {{unsequenced modification and access}}
42*67e74705SXin Li 
43*67e74705SXin Li   (xs[2] && (a = 0)) + a; // ok
44*67e74705SXin Li   (0 && (a = 0)) + a; // ok
45*67e74705SXin Li   (1 && (a = 0)) + a; // expected-warning {{unsequenced modification and access}}
46*67e74705SXin Li 
47*67e74705SXin Li   (xs[3] || (a = 0)) + a; // ok
48*67e74705SXin Li   (0 || (a = 0)) + a; // expected-warning {{unsequenced modification and access}}
49*67e74705SXin Li   (1 || (a = 0)) + a; // ok
50*67e74705SXin Li 
51*67e74705SXin Li   (xs[4] ? a : ++a) + a; // ok
52*67e74705SXin Li   (0 ? a : ++a) + a; // expected-warning {{unsequenced modification and access}}
53*67e74705SXin Li   (1 ? a : ++a) + a; // ok
54*67e74705SXin Li   (xs[5] ? ++a : ++a) + a; // FIXME: warn here
55*67e74705SXin Li 
56*67e74705SXin Li   (++a, xs[6] ? ++a : 0) + a; // expected-warning {{unsequenced modification and access}}
57*67e74705SXin Li 
58*67e74705SXin Li   // Here, the read of the fourth 'a' might happen before or after the write to
59*67e74705SXin Li   // the second 'a'.
60*67e74705SXin Li   a += (a++, a) + a; // expected-warning {{unsequenced modification and access}}
61*67e74705SXin Li 
62*67e74705SXin Li   int *p = xs;
63*67e74705SXin Li   a = *(a++, p); // ok
64*67e74705SXin Li   a = a++ && a; // ok
65*67e74705SXin Li 
66*67e74705SXin Li   A *q = &agg1;
67*67e74705SXin Li   (q = &agg2)->y = q->x; // expected-warning {{unsequenced modification and access to 'q'}}
68*67e74705SXin Li 
69*67e74705SXin Li   // This has undefined behavior if a == 0; otherwise, the side-effect of the
70*67e74705SXin Li   // increment is sequenced before the value computation of 'f(a, a)', which is
71*67e74705SXin Li   // sequenced before the value computation of the '&&', which is sequenced
72*67e74705SXin Li   // before the assignment. We treat the sequencing in '&&' as being
73*67e74705SXin Li   // unconditional.
74*67e74705SXin Li   a = a++ && f(a, a);
75*67e74705SXin Li 
76*67e74705SXin Li   // This has undefined behavior if a != 0. FIXME: We should diagnose this.
77*67e74705SXin Li   (a && a++) + a;
78*67e74705SXin Li 
79*67e74705SXin Li   (xs[7] && ++a) * (!xs[7] && ++a); // ok
80*67e74705SXin Li 
81*67e74705SXin Li   xs[0] = (a = 1, a); // ok
82*67e74705SXin Li 
83*67e74705SXin Li   xs[8] ? ++a + a++ : 0; // expected-warning {{multiple unsequenced modifications}}
84*67e74705SXin Li   xs[8] ? 0 : ++a + a++; // expected-warning {{multiple unsequenced modifications}}
85*67e74705SXin Li   xs[8] ? ++a : a++; // ok
86*67e74705SXin Li 
87*67e74705SXin Li   xs[8] && (++a + a++); // expected-warning {{multiple unsequenced modifications}}
88*67e74705SXin Li   xs[8] || (++a + a++); // expected-warning {{multiple unsequenced modifications}}
89*67e74705SXin Li 
90*67e74705SXin Li   (__builtin_classify_type(++a) ? 1 : 0) + ++a; // ok
91*67e74705SXin Li   (__builtin_constant_p(++a) ? 1 : 0) + ++a; // ok
92*67e74705SXin Li   (__builtin_expect(++a, 0) ? 1 : 0) + ++a; // expected-warning {{multiple unsequenced modifications}}
93*67e74705SXin Li   _Generic(++a, default: 0) + ++a; // ok
94*67e74705SXin Li   sizeof(++a) + ++a; // ok
95*67e74705SXin Li   _Alignof(++a) + ++a; // expected-warning {{extension}}
96*67e74705SXin Li }
97