1*67e74705SXin Li // RUN: %clang_cc1 -fsyntax-only -verify -std=c++11 -Wno-unused %s
2*67e74705SXin Li
3*67e74705SXin Li int f(int, int = 0);
4*67e74705SXin Li
5*67e74705SXin Li struct A {
6*67e74705SXin Li int x, y;
7*67e74705SXin Li };
8*67e74705SXin Li struct S {
9*67e74705SXin Li S(int, int);
10*67e74705SXin Li int n;
11*67e74705SXin Li };
12*67e74705SXin Li
test()13*67e74705SXin Li void test() {
14*67e74705SXin Li int a;
15*67e74705SXin Li int xs[10];
16*67e74705SXin Li ++a = 0; // ok
17*67e74705SXin Li a + ++a; // expected-warning {{unsequenced modification and access to 'a'}}
18*67e74705SXin Li a = ++a; // ok
19*67e74705SXin Li a + a++; // expected-warning {{unsequenced modification and access to 'a'}}
20*67e74705SXin Li a = a++; // expected-warning {{multiple unsequenced modifications to 'a'}}
21*67e74705SXin Li ++ ++a; // ok
22*67e74705SXin Li (a++, a++); // ok
23*67e74705SXin Li ++a + ++a; // expected-warning {{multiple unsequenced modifications to 'a'}}
24*67e74705SXin Li a++ + a++; // expected-warning {{multiple unsequenced modifications}}
25*67e74705SXin Li (a++, a) = 0; // ok, increment is sequenced before value computation of LHS
26*67e74705SXin Li a = xs[++a]; // ok
27*67e74705SXin Li a = xs[a++]; // expected-warning {{multiple unsequenced modifications}}
28*67e74705SXin Li (a ? xs[0] : xs[1]) = ++a; // expected-warning {{unsequenced modification and access}}
29*67e74705SXin Li a = (++a, ++a); // ok
30*67e74705SXin Li a = (a++, ++a); // ok
31*67e74705SXin Li a = (a++, a++); // expected-warning {{multiple unsequenced modifications}}
32*67e74705SXin Li f(a, a); // ok
33*67e74705SXin Li f(a = 0, a); // expected-warning {{unsequenced modification and access}}
34*67e74705SXin Li f(a, a += 0); // expected-warning {{unsequenced modification and access}}
35*67e74705SXin Li f(a = 0, a = 0); // expected-warning {{multiple unsequenced modifications}}
36*67e74705SXin Li a = f(++a); // ok
37*67e74705SXin Li a = f(a++); // ok
38*67e74705SXin Li a = f(++a, a++); // expected-warning {{multiple unsequenced modifications}}
39*67e74705SXin Li
40*67e74705SXin Li // Compound assignment "A OP= B" is equivalent to "A = A OP B" except that A
41*67e74705SXin Li // is evaluated only once.
42*67e74705SXin Li (++a, a) = 1; // ok
43*67e74705SXin Li (++a, a) += 1; // ok
44*67e74705SXin Li a = ++a; // ok
45*67e74705SXin Li a += ++a; // expected-warning {{unsequenced modification and access}}
46*67e74705SXin Li
47*67e74705SXin Li A agg1 = { a++, a++ }; // ok
48*67e74705SXin Li A agg2 = { a++ + a, a++ }; // expected-warning {{unsequenced modification and access}}
49*67e74705SXin Li
50*67e74705SXin Li S str1(a++, a++); // expected-warning {{multiple unsequenced modifications}}
51*67e74705SXin Li S str2 = { a++, a++ }; // ok
52*67e74705SXin Li S str3 = { a++ + a, a++ }; // expected-warning {{unsequenced modification and access}}
53*67e74705SXin Li
54*67e74705SXin Li struct Z { A a; S s; } z = { { ++a, ++a }, { ++a, ++a } }; // ok
55*67e74705SXin Li a = S { ++a, a++ }.n; // ok
56*67e74705SXin Li A { ++a, a++ }.x; // ok
57*67e74705SXin Li a = A { ++a, a++ }.x; // expected-warning {{unsequenced modifications}}
58*67e74705SXin Li A { ++a, a++ }.x + A { ++a, a++ }.y; // expected-warning {{unsequenced modifications}}
59*67e74705SXin Li
60*67e74705SXin Li (xs[2] && (a = 0)) + a; // ok
61*67e74705SXin Li (0 && (a = 0)) + a; // ok
62*67e74705SXin Li (1 && (a = 0)) + a; // expected-warning {{unsequenced modification and access}}
63*67e74705SXin Li
64*67e74705SXin Li (xs[3] || (a = 0)) + a; // ok
65*67e74705SXin Li (0 || (a = 0)) + a; // expected-warning {{unsequenced modification and access}}
66*67e74705SXin Li (1 || (a = 0)) + a; // ok
67*67e74705SXin Li
68*67e74705SXin Li (xs[4] ? a : ++a) + a; // ok
69*67e74705SXin Li (0 ? a : ++a) + a; // expected-warning {{unsequenced modification and access}}
70*67e74705SXin Li (1 ? a : ++a) + a; // ok
71*67e74705SXin Li (0 ? a : a++) + a; // expected-warning {{unsequenced modification and access}}
72*67e74705SXin Li (1 ? a : a++) + a; // ok
73*67e74705SXin Li (xs[5] ? ++a : ++a) + a; // FIXME: warn here
74*67e74705SXin Li
75*67e74705SXin Li (++a, xs[6] ? ++a : 0) + a; // expected-warning {{unsequenced modification and access}}
76*67e74705SXin Li
77*67e74705SXin Li // Here, the read of the fourth 'a' might happen before or after the write to
78*67e74705SXin Li // the second 'a'.
79*67e74705SXin Li a += (a++, a) + a; // expected-warning {{unsequenced modification and access}}
80*67e74705SXin Li
81*67e74705SXin Li int *p = xs;
82*67e74705SXin Li a = *(a++, p); // ok
83*67e74705SXin Li a = a++ && a; // ok
84*67e74705SXin Li
85*67e74705SXin Li A *q = &agg1;
86*67e74705SXin Li (q = &agg2)->y = q->x; // expected-warning {{unsequenced modification and access to 'q'}}
87*67e74705SXin Li
88*67e74705SXin Li // This has undefined behavior if a == 0; otherwise, the side-effect of the
89*67e74705SXin Li // increment is sequenced before the value computation of 'f(a, a)', which is
90*67e74705SXin Li // sequenced before the value computation of the '&&', which is sequenced
91*67e74705SXin Li // before the assignment. We treat the sequencing in '&&' as being
92*67e74705SXin Li // unconditional.
93*67e74705SXin Li a = a++ && f(a, a);
94*67e74705SXin Li
95*67e74705SXin Li // This has undefined behavior if a != 0. FIXME: We should diagnose this.
96*67e74705SXin Li (a && a++) + a;
97*67e74705SXin Li
98*67e74705SXin Li (xs[7] && ++a) * (!xs[7] && ++a); // ok
99*67e74705SXin Li
100*67e74705SXin Li xs[0] = (a = 1, a); // ok
101*67e74705SXin Li (a -= 128) &= 128; // ok
102*67e74705SXin Li ++a += 1; // ok
103*67e74705SXin Li
104*67e74705SXin Li xs[8] ? ++a + a++ : 0; // expected-warning {{multiple unsequenced modifications}}
105*67e74705SXin Li xs[8] ? 0 : ++a + a++; // expected-warning {{multiple unsequenced modifications}}
106*67e74705SXin Li xs[8] ? ++a : a++; // ok
107*67e74705SXin Li
108*67e74705SXin Li xs[8] && (++a + a++); // expected-warning {{multiple unsequenced modifications}}
109*67e74705SXin Li xs[8] || (++a + a++); // expected-warning {{multiple unsequenced modifications}}
110*67e74705SXin Li
111*67e74705SXin Li (__builtin_classify_type(++a) ? 1 : 0) + ++a; // ok
112*67e74705SXin Li (__builtin_constant_p(++a) ? 1 : 0) + ++a; // ok
113*67e74705SXin Li (__builtin_object_size(&(++a, a), 0) ? 1 : 0) + ++a; // ok
114*67e74705SXin Li (__builtin_expect(++a, 0) ? 1 : 0) + ++a; // expected-warning {{multiple unsequenced modifications}}
115*67e74705SXin Li }
116