168 lines
6.5 KiB
Diff
168 lines
6.5 KiB
Diff
http://sourceware.org/ml/gdb-patches/2014-08/msg00025.html
|
|
Subject: [patch 1/2] Re: Crash regression(?) printing Fortran strings in bt [Re: [V2 00/23] Fortran dynamic array support]
|
|
|
|
|
|
--FCuugMFkClbJLl1L
|
|
Content-Type: text/plain; charset=us-ascii
|
|
Content-Disposition: inline
|
|
|
|
On Fri, 01 Aug 2014 09:20:19 +0200, Keven Boell wrote:
|
|
> I just tried it on Fedora 20 i686. Applied the patch, you mentioned, on top of
|
|
> the Fortran VLA series and executed your dynamic-other-frame test. Everything
|
|
> is working fine here, I cannot reproduce the crash.
|
|
|
|
I have it reproducible on Fedora 20 i686 with plain
|
|
CFLAGS=-g ./configure;make;cd gdb/testsuite;make site.exp;runtest gdb.fortran/dynamic-other-frame.exp
|
|
|
|
Besides that I have updated the testcase with
|
|
gdb_test_no_output "set print frame-arguments all"
|
|
so that there is no longer needed the patch:
|
|
[patch] Display Fortran strings in backtraces
|
|
https://sourceware.org/ml/gdb-patches/2014-07/msg00709.html
|
|
|
|
The fix below has no regressions for me. Unfortunately I do not see why you
|
|
cannot reproduce it.
|
|
|
|
|
|
Thanks,
|
|
Jan
|
|
|
|
--FCuugMFkClbJLl1L
|
|
Content-Type: text/plain; charset=us-ascii
|
|
Content-Disposition: inline; filename="vlastringonly.patch"
|
|
|
|
Index: gdb-7.9.50.20150520/gdb/gdbtypes.c
|
|
===================================================================
|
|
--- gdb-7.9.50.20150520.orig/gdb/gdbtypes.c 2015-05-31 16:14:04.164533833 +0200
|
|
+++ gdb-7.9.50.20150520/gdb/gdbtypes.c 2015-05-31 16:14:05.844545344 +0200
|
|
@@ -1851,6 +1851,7 @@ is_dynamic_type_internal (struct type *t
|
|
}
|
|
|
|
case TYPE_CODE_ARRAY:
|
|
+ case TYPE_CODE_STRING:
|
|
{
|
|
gdb_assert (TYPE_NFIELDS (type) == 1);
|
|
|
|
@@ -2192,6 +2193,7 @@ resolve_dynamic_type_internal (struct ty
|
|
}
|
|
|
|
case TYPE_CODE_ARRAY:
|
|
+ case TYPE_CODE_STRING:
|
|
resolved_type = resolve_dynamic_array (type, addr_stack);
|
|
break;
|
|
|
|
Index: gdb-7.9.50.20150520/gdb/testsuite/gdb.fortran/dynamic-other-frame-stub.f90
|
|
===================================================================
|
|
--- /dev/null 1970-01-01 00:00:00.000000000 +0000
|
|
+++ gdb-7.9.50.20150520/gdb/testsuite/gdb.fortran/dynamic-other-frame-stub.f90 2015-05-31 16:14:05.844545344 +0200
|
|
@@ -0,0 +1,24 @@
|
|
+! Copyright 2010 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 2 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, write to the Free Software
|
|
+! Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
|
|
+!
|
|
+! Ihis file is the Fortran source file for dynamic.exp.
|
|
+! Original file written by Jakub Jelinek <jakub@redhat.com>.
|
|
+! Modified for the GDB testcase by Jan Kratochvil <jan.kratochvil@redhat.com>.
|
|
+
|
|
+subroutine bar
|
|
+ real :: dummy
|
|
+ dummy = 1
|
|
+end subroutine bar
|
|
Index: gdb-7.9.50.20150520/gdb/testsuite/gdb.fortran/dynamic-other-frame.exp
|
|
===================================================================
|
|
--- /dev/null 1970-01-01 00:00:00.000000000 +0000
|
|
+++ gdb-7.9.50.20150520/gdb/testsuite/gdb.fortran/dynamic-other-frame.exp 2015-05-31 16:14:05.845545351 +0200
|
|
@@ -0,0 +1,39 @@
|
|
+# Copyright 2010 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 2 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, write to the Free Software
|
|
+# Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
|
|
+
|
|
+set testfile "dynamic-other-frame"
|
|
+set srcfile1 ${testfile}.f90
|
|
+set srcfile2 ${testfile}-stub.f90
|
|
+set objfile2 [standard_output_file ${testfile}-stub.o]
|
|
+set executable ${testfile}
|
|
+set binfile [standard_output_file ${executable}]
|
|
+
|
|
+if { [gdb_compile "${srcdir}/${subdir}/${srcfile2}" "${objfile2}" object {f90}] != ""
|
|
+ || [gdb_compile "${srcdir}/${subdir}/${srcfile1} ${objfile2}" "${binfile}" executable {debug f90}] != "" } {
|
|
+ untested "Couldn't compile ${srcfile1} or ${srcfile2}"
|
|
+ return -1
|
|
+}
|
|
+
|
|
+clean_restart ${executable}
|
|
+
|
|
+gdb_test_no_output "set print frame-arguments all"
|
|
+
|
|
+if ![runto bar_] then {
|
|
+ perror "couldn't run to bar_"
|
|
+ continue
|
|
+}
|
|
+
|
|
+gdb_test "bt" {foo \(string='hello'.*}
|
|
Index: gdb-7.9.50.20150520/gdb/testsuite/gdb.fortran/dynamic-other-frame.f90
|
|
===================================================================
|
|
--- /dev/null 1970-01-01 00:00:00.000000000 +0000
|
|
+++ gdb-7.9.50.20150520/gdb/testsuite/gdb.fortran/dynamic-other-frame.f90 2015-05-31 16:14:05.845545351 +0200
|
|
@@ -0,0 +1,36 @@
|
|
+! Copyright 2010 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 2 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, write to the Free Software
|
|
+! Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
|
|
+!
|
|
+! Ihis file is the Fortran source file for dynamic.exp.
|
|
+! Original file written by Jakub Jelinek <jakub@redhat.com>.
|
|
+! Modified for the GDB testcase by Jan Kratochvil <jan.kratochvil@redhat.com>.
|
|
+
|
|
+subroutine foo (string)
|
|
+ interface
|
|
+ subroutine bar
|
|
+ end subroutine
|
|
+ end interface
|
|
+ character string*(*)
|
|
+ call bar ! stop-here
|
|
+end subroutine foo
|
|
+program test
|
|
+ interface
|
|
+ subroutine foo (string)
|
|
+ character string*(*)
|
|
+ end subroutine
|
|
+ end interface
|
|
+ call foo ('hello')
|
|
+end
|