Fix -Wreturn-type fallout.
authorMartin Liska <mliska@suse.cz>
Mon, 6 Nov 2017 13:41:35 +0000 (14:41 +0100)
committerMartin Liska <marxin@gcc.gnu.org>
Mon, 6 Nov 2017 13:41:35 +0000 (13:41 +0000)
2017-11-06  Martin Liska  <mliska@suse.cz>

* c-c++-common/cilk-plus/AN/pr57541-2.c (foo1): Return a value
for functions with non-void return type, or change type to void,
or add -Wno-return-type for test.
(foo2): Likewise.
* c-c++-common/cilk-plus/AN/pr57541.c (foo): Likewise.
(foo1): Likewise.
* c-c++-common/cilk-plus/CK/errors.c: Likewise.
* c-c++-common/cilk-plus/CK/pr60197.c: Likewise.
* c-c++-common/cilk-plus/CK/spawn_in_return.c: Likewise.
* c-c++-common/fold-masked-cmp-1.c (test_pic): Likewise.
(test_exe): Likewise.
* c-c++-common/fold-masked-cmp-2.c (test_exe): Likewise.
* g++.dg/cilk-plus/AN/builtin_fn_mutating_tplt.cc (my_func): Likewise.
* g++.dg/cilk-plus/CK/pr68997.cc (fa2): Likewise.
* g++.dg/eh/sighandle.C (dosegv): Likewise.
* g++.dg/ext/vector14.C (foo): Likewise.
(main): Likewise.
* g++.dg/graphite/pr41305.C: Likewise.
* g++.dg/graphite/pr42930.C: Likewise.
* g++.dg/opt/pr46640.C (struct QBasicAtomicInt): Likewise.
(makeDir): Likewise.
* g++.dg/other/i386-8.C (foo): Likewise.
* g++.dg/pr45788.C: Likewise.
* g++.dg/pr64688.C (at_c): Likewise.
* g++.dg/pr65032.C (G::DecodeVorbis): Likewise.
* g++.dg/pr71633.C (c3::fn2): Likewise.
* g++.dg/stackprotectexplicit2.C (A): Likewise.
* g++.old-deja/g++.law/weak.C (main): Likewise.
2017-11-06  Martin Liska  <mliska@suse.cz>

* testsuite/libgomp.c++/loop-2.C: Return a value
for functions with non-void return type, or change type to void,
or add -Wno-return-type for test.
* testsuite/libgomp.c++/loop-4.C: Likewise.
* testsuite/libgomp.c++/parallel-1.C: Likewise.
* testsuite/libgomp.c++/shared-1.C: Likewise.
* testsuite/libgomp.c++/single-1.C: Likewise.
* testsuite/libgomp.c++/single-2.C: Likewise.
2017-11-06  Martin Liska  <mliska@suse.cz>

* testsuite/27_io/basic_fstream/cons/char/path.cc (main):
  Return a value for functions with non-void return type,
  or change type to void, or add -Wno-return-type for test.
* testsuite/27_io/basic_ifstream/cons/char/path.cc (main):
Likewise.
* testsuite/27_io/basic_ofstream/open/char/path.cc (main):
Likewise.

From-SVN: r254451

33 files changed:
gcc/testsuite/ChangeLog
gcc/testsuite/c-c++-common/cilk-plus/AN/pr57541-2.c
gcc/testsuite/c-c++-common/cilk-plus/AN/pr57541.c
gcc/testsuite/c-c++-common/cilk-plus/CK/errors.c
gcc/testsuite/c-c++-common/cilk-plus/CK/pr60197.c
gcc/testsuite/c-c++-common/cilk-plus/CK/spawn_in_return.c
gcc/testsuite/c-c++-common/fold-masked-cmp-1.c
gcc/testsuite/c-c++-common/fold-masked-cmp-2.c
gcc/testsuite/g++.dg/cilk-plus/AN/builtin_fn_mutating_tplt.cc
gcc/testsuite/g++.dg/cilk-plus/CK/pr68997.cc
gcc/testsuite/g++.dg/eh/sighandle.C
gcc/testsuite/g++.dg/ext/vector14.C
gcc/testsuite/g++.dg/graphite/pr41305.C
gcc/testsuite/g++.dg/graphite/pr42930.C
gcc/testsuite/g++.dg/opt/pr46640.C
gcc/testsuite/g++.dg/other/i386-8.C
gcc/testsuite/g++.dg/pr45788.C
gcc/testsuite/g++.dg/pr64688.C
gcc/testsuite/g++.dg/pr65032.C
gcc/testsuite/g++.dg/pr71633.C
gcc/testsuite/g++.dg/stackprotectexplicit2.C
gcc/testsuite/g++.old-deja/g++.law/weak.C
libgomp/ChangeLog
libgomp/testsuite/libgomp.c++/loop-2.C
libgomp/testsuite/libgomp.c++/loop-4.C
libgomp/testsuite/libgomp.c++/parallel-1.C
libgomp/testsuite/libgomp.c++/shared-1.C
libgomp/testsuite/libgomp.c++/single-1.C
libgomp/testsuite/libgomp.c++/single-2.C
libstdc++-v3/ChangeLog
libstdc++-v3/testsuite/27_io/basic_fstream/cons/char/path.cc
libstdc++-v3/testsuite/27_io/basic_ifstream/cons/char/path.cc
libstdc++-v3/testsuite/27_io/basic_ofstream/open/char/path.cc

