mirror of
https://github.com/bminor/binutils-gdb.git
synced 2025-12-05 15:15:42 +00:00
c7a45b98a6
Setting a BP on a line like this would incorrectly yield two BP locations:
01 void two () { {int var = 0;} }
(gdb) break 1
Breakpoint 1 at 0x1164: main.cpp:1. (2 locations)
(gdb) info breakpoints
Num Type Disp Enb Address What
1 breakpoint keep y <MULTIPLE>
1.1 y 0x0000000000001164 in two() at main.cpp:1
1.2 y 0x0000000000001164 in two() at main.cpp:1
In this case decode_digits_ordinary () returns two SALs, exactly matching the
requested line. One for the entry PC and one for the prologue end PC. This
was
tested with GCC, CLANG and ICPX. Subsequent code tries to skip the prologue
on these PCs, which in turn makes them the same.
To fix this, ignore SALs with the same PC and program space when adding to the
list of SALs.
This will then properly set only one location:
(gdb) break 1
Breakpoint 1 at 0x1164: file main.cpp, line 1
(gdb) info breakpoints
Num Type Disp Enb Address What
1 breakpoint keep y 0x0000000000001164 in two() at main.cpp:1
Approved-By: Simon Marchi <simon.marchi@efficios.com>
22 lines
323 B
C++
22 lines
323 B
C++
#include "lspec.h"
|
|
|
|
static int dupname (int x) { label: return x; }
|
|
|
|
int NameSpace::overload()
|
|
{
|
|
return 23;
|
|
}
|
|
|
|
int body_elsewhere()
|
|
{
|
|
int x = 5; /* body_elsewhere marker */
|
|
#include "body.h"
|
|
}
|
|
|
|
void no_multi_locs () { {int var = 0;} }
|
|
|
|
int main()
|
|
{
|
|
return dupname(0) + m(0) + n(0) + f1() + f2() + body_elsewhere();
|
|
}
|