diff options
| author | Mark Kettenis <kettenis@gnu.org> | 2004-11-18 22:37:22 +0000 |
|---|---|---|
| committer | Mark Kettenis <kettenis@gnu.org> | 2004-11-18 22:37:22 +0000 |
| commit | ceddaf06acf699346b5e2fc88e564dec52084262 (patch) | |
| tree | a120f6c005df0d2f083af94e4bfa2eb029f310bf | |
| parent | 915a2d21dd9c2838971b7800795fa6ece49baedc (diff) | |
| download | gdb-ceddaf06acf699346b5e2fc88e564dec52084262.zip gdb-ceddaf06acf699346b5e2fc88e564dec52084262.tar.gz gdb-ceddaf06acf699346b5e2fc88e564dec52084262.tar.bz2 | |
* dbxread.c (process_one_symbol): Do not adjust address of first
N_SLINE stab for a function for code generated by non-GCC
compilers.
| -rw-r--r-- | gdb/ChangeLog | 6 | ||||
| -rw-r--r-- | gdb/dbxread.c | 21 |
2 files changed, 24 insertions, 3 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 95fbd89..47a2481 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2004-11-18 Mark Kettenis <kettenis@gnu.org> + + * dbxread.c (process_one_symbol): Do not adjust address of first + N_SLINE stab for a function for code generated by non-GCC + compilers. + 2004-11-18 Kevin Buettner <kevinb@redhat.com> * solib-null.c: New file. diff --git a/gdb/dbxread.c b/gdb/dbxread.c index 9ac6a3c..21f4399 100644 --- a/gdb/dbxread.c +++ b/gdb/dbxread.c @@ -2927,11 +2927,26 @@ process_one_symbol (int type, int desc, CORE_ADDR valu, char *name, /* Relocate for dynamic loading and for ELF acc fn-relative syms. */ valu += function_start_offset; - /* If this is the first SLINE note in the function, record it at - the start of the function instead of at the listed location. */ + /* GCC 2.95.3 emits the first N_SLINE stab somwehere in the + middle of the prologue instead of right at the start of the + function. To deal with this we record the address for the + first N_SLINE stab to be the start of the function instead of + the listed location. We really shouldn't to this. When + compiling with optimization, this first N_SLINE stab might be + optimized away. Other (non-GCC) compilers don't emit this + stab at all. There is no real harm in having an extra + numbered line, although it can be a bit annoying for the + user. However, it totally screws up our testsuite. + + So for now, keep adjusting the address of the first N_SLINE + stab, but only for code compiled with GCC. */ + if (within_function && sline_found_in_function == 0) { - record_line (current_subfile, desc, last_function_start); + if (processing_gcc_compilation == 2) + record_line (current_subfile, desc, last_function_start); + else + record_line (current_subfile, desc, valu); sline_found_in_function = 1; } else |