This is the mail archive of the gdb-cvs@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[binutils-gdb] Add testcase for stub-method reading in stabs.


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=85317fbbca85af8786ba3ce0523a1b85b4457f09

commit 85317fbbca85af8786ba3ce0523a1b85b4457f09
Author: Doug Evans <xdje42@gmail.com>
Date:   Mon Apr 6 08:59:58 2015 -0700

    Add testcase for stub-method reading in stabs.
    
    This patch is based on the testcase provided here:
    https://sourceware.org/ml/gdb-patches/2015-02/msg00181.html
    
    I've verified that it catches the internal error discovered here:
    https://sourceware.org/ml/gdb-patches/2015-02/msg00139.html
    
    gdb/testsuite/ChangeLog:
    
    	* lib/gdb.exp (clean_restart): Return result of gdb_load.
    	* gdb.pascal/stub-method.exp: New file.
    	* gdb.pascal/stub-method.pas: New file.

Diff:
---
 gdb/testsuite/gdb.pascal/stub-method.exp | 30 +++++++++++++++++++++++++++++
 gdb/testsuite/gdb.pascal/stub-method.pas | 33 ++++++++++++++++++++++++++++++++
 gdb/testsuite/lib/gdb.exp                |  3 ++-
 3 files changed, 65 insertions(+), 1 deletion(-)

diff --git a/gdb/testsuite/gdb.pascal/stub-method.exp b/gdb/testsuite/gdb.pascal/stub-method.exp
new file mode 100644
index 0000000..f6be83e
--- /dev/null
+++ b/gdb/testsuite/gdb.pascal/stub-method.exp
@@ -0,0 +1,30 @@
+# Copyright 2007-2015 Free Software Foundation, Inc.
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+# Test loading of a file that when compiled with stabs uses a stub method.
+# This exercises gdbtypes.c:allocate_stub_method.
+
+load_lib "pascal.exp"
+
+standard_testfile .pas
+
+if {[gdb_compile_pascal "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable [list debug ]] != "" } {
+  return -1
+}
+
+# If this fails, the failure will have already been recorded.
+if { [clean_restart ${testfile}] == 0 } {
+    pass "loading file with stub method"
+}
diff --git a/gdb/testsuite/gdb.pascal/stub-method.pas b/gdb/testsuite/gdb.pascal/stub-method.pas
new file mode 100644
index 0000000..30c45fb
--- /dev/null
+++ b/gdb/testsuite/gdb.pascal/stub-method.pas
@@ -0,0 +1,33 @@
+{
+ Copyright 2015 Free Software Foundation, Inc.
+
+ This program is free software; you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published by
+ the Free Software Foundation; either version 3 of the License, or
+ (at your option) any later version.
+
+ This program is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ GNU General Public License for more details.
+
+ You should have received a copy of the GNU General Public License
+ along with this program.  If not, see <http://www.gnu.org/licenses/>.
+}
+
+type
+  tobj = object
+constructor create;
+end;
+
+constructor tobj.create;
+
+begin
+end;
+
+var
+   t :  tobj;
+
+begin
+   t.create;
+end.
diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp
index 1a576c0..e0a03ec 100644
--- a/gdb/testsuite/lib/gdb.exp
+++ b/gdb/testsuite/lib/gdb.exp
@@ -4711,6 +4711,7 @@ proc build_executable { testname executable {sources ""} {options {debug}} } {
 
 # Starts fresh GDB binary and loads EXECUTABLE into GDB. EXECUTABLE is
 # the basename of the binary.
+# The return value is 0 for success, -1 for failure.
 proc clean_restart { executable } {
     global srcdir
     global subdir
@@ -4719,7 +4720,7 @@ proc clean_restart { executable } {
     gdb_exit
     gdb_start
     gdb_reinitialize_dir $srcdir/$subdir
-    gdb_load ${binfile}
+    return [gdb_load ${binfile}]
 }
 
 # Prepares for testing by calling build_executable_full, then


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]