public inbox for archer-commits@sourceware.org
help / color / mirror / Atom feed
* [SCM]  archer-gbenson-break-on-inline: Tweak
@ 2011-12-02 13:05 gary
  0 siblings, 0 replies; only message in thread
From: gary @ 2011-12-02 13:05 UTC (permalink / raw)
  To: archer-commits

The branch, archer-gbenson-break-on-inline has been updated
       via  736dcc37da51aea037297bba18dd9fd886e1469b (commit)
       via  01a655fe9f38b92b9bd52db2d0384a1233cec0ed (commit)
       via  14ac10a35a96b81b09fd4ed0edb58e8ccdf081ff (commit)
       via  16489f41a30139d6fa9719d44977344b19098ff5 (commit)
       via  8171d926c2c451f68a176ef830b17bb1585a60df (commit)
      from  7f8e59be400571cab308872e1120b6e060a1e372 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email.

- Log -----------------------------------------------------------------
commit 736dcc37da51aea037297bba18dd9fd886e1469b
Author: Gary Benson <gbenson@redhat.com>
Date:   Fri Dec 2 13:05:06 2011 +0000

    Tweak

commit 01a655fe9f38b92b9bd52db2d0384a1233cec0ed
Author: Gary Benson <gbenson@redhat.com>
Date:   Fri Dec 2 13:03:49 2011 +0000

    Documentation update (from Tom)

commit 14ac10a35a96b81b09fd4ed0edb58e8ccdf081ff
Author: Gary Benson <gbenson@redhat.com>
Date:   Fri Dec 2 13:02:49 2011 +0000

    Tweak

commit 16489f41a30139d6fa9719d44977344b19098ff5
Author: Gary Benson <gbenson@redhat.com>
Date:   Fri Dec 2 12:52:13 2011 +0000

    NEWS entry

commit 8171d926c2c451f68a176ef830b17bb1585a60df
Author: Gary Benson <gbenson@redhat.com>
Date:   Fri Dec 2 11:11:57 2011 +0000

    Filter out inlined symbols for printing, etc, as per Tom's suggestion

-----------------------------------------------------------------------

Summary of changes:
 gdb/ChangeLog       |    6 +++++-
 gdb/NEWS            |    2 ++
 gdb/doc/ChangeLog   |    6 ++++++
 gdb/doc/gdb.texinfo |    8 --------
 gdb/symtab.c        |    5 +++--
 5 files changed, 16 insertions(+), 11 deletions(-)

First 500 lines of diff:
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 056f327..5a75d2d 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,4 +1,4 @@
-2011-11-29  Gary Benson  <gbenson@redhat.com>
+2011-12-02  Gary Benson  <gbenson@redhat.com>
 
 	PR breakpoints/10738
 	* dwarf2read.c (struct partial_die_info): New member
@@ -8,6 +8,10 @@
 	DIEs that may be inlined.
 	(new_symbol_full): Add inlined subroutines to the static
 	symbol table.
+	* symtab.c (lookup_block_symbol): Don't return symbols of
+	inlined subroutines.
+	* NEWS: Mention that GDB can now set breakpoints on inlined
+	functions.
 
 2011-10-25  Paul Koning  <paul_koning@dell.com>
 
diff --git a/gdb/NEWS b/gdb/NEWS
index 5cdb63e..f2d50d5 100644
--- a/gdb/NEWS
+++ b/gdb/NEWS
@@ -3,6 +3,8 @@
 
 *** Changes since GDB 7.3.1
 
+* GDB can now set breakpoints on inlined functions.
+
 * GDB has two new commands: "set remote hardware-watchpoint-length-limit"
   and "show remote hardware-watchpoint-length-limit".  These allows to
   set or show the maximum length limit (in bytes) of a remote
diff --git a/gdb/doc/ChangeLog b/gdb/doc/ChangeLog
index 5b38a67..c63ed12 100644
--- a/gdb/doc/ChangeLog
+++ b/gdb/doc/ChangeLog
@@ -1,3 +1,9 @@
+2011-12-02  Gary Benson  <gbenson@redhat.com>
+
+	PR breakpoints/10738
+	* gdb.texinfo (Inline Functions): Remove the now-unnecessary @item
+	stating that GDB cannot set breakpoints on inlined functions.
+
 2011-10-25  Paul Koning  <paul_koning@dell.com>
 
 	PR python/13327
diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo
index 853692c..a9baf0b 100644
--- a/gdb/doc/gdb.texinfo
+++ b/gdb/doc/gdb.texinfo
@@ -9663,14 +9663,6 @@ function calls are the same as normal calls:
 
 @itemize @bullet
 @item
-You cannot set breakpoints on inlined functions.  @value{GDBN}
-either reports that there is no symbol with that name, or else sets the
-breakpoint only on non-inlined copies of the function.  This limitation
-will be removed in a future version of @value{GDBN}; until then,
-set a breakpoint by line number on the first line of the inlined
-function instead.
-
-@item
 Setting breakpoints at the call site of an inlined function may not
 work, because the call site does not contain any code.  @value{GDBN}
 may incorrectly move the breakpoint to the next line of the enclosing
diff --git a/gdb/symtab.c b/gdb/symtab.c
index 68b80be..275c201 100644
--- a/gdb/symtab.c
+++ b/gdb/symtab.c
@@ -1781,8 +1781,9 @@ lookup_block_symbol (const struct block *block, const char *name,
 	   sym != NULL;
 	   sym = dict_iter_name_next (name, &iter))
 	{
-	  if (symbol_matches_domain (SYMBOL_LANGUAGE (sym),
-				     SYMBOL_DOMAIN (sym), domain))
+	  if (!SYMBOL_INLINED (sym)
+	      && symbol_matches_domain (SYMBOL_LANGUAGE (sym),
+					SYMBOL_DOMAIN (sym), domain))
 	    return sym;
 	}
       return NULL;


hooks/post-receive
--
Repository for Project Archer.


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2011-12-02 13:05 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-12-02 13:05 [SCM] archer-gbenson-break-on-inline: Tweak gary

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