From: Daniel Jacobowitz <drow@false.org>
Date: Fri, 13 Nov 2009 23:00:17 +0000 (+0000)
Subject: 	* inline-frame.c (find_inline_frame_state): Check for changed PC
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=4e48512954ed27c1c1e1cf873e01f9028dcb2b3b;p=binutils-gdb.git

	* inline-frame.c (find_inline_frame_state): Check for changed PC
	here...
	(inline_frame_sniffer): ... not here.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index f6aa0cf9f1d..29539b12e1e 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2009-11-13  Daniel Jacobowitz  <dan@codesourcery.com>
+
+	* inline-frame.c (find_inline_frame_state): Check for changed PC
+	here...
+	(inline_frame_sniffer): ... not here.
+
 2009-11-13  Daniel Jacobowitz  <dan@codesourcery.com>
 
 	* ui-file.c (stdio_file_read): Call gdb_select before read.
diff --git a/gdb/inline-frame.c b/gdb/inline-frame.c
index 9af88f44159..5408e88002e 100644
--- a/gdb/inline-frame.c
+++ b/gdb/inline-frame.c
@@ -22,6 +22,7 @@
 #include "block.h"
 #include "frame-unwind.h"
 #include "inferior.h"
+#include "regcache.h"
 #include "symtab.h"
 #include "vec.h"
 
@@ -59,7 +60,8 @@ DEF_VEC_O(inline_state_s);
 
 static VEC(inline_state_s) *inline_states;
 
-/* Locate saved inlined frame state for PTID, if it exists.  */
+/* Locate saved inlined frame state for PTID, if it exists
+   and is valid.  */
 
 static struct inline_state *
 find_inline_frame_state (ptid_t ptid)
@@ -70,7 +72,19 @@ find_inline_frame_state (ptid_t ptid)
   for (ix = 0; VEC_iterate (inline_state_s, inline_states, ix, state); ix++)
     {
       if (ptid_equal (state->ptid, ptid))
-	return state;
+	{
+	  struct regcache *regcache = get_thread_regcache (ptid);
+	  CORE_ADDR current_pc = regcache_read_pc (regcache);
+	  if (current_pc != state->saved_pc)
+	    {
+	      /* PC has changed - this context is invalid.  Use the
+		 default behavior.  */
+	      VEC_unordered_remove (inline_state_s, inline_states, ix);
+	      return NULL;
+	    }
+	  else
+	    return state;
+	}
     }
 
   return NULL;
@@ -225,13 +239,8 @@ inline_frame_sniffer (const struct frame_unwind *self,
      can be stepped into later).  */
   if (state != NULL && state->skipped_frames > 0 && next_frame == NULL)
     {
-      if (this_pc != state->saved_pc)
-	state->skipped_frames = 0;
-      else
-	{
-	  gdb_assert (depth >= state->skipped_frames);
-	  depth -= state->skipped_frames;
-	}
+      gdb_assert (depth >= state->skipped_frames);
+      depth -= state->skipped_frames;
     }
 
   /* If all the inlined functions here already have frames, then pass