index 1c3a2fa2abb5ae0c379939708da2788860bd1624..1d5ba34ff9fc06e1d3a744440da5b1430baf3569 100644 (file)
@@ -1,3 +1,34 @@
+2017-11-06  Martin Liska  <mliska@suse.cz>
+
+       * c-c++-common/cilk-plus/AN/pr57541-2.c (foo1): Return a value
+       for functions with non-void return type, or change type to void,
+       or add -Wno-return-type for test.
+       (foo2): Likewise.
+       * c-c++-common/cilk-plus/AN/pr57541.c (foo): Likewise.
+       (foo1): Likewise.
+       * c-c++-common/cilk-plus/CK/errors.c: Likewise.
+       * c-c++-common/cilk-plus/CK/pr60197.c: Likewise.
+       * c-c++-common/cilk-plus/CK/spawn_in_return.c: Likewise.
+       * c-c++-common/fold-masked-cmp-1.c (test_pic): Likewise.
+       (test_exe): Likewise.
+       * c-c++-common/fold-masked-cmp-2.c (test_exe): Likewise.
+       * g++.dg/cilk-plus/AN/builtin_fn_mutating_tplt.cc (my_func): Likewise.
+       * g++.dg/cilk-plus/CK/pr68997.cc (fa2): Likewise.
+       * g++.dg/eh/sighandle.C (dosegv): Likewise.
+       * g++.dg/ext/vector14.C (foo): Likewise.
+       (main): Likewise.
+       * g++.dg/graphite/pr41305.C: Likewise.
+       * g++.dg/graphite/pr42930.C: Likewise.
+       * g++.dg/opt/pr46640.C (struct QBasicAtomicInt): Likewise.
+       (makeDir): Likewise.
+       * g++.dg/other/i386-8.C (foo): Likewise.
+       * g++.dg/pr45788.C: Likewise.
+       * g++.dg/pr64688.C (at_c): Likewise.
+       * g++.dg/pr65032.C (G::DecodeVorbis): Likewise.
+       * g++.dg/pr71633.C (c3::fn2): Likewise.
+       * g++.dg/stackprotectexplicit2.C (A): Likewise.
+       * g++.old-deja/g++.law/weak.C (main): Likewise.
+
 2017-11-06  Eric Botcazou  <ebotcazou@adacore.com>
 
        * gnat.dg/gcov: New directory.
index 83325a77501d9bed2a5e4c98c8931deb12c5315c..89a3d57ebdd129aab2f638c10393be07a946d5b3 100644 (file)
@@ -2,13 +2,13 @@
 /* { dg-do compile } */
 /* { dg-options "-fcilkplus" } */
 
-int foo1 ()
+void foo1 ()
 {
   int a;
   a = __sec_reduce_add (1); /* { dg-error "Invalid builtin arguments" } */
 }
 
