From: Karthik Bhat <kvbhat@sourceware.org>
Date: Wed, 21 Nov 2012 07:39:02 +0000 (+0000)
Subject: Fix for incorrect breakpoint set in case of clang compiled binary
X-Git-Tag: cgen-snapshot-20121201~83
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=4e879fc2fd0322fb016aa80af907c052b326430a;p=platform%2Fupstream%2Fbinutils.git

Fix for incorrect breakpoint set in case of clang compiled binary
---

diff --git a/gdb/i386-tdep.c b/gdb/i386-tdep.c
index f0056be..5e12cb5 100644
--- a/gdb/i386-tdep.c
+++ b/gdb/i386-tdep.c
@@ -1582,8 +1582,30 @@ i386_skip_prologue (struct gdbarch *gdbarch, CORE_ADDR start_pc)
   CORE_ADDR pc;
   gdb_byte op;
   int i;
-
   cache.locals = -1;
+  CORE_ADDR func_addr;
+  struct symtab *s = find_pc_symtab (func_addr);
+
+  if (find_pc_partial_function (start_pc, NULL, &func_addr, NULL))
+  {
+    CORE_ADDR post_prologue_pc
+      = skip_prologue_using_sal (gdbarch, func_addr);
+    
+    /* GCC always emits a line note before the prologue and another
+	 one after, even if the two are at the same address or on the
+	 same line.  Take advantage of this so that we do not need to
+	 know every instruction that might appear in the prologue.  We
+	 will have producer information for most binaries; if it is
+	 missing (e.g. for -gstabs), assuming the GNU tools.  */
+    if (post_prologue_pc
+	  && (s == NULL
+	      || s->producer == NULL
+	      || strncmp (s->producer, "GNU ", sizeof ("GNU ") - 1) == 0
+	      || strncmp (s->producer, "clang ", sizeof ("clang ") - 1) == 0))
+	  return  max (start_pc, post_prologue_pc);
+  }
+
+
   pc = i386_analyze_prologue (gdbarch, start_pc, 0xffffffff, &cache);
   if (cache.locals < 0)
     return start_pc;