From 9011945e46bf8ba4722de6cac73711863dc58b39 Mon Sep 17 00:00:00 2001 From: Andrew Burgess Date: Thu, 3 Feb 2011 14:45:37 +0000 Subject: [PATCH] http://sourceware.org/ml/gdb-patches/2010-12/msg00300.html Improve the ordering of line table entries when dealing with the end of sequence markers. This will allow us to disassemble over the border between two compile units while also displaying source code information. --- gdb/ChangeLog | 6 +++ gdb/disasm.c | 21 +++++++--- gdb/testsuite/ChangeLog | 6 +++ gdb/testsuite/gdb.base/disasm-end-cu-1.c | 28 ++++++++++++++ gdb/testsuite/gdb.base/disasm-end-cu-2.c | 28 ++++++++++++++ gdb/testsuite/gdb.base/disasm-end-cu.exp | 49 ++++++++++++++++++++++++ 6 files changed, 132 insertions(+), 6 deletions(-) create mode 100644 gdb/testsuite/gdb.base/disasm-end-cu-1.c create mode 100644 gdb/testsuite/gdb.base/disasm-end-cu-2.c create mode 100644 gdb/testsuite/gdb.base/disasm-end-cu.exp diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 1c467ea938..7e1511849a 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2011-02-03 Andrew Burgess + + * disasm.c (compare_lines): Handle the end of sequence markers + within the line table to better support disassembling over + compilation unit boundaries. + 2011-02-02 Ulrich Weigand * arm-tdep.c (skip_prologue_function): Add GDBARCH and IS_THUMB diff --git a/gdb/disasm.c b/gdb/disasm.c index 9180bc578e..ee19f70c03 100644 --- a/gdb/disasm.c +++ b/gdb/disasm.c @@ -77,12 +77,21 @@ compare_lines (const void *mle1p, const void *mle2p) mle1 = (struct dis_line_entry *) mle1p; mle2 = (struct dis_line_entry *) mle2p; - val = mle1->line - mle2->line; - - if (val != 0) - return val; - - return mle1->start_pc - mle2->start_pc; + /* End of sequence markers have a line number of 0 but don't want to + be sorted to the head of the list, instead sort by PC. */ + if (mle1->line == 0 || mle2->line == 0) + { + val = mle1->start_pc - mle2->start_pc; + if (val == 0) + val = mle1->line - mle2->line; + } + else + { + val = mle1->line - mle2->line; + if (val == 0) + val = mle1->start_pc - mle2->start_pc; + } + return val; } static int diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 0b6095c8d1..a287e4b862 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,9 @@ +2011-02-03 Andrew Burgess + + * gdb.base/disasm-end-cu-1.c, gdb.base/disasm-end-cu-2.c, + gdb.base/disasm-end-cu.exp: New test for disassembling over the + boundary between two compilation units. + 2011-02-02 Pedro Alves * gdb.cp/virtbase.cc (VirtualBase, VirtualMiddleA, VirtualMiddleB) diff --git a/gdb/testsuite/gdb.base/disasm-end-cu-1.c b/gdb/testsuite/gdb.base/disasm-end-cu-1.c new file mode 100644 index 0000000000..194b8b1ae3 --- /dev/null +++ b/gdb/testsuite/gdb.base/disasm-end-cu-1.c @@ -0,0 +1,28 @@ +/* This testcase is part of GDB, the GNU debugger. + + Copyright (C) 2010, 2011 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 . */ + +int +dummy_1 (int a, int b, int c) +{ + return a + b + c; +} + +int +main () +{ + return 0; +} diff --git a/gdb/testsuite/gdb.base/disasm-end-cu-2.c b/gdb/testsuite/gdb.base/disasm-end-cu-2.c new file mode 100644 index 0000000000..be716e7fe2 --- /dev/null +++ b/gdb/testsuite/gdb.base/disasm-end-cu-2.c @@ -0,0 +1,28 @@ +/* This testcase is part of GDB, the GNU debugger. + + Copyright (C) 2010, 2011 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 . */ + +int +dummy_2 (int a, int b, int c) +{ + return a + b + c; +} + +int +dummy_3 (int a, int b, int c) +{ + return a - b - c; +} diff --git a/gdb/testsuite/gdb.base/disasm-end-cu.exp b/gdb/testsuite/gdb.base/disasm-end-cu.exp new file mode 100644 index 0000000000..5184f11afb --- /dev/null +++ b/gdb/testsuite/gdb.base/disasm-end-cu.exp @@ -0,0 +1,49 @@ +# Copyright (C) 2010, 2011 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 . + +# This test tries to disassemble over the boundary between two compilation +# units displaying source lines. This checks that the disassemble routine +# can handle our use of line number 0 to mark the end of sequence. + +if { [prepare_for_testing disasm-end-cu.exp "disasm-end-cu" {disasm-end-cu-1.c disasm-end-cu-2.c} {debug}] } { + return -1 +} + +if ![runto_main] { + return -1 +} + +set main_addr [get_hexadecimal_valueof "&main" "0"] +set dummy_3_addr [get_hexadecimal_valueof "&dummy_3" "0"] + +if {$main_addr == 0 || $dummy_3_addr == 0 || $dummy_3_addr <= $main_addr} { + fail "Unable to extract required addresses, or addresses out of order" + return -1 +} + +gdb_test_multiple "disassemble /m ${main_addr},${dummy_3_addr}" "Disassemble address range with source" { + -re "Dump of assembler code from ${main_addr} to ${dummy_3_addr}:\r\nEnd of assembler dump\." { + fail "No output from the disassemble command" + } + -re "Line number 0 out of range;.* has $decimal lines\." { + fail "The disassemble command failed" + } + -re "Dump of assembler code from ${main_addr} to ${dummy_3_addr}:\r\n.*main.*End of assembler dump\." { + pass "disassemble command returned some output" + } + -re ".*$gdb_prompt $" { + fail "Unexpected output from disassemble command" + } +}