-int foo2 ()
+void foo2 ()
 {
   int a;
   a = __sec_reduce_add (); /* { dg-error "Invalid builtin arguments" } */
index a956d0e18ab63d1ea4020dafc7c31e7bd9806712..b47de1e7ebccd9a3611edc16669366d575fdd94b 100755 (executable)
@@ -4,8 +4,7 @@
 
 int A[10];
 
-int foo () {
-
+void foo () {
   /* C compiler uses the term "undeclared" whereas C++ compiler uses
     "not declared".  Thus, grepping for declared seem to be the easiest.  */
   char c = (char)N; /* { dg-error "declared" } */
@@ -15,12 +14,8 @@ int foo () {
   A[l:s:c];
 }
 
-int foo1 (int N) {
-
+void foo1 (int N) {
   char c = (char)N;
   short s = (short)N;
   A[l:s:c]; /* { dg-error "declared" } */
 }
-
-
-
index b1e336e332843fa7b2abab99ff16a0620feea13a..d637924e9e1596775326391c5635c1f8075d059c 100644 (file)
@@ -1,5 +1,5 @@
 /* { dg-do compile } */
-/* { dg-options "-fcilkplus" } */
+/* { dg-options "-fcilkplus -Wno-return-type" } */
 
 int func_2(void);
 
index 2b47d1efb7dc4d2b0594ca120f51e97f4fdf40f2..301a6f2e9d12c1587fc865283349e145f552cd6a 100644 (file)
@@ -1,6 +1,6 @@
 /* PR c/60197 */
 /* { dg-do compile } */
-/* { dg-options "-fcilkplus" } */
+/* { dg-options "-fcilkplus -Wno-return-type" } */
 
 extern int foo (void);
 extern int bar (int);
index 14b7eef1276a1718b0738413b64b2b58659cb7de..602971e02c9c021c24e76dd103de80edcaa0c872 100644 (file)
@@ -1,5 +1,5 @@
 /* { dg-do compile } */
-/* { dg-options "-fcilkplus" } */
+/* { dg-options "-fcilkplus -Wno-return-type" } */
 
 int main (void)
 {
index c56adc4d662e51e927429e1371ae7fb2fb6c5bbc..658cc092995369a474bcc043db7eeae1558becf8 100644 (file)
@@ -29,13 +29,15 @@ int result;
 void test_pic (struct bfd_link_info *info)
 {
   if (bfd_link_pic (info))
-    result++;
+    result++;  
 }
 
 int test_exe (struct bfd_link_info *info)
 {
   if (bfd_link_executable (info))
     result++;
+
+  return 0;
 }
 
 /* { dg-final { scan-assembler-times "testn?b" 2 } } */
index a14bceb4566eb167e1a23d08aecbd95da006ba49..c95cc56dfa9acd8380cbc8b558d5c2d57aee4433 100644 (file)
@@ -36,6 +36,8 @@ int test_exe (struct bfd_link_info *info)
 {
   if (bfd_link_executable (info))
     result++;
+
+  return 0;
 }
 
 /* { dg-final { scan-assembler-times "testn?b" 2 } } */
index db81912cbe530e296a4f51f98905788547d5da04..111a2a29686188d55b8b446ef2bfb43032f7ee4c 100644 (file)
@@ -15,6 +15,8 @@ T my_func (T *x, T y)
     *x = y;
   else
     *x = *x;
+
+  return T();
 }
 
 template <class T> T my_func (T *x, T y);
index b442bf9cc2426b9cca2930d27c31c2d355fee530..a9a8a51ce7e8000679bdadca68240b317118cff4 100644 (file)
@@ -16,7 +16,7 @@ struct A2 {
 };
 
 A2 fa2 () {
-  A2 ();
+  return A2 ();
 }
 
 struct B1 {
index 5c4995e7677611fad56e381bd5a8edb6746396e4..c2200024687efc5acb6529cd150c859e14ab6937 100644 (file)
@@ -12,6 +12,7 @@ void sighandler (int signo, siginfo_t * si, void * uc)
 char * dosegv ()
 {    
   * ((volatile int *)0) = 12;
+  return 0;
 }
 
 int main ()
index 8e792108fb861d836760a37a8d6328a1951a9a47..eecff39de471c04a3f9720911a97695f9426c297 100644 (file)
@@ -8,7 +8,11 @@
 
 #define vector __attribute__((vector_size(16)))
 
-template<int N> vector signed int foo (vector float value) {}
+template<int N> vector signed int foo (vector float value)
+{
+  vector signed int a;
+  return a;
+}
 
 template<int> void foo (float) {}
 
@@ -19,4 +23,6 @@ main ()
   float f;
   foo<1> (v);
   foo<1> (f);
+
+  return 0;
 }
index 6a30b0e9e39855532a4b16fe662ab8c6d3f7caa0..756b1267288288aff036c0476d885fb89fb98871 100644 (file)
@@ -1,5 +1,5 @@
 // { dg-do compile }
-// { dg-options "-O3 -floop-interchange -Wno-conversion-null" }
+// { dg-options "-O3 -floop-interchange -Wno-conversion-null -Wno-return-type" }
 
 void __throw_bad_alloc ();
 
index c1150ce935317e74f5167334b79a81e20f70de11..e569cea0177a2301fa8a586063a2e535d6f0b0b8 100644 (file)
@@ -1,4 +1,4 @@
-/* { dg-options "-O1 -floop-block" } */
+/* { dg-options "-O1 -floop-block -Wno-return-type" } */
 
 typedef unsigned char byte;
 typedef unsigned int uint;
index 1c25778e8da530635206c0b9e013dd52864bd09f..b55afd4937e883a8253026b2e67fd85b5c8dbb8c 100644 (file)
@@ -7,6 +7,7 @@ struct QBasicAtomicInt
   bool deref ()
   {
     asm volatile ("":"=m" (i), "=qm" (j));
+    return true;
   }
 };
 
@@ -41,4 +42,6 @@ bool makeDir (unsigned len)
         return false;
       i = pos;
     }
+
+  return true;
 }
index cf833a524de6e2f4f630f8ed6580f3a768035a37..a9465ef12bc319f845135ec12d14bfe4a3091a6e 100644 (file)
@@ -19,4 +19,6 @@ foo (float *x, short *y)
   __m64 c = _mm_cvtps_pi16 (b);
   __builtin_memcpy (y, &c, sizeof (short) * 4);
   y[0] = bar (y[0]);
+
+  return 0;
 }
