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" + } +}