[gdb/testsuite] Fix return type of psymtab-parameter
authorThomas Preud'homme <thomas.preudhomme@arm.com>
Wed, 29 Nov 2017 13:40:59 +0000 (13:40 +0000)
committerThomas Preud'homme <thomas.preudhomme@arm.com>
Wed, 29 Nov 2017 13:41:32 +0000 (13:41 +0000)
As pointed out by Pedro Alves, psymtab-parameter testcase rely on the
return type being long. This patch revert the changes made in
f106e10e5e80265e1c43532bba4cb997a7dfa022 and change psymtab-parameter.cc
to return 0 long instead.

2017-11-29  Thomas Preud'homme  <thomas.preudhomme@arm.com>

gdb/testsuite/
* gdb.cp/psymtab-parameter.cc (func): Change return type back to long.
Return 0 as a long.
* gdb.cp/psymtab-parameter.exp: Change func's return type back to long.

gdb/testsuite/ChangeLog
gdb/testsuite/gdb.cp/psymtab-parameter.cc
gdb/testsuite/gdb.cp/psymtab-parameter.exp

index 23d3cc2..87b917e 100644 (file)
@@ -1,5 +1,11 @@
 2017-11-29  Thomas Preud'homme  <thomas.preudhomme@arm.com>
 
+       * gdb.cp/psymtab-parameter.cc (func): Change return type back to long.
+       Return 0 as a long.
+       * gdb.cp/psymtab-parameter.exp: Change func's return type back to long.
+
+2017-11-29  Thomas Preud'homme  <thomas.preudhomme@arm.com>
+
        * gdb.cp/breakpoint.cc (bar): Set return type to void.
        * gdb.cp/psymtab-parameter.cc (func): Likewise.
        * gdb.cp/psymtab-parameter.exp: Update comment regarding prototype of
index 9f63fef..63038f0 100644 (file)
    along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 
 template <typename T>
-void
+long
 func ()
 {
+  return 0L;
 }
 
 void
index 5c4b6ac..3d78de9 100644 (file)
@@ -31,6 +31,6 @@ gdb_test_no_output "set language c++"
 # XFAIL than FAIL here.  For example -readnow breaks it.
 gdb_test_no_output "maintenance info symtabs"
 
-# GDB has shown only the `void func<short>()' ELF symbol before, not the DWARF
+# GDB has shown only the `long func<short>()' ELF symbol before, not the DWARF
 # symbol
 gdb_test "complete p 'func<short>(" "p 'func<short>\\(\\)"