public inbox for gdb-prs@sourceware.org
help / color / mirror / Atom feed
* [Bug symtab/17684] New: lookup of builtin types: correctness and performance issues
@ 2014-12-07 17:56 xdje42 at gmail dot com
  2015-03-18 19:06 ` [Bug symtab/17684] " dje at google dot com
  0 siblings, 1 reply; 2+ messages in thread
From: xdje42 at gmail dot com @ 2014-12-07 17:56 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=17684

            Bug ID: 17684
           Summary: lookup of builtin types: correctness and performance
                    issues
           Product: gdb
           Version: HEAD
            Status: NEW
          Severity: normal
          Priority: P2
         Component: symtab
          Assignee: unassigned at sourceware dot org
          Reporter: xdje42 at gmail dot com

This bug is to address the correctness and performance issues of gdb's current
handling of the lookup of builtin types, as discussed in PR 16994.

correctness: Though rare, we should still properly handle it,
which is if we're in a file where the STATIC_BLOCK doesn't provide a definition
for "double" (basetypes live in STATIC_BLOCK) and the fallback that gdb uses
(search all static blocks of all objfiles) happens to find a find a definition
of "double" in a file that was compiled with -fshort-double.

performance: We search in all static blocks of all objfiles.
The definition of "void" is typically not specified in debug info and yet gdb
can look up "void" a lot.  Thus every time gdb goes to search "void" it will
look in all static blocks of all objfiles, find nothing, and only *then* does
c-exp.y:classify_name, for example, call
language_lookup_primitive_type_by_name.

Fixing this will require giving lookup_symbol (et.al.) the ability to return
builtin types as symbols.  Patch to follow.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


^ permalink raw reply	[flat|nested] 2+ messages in thread

* [Bug symtab/17684] lookup of builtin types: correctness and performance issues
  2014-12-07 17:56 [Bug symtab/17684] New: lookup of builtin types: correctness and performance issues xdje42 at gmail dot com
@ 2015-03-18 19:06 ` dje at google dot com
  0 siblings, 0 replies; 2+ messages in thread
From: dje at google dot com @ 2015-03-18 19:06 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=17684

dje at google dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |dje at google dot com
         Resolution|---                         |FIXED

--- Comment #1 from dje at google dot com ---
Patch set is here:
https://sourceware.org/ml/gdb-patches/2014-12/msg00521.html

-- 
You are receiving this mail because:
You are on the CC list for the bug.


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-03-18 18:14 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-12-07 17:56 [Bug symtab/17684] New: lookup of builtin types: correctness and performance issues xdje42 at gmail dot com
2015-03-18 19:06 ` [Bug symtab/17684] " dje at google dot com

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).