public inbox for gdb-prs@sourceware.org
help / color / mirror / Atom feed
* [Bug symtab/13795] New: .gdb_index misses fortran calling convention setting?
@ 2012-03-02 17:02 dje at google dot com
2021-02-23 18:57 ` [Bug symtab/13795] .gdb_index misses main name tromey at sourceware dot org
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: dje at google dot com @ 2012-03-02 17:02 UTC (permalink / raw)
To: gdb-prs
http://sourceware.org/bugzilla/show_bug.cgi?id=13795
Bug #: 13795
Summary: .gdb_index misses fortran calling convention setting?
Product: gdb
Version: HEAD
Status: NEW
Severity: minor
Priority: P2
Component: symtab
AssignedTo: unassigned@sourceware.org
ReportedBy: dje@google.com
Classification: Unclassified
Found this while reading the code.
Filing it so it's not forgotten.
read_partial_die has this:
case DW_AT_calling_convention:
/* DWARF doesn't provide a way to identify a program's source-level
entry point. DW_AT_calling_convention attributes are only meant
to describe functions' calling conventions.
However, because it's a necessary piece of information in
Fortran, and because DW_CC_program is the only piece of debugging
information whose definition refers to a 'main program' at all,
several compilers have begun marking Fortran main programs with
DW_CC_program --- even when those functions use the standard
calling conventions.
So until DWARF specifies a way to provide this information and
compilers pick up the new representation, we'll support this
practice. */
if (DW_UNSND (&attr) == DW_CC_program
&& cu->language == language_fortran)
{
set_main_name (part_die->name);
/* As this DIE has a static linkage the name would be difficult
to look up later. */
language_of_main = language_fortran;
}
break;
I couldn't see the corresponding support in .gdb_index handling.
--
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug symtab/13795] .gdb_index misses main name
2012-03-02 17:02 [Bug symtab/13795] New: .gdb_index misses fortran calling convention setting? dje at google dot com
@ 2021-02-23 18:57 ` tromey at sourceware dot org
2021-02-23 18:57 ` tromey at sourceware dot org
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: tromey at sourceware dot org @ 2021-02-23 18:57 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=13795
Tom Tromey <tromey at sourceware dot org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |tromey at sourceware dot org
Summary|.gdb_index misses fortran |.gdb_index misses main name
|calling convention setting? |
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug symtab/13795] .gdb_index misses main name
2012-03-02 17:02 [Bug symtab/13795] New: .gdb_index misses fortran calling convention setting? dje at google dot com
2021-02-23 18:57 ` [Bug symtab/13795] .gdb_index misses main name tromey at sourceware dot org
@ 2021-02-23 18:57 ` tromey at sourceware dot org
2021-03-28 16:00 ` tromey at sourceware dot org
2023-12-10 15:27 ` tromey at sourceware dot org
3 siblings, 0 replies; 5+ messages in thread
From: tromey at sourceware dot org @ 2021-02-23 18:57 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=13795
--- Comment #1 from Tom Tromey <tromey at sourceware dot org> ---
Updating this to reflect the lack of "main name" handling.
gdb still supports the calling convention hack, but there's
a DWARF way to achieve this as well now.
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug symtab/13795] .gdb_index misses main name
2012-03-02 17:02 [Bug symtab/13795] New: .gdb_index misses fortran calling convention setting? dje at google dot com
2021-02-23 18:57 ` [Bug symtab/13795] .gdb_index misses main name tromey at sourceware dot org
2021-02-23 18:57 ` tromey at sourceware dot org
@ 2021-03-28 16:00 ` tromey at sourceware dot org
2023-12-10 15:27 ` tromey at sourceware dot org
3 siblings, 0 replies; 5+ messages in thread
From: tromey at sourceware dot org @ 2021-03-28 16:00 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=13795
--- Comment #2 from Tom Tromey <tromey at sourceware dot org> ---
Created attachment 13332
--> https://sourceware.org/bugzilla/attachment.cgi?id=13332&action=edit
start of a patch
I started a patch to add this, but:
* It would require a change to gold as well;
* It would require a change to binutils for the dumping;
* Ideally, if we're bumping the version anyway, we'd add data
addresses to the address map (this would be useful for Rust);
* but, I feel that gdb index ought to be deprecated anyway,
so it's hard to motivate
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug symtab/13795] .gdb_index misses main name
2012-03-02 17:02 [Bug symtab/13795] New: .gdb_index misses fortran calling convention setting? dje at google dot com
` (2 preceding siblings ...)
2021-03-28 16:00 ` tromey at sourceware dot org
@ 2023-12-10 15:27 ` tromey at sourceware dot org
3 siblings, 0 replies; 5+ messages in thread
From: tromey at sourceware dot org @ 2023-12-10 15:27 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=13795
Tom Tromey <tromey at sourceware dot org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |FIXED
Status|NEW |RESOLVED
Target Milestone|--- |15.1
--- Comment #3 from Tom Tromey <tromey at sourceware dot org> ---
This was fixed by the commit in bug #24549.
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-12-10 15:27 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-03-02 17:02 [Bug symtab/13795] New: .gdb_index misses fortran calling convention setting? dje at google dot com
2021-02-23 18:57 ` [Bug symtab/13795] .gdb_index misses main name tromey at sourceware dot org
2021-02-23 18:57 ` tromey at sourceware dot org
2021-03-28 16:00 ` tromey at sourceware dot org
2023-12-10 15:27 ` tromey at sourceware dot org
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).