re PR c++/58950 (Missing "statement has no effect")

2014-01-03  Marc Glisse  <marc.glisse@inria.fr>

	PR c++/58950
gcc/cp/
	* cvt.c (convert_to_void): Handle VEC_PERM_EXPR and VEC_COND_EXPR.
gcc/testsuite/
	* g++.dg/pr58950.C: New file.

From-SVN: r206325

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index 7021108..1f5ddb5 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,8 @@
+2014-01-03  Marc Glisse  <marc.glisse@inria.fr>
+
+       PR c++/58950
+       * cvt.c (convert_to_void): Handle VEC_PERM_EXPR and VEC_COND_EXPR.
+
 2014-01-03  Tobias Burnus  <burnus@net-b.de>
 
        PR c++/58567

diff --git a/gcc/cp/cvt.c b/gcc/cp/cvt.c
index 95105cd..df208cc 100644 (file)
--- a/gcc/cp/cvt.c
+++ b/gcc/cp/cvt.c
@@ -1403,7 +1403,9 @@ convert_to_void (tree expr, impl_conv_void implicit, tsubst_flags_t complain)
                            || code == PREDECREMENT_EXPR
                            || code == PREINCREMENT_EXPR
                            || code == POSTDECREMENT_EXPR
-                           || code == POSTINCREMENT_EXPR)))
+                           || code == POSTINCREMENT_EXPR))
+                  || code == VEC_PERM_EXPR
+                  || code == VEC_COND_EXPR)
                   && (complain & tf_warning))
                warning_at (loc, OPT_Wunused_value, "value computed is not used");
            }

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index ebf4034..517c724 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2014-01-03  Marc Glisse  <marc.glisse@inria.fr>
+
+       PR c++/58950
+       * g++.dg/pr58950.C: New file.
+
 2014-01-03  Tobias Burnus  <burnus@net-b.de>
 
        PR c++/58567

diff --git a/gcc/testsuite/g++.dg/pr58950.C b/gcc/testsuite/g++.dg/pr58950.C
new file mode 100644 (file)
index 0000000..10a2032
--- /dev/null
+++ b/gcc/testsuite/g++.dg/pr58950.C
@@ -0,0 +1,8 @@
+/* { dg-do compile } */
+/* { dg-options "-Wall" } */
+
+void f(){
+  int i __attribute__((vector_size(2*sizeof(int)))) = { 2, 3 };
+  __builtin_shuffle (i, i); /* { dg-warning "value computed is not used" } */
+  ++i?1:0; /* { dg-warning "value computed is not used" } */
+}