index 0f4db20c7693bde61163fc1d826649968c49c244..9148585a7f6049260d38061d5192d9deb0315594 100644 (file)
@@ -1,5 +1,5 @@
 // { dg-do compile { target i?86-*-* x86_64-*-* } }
-// { dg-options "-O3 -fwhole-program -msse2" }
+// { dg-options "-O3 -fwhole-program -msse2 -Wno-return-type" }
 
 typedef long unsigned int __darwin_size_t;
 typedef __darwin_size_t size_t;
index 3525e49b735bdf08a27b750932975daa952a0260..bf85f1abb45a77e22173c78b2a51d4303b693dee 100644 (file)
@@ -24,7 +24,10 @@ template <typename> struct F;
 template <typename> struct G;
 template <typename, typename, int> struct H;
 template <typename Element, typename Layout> struct H<Element, Layout, 3> {};
-template <int, typename E, typename L, int N> unsigned char at_c(H<E, L, N>) {}
+template <int, typename E, typename L, int N> unsigned char at_c(H<E, L, N>)
+{
+  return 0;
+}
 template <typename> class I;
 template <typename> class J;
 template <typename> class K;
index a62f50b49bf5292a45cd0c774bffc40c43089e20..d6b6768d25a680024f4b4543931804050cb5390b 100644 (file)
@@ -84,4 +84,6 @@ G::DecodeVorbis (int *p1)
       mDecodedAudioFrames -= b;
       fn2 (b);
     }
+
+  return nsresult();
 }
index 48e9c9833ec63b0218932fface3e125082d1d61b..8852695b910a00c0e9eb6bca4fa3774cf0f8fbae 100644 (file)
@@ -20,6 +20,7 @@ class c3 : c1, c2
 
 int *c3::fn2 () const
 {
+  return 0;
 }
 
 int *c3::fn3 (int p) const
index 9cf9ab909c98f8a86655a3ad2322928d711a5402..35d9e886ccf7045c69ffa34e5186c4c42210baa6 100644 (file)
@@ -7,6 +7,7 @@ int A()
 {
        int A[23];
        char b[22];
+       return 0;
 }
 
 int __attribute__((stack_protect)) B()
@@ -24,4 +25,4 @@ int __attribute__((stack_protect)) c()
 }
 
 
-/* { dg-final { scan-assembler-times "stack_chk_fail" 2 } } */
\ No newline at end of file
+/* { dg-final { scan-assembler-times "stack_chk_fail" 2 } } */
index 49132adbeb3f47f60be82c52114477601c271460..52444e77946dc74752ce981d1bad452fc0c7becc 100644 (file)
 
 std::istream x (0);
 
+int
 main () {
   x.get();
   std::putc(0, 0);
   std::fgets(0, 0, 0); 
   x.get((char*) 0, 0);
+  return 0;
 }
 
index f68604ca424b95a905fccdbb1116f1f50f1d686b..7c0e808d587250092e12647b44ad29c70e1f0103 100644 (file)
@@ -1,3 +1,14 @@
+2017-11-06  Martin Liska  <mliska@suse.cz>
+
+       * testsuite/libgomp.c++/loop-2.C: Return a value
+       for functions with non-void return type, or change type to void,
+       or add -Wno-return-type for test.
+       * testsuite/libgomp.c++/loop-4.C: Likewise.
+       * testsuite/libgomp.c++/parallel-1.C: Likewise.
+       * testsuite/libgomp.c++/shared-1.C: Likewise.
+       * testsuite/libgomp.c++/single-1.C: Likewise.
+       * testsuite/libgomp.c++/single-2.C: Likewise.
+
 2017-10-31  Tom de Vries  <tom@codesourcery.com>
 
        * plugin/plugin-hsa.c (HSA_LOG): Remove semicolon after
index ea3dc588afd1a33a35c75077a62abd1eb26db6ea..77144b8e7c69f82fc4abd586319dca8bdcb585b7 100644 (file)
@@ -15,6 +15,7 @@ void parloop (int *a)
     a[i] = i + 3;
 }
 
+int
 main()
 {
   int i, a[N];
index 731f234502176f239305d240b1d377baeca475bb..a940854c637605f2edff97130d3ff2915e8aa97a 100644 (file)
@@ -1,5 +1,6 @@
 extern "C" void abort (void);
 
+int
 main()
 {
   int i, a;
index 3c93147132840faa7226b500b28ee2497ee8f9f6..ce338d0ddf9abab8bc720b2f58a0ad51f8bd8a8b 100644 (file)
@@ -8,6 +8,7 @@ foo (void)
   return 10;
 }
 
+int
 main ()
 {
   int A = 0;
index 334a553ce238c1ce0b00727e58eeb34f3fd52ca3..2f61daa12130411b9cd7a59107db5da20092f057 100644 (file)
@@ -53,6 +53,7 @@ parallel (int a, int b)
     abort ();
 }
 
+int
 main()
 {
   parallel (1, 2);
index e318a48ca5ccc10d6d5fb316c05fd03a0780b690..221236f24f2ea75f7dec1b6655c50d951d2aa3ad 100644 (file)
@@ -1,5 +1,6 @@
 extern "C" void abort (void);
 
+int
 main()
 {
   int i = 0;
index c2dd228568dce187e88fbd3b3bfde0442539ecef..d24b1d85e667c502524c13fa7534d61d22e927ae 100644 (file)
@@ -7,6 +7,7 @@ struct X
   int c;
 };
 
+int
 main()
 {
   int i = 0;
index 70061e9377956301d5ccb625fa9978e36876dccc..a9a19a4871a9aa7fa8beb0e7aeac3738790c1d7e 100644 (file)
@@ -1,3 +1,13 @@
+2017-11-06  Martin Liska  <mliska@suse.cz>
+
+       * testsuite/27_io/basic_fstream/cons/char/path.cc (main):
+         Return a value for functions with non-void return type,
+         or change type to void, or add -Wno-return-type for test.
+       * testsuite/27_io/basic_ifstream/cons/char/path.cc (main):
+       Likewise.
+       * testsuite/27_io/basic_ofstream/open/char/path.cc (main):
+       Likewise.
+
 2017-11-06  Paolo Carlini  <paolo.carlini@oracle.com>
 
        * testsuite/20_util/optional/cons/deduction.cc: Avoid -Wreturn-type
index 4442c28c56b3b6e3c08c45d584a4403127d47208..51337ebd909237b3bdfbd7331ec93c49fde0dc0c 100644 (file)
@@ -45,4 +45,5 @@ main()
 {
   test01();
   test02();
+  return 0;
 }
index 24286f5eeaf9ab31590e9d015c433f1dda87af38..a0de4ba55b2c3ee1ac9dfe042ad2532f6456ae33 100644 (file)
@@ -45,4 +45,5 @@ main()
 {
   test01();
   test02();
+  return 0;
 }
index a3fc0c7ff68e1315bd2b9cc498c6e78aa85a83af..38078c97ef75f7fdd95ed20b642102b3bbafc422 100644 (file)
@@ -47,4 +47,5 @@ main()
 {
   test01();
   test02();
+  return 0;
 }