[commit] Fix Xtensa Call0 ABI prologue analysis

[commit] Fix Xtensa Call0 ABI prologue analysis

[Maxim Grigoriev]

[-- Attachment #1: Type: text/plain, Size: 92 bytes --]

On Xtensa CALL0 ABI, prologue analysis does not work, when MSB is set in 
the address(es).


[-- Attachment #2: xtensa-call0-MSB.diff --]
[-- Type: text/x-patch, Size: 682 bytes --]

2009-09-18  Maxim Grigoriev  <maxim2405@gmail.com>

        * xtensa-tdep.c (call0_analyze_prologue): Replace INT_MAX by UNIT_MAX.

Index: gdb/xtensa-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/xtensa-tdep.c,v
retrieving revision 1.39
diff -u -r1.39 xtensa-tdep.c
--- gdb/xtensa-tdep.c	14 Aug 2009 00:32:32 -0000	1.39
+++ gdb/xtensa-tdep.c	19 Sep 2009 03:48:44 -0000
@@ -2125,7 +2125,7 @@
      Assume we may be in the prologue until we hit a flow control instr.  */
 
   rtmp = NULL;
-  body_pc = INT_MAX;
+  body_pc = UINT_MAX;
   end_pc = 0;
 
   /* Find out, if we have an information about the prologue from DWARF.  */

Re: [commit] Fix Xtensa Call0 ABI prologue analysis

[Maxim Grigoriev]

[-- Attachment #1: Type: text/plain, Size: 349 bytes --]

Can somebody, please, give me an instruction on how to check this patch 
in to GDB 7.0 branch ?
Or, just apply the patch.

I don't have an access to the branch and wasn't able to find out how to 
get it.

Thanks,
-- Maxim

Maxim Grigoriev wrote:
> On Xtensa CALL0 ABI, prologue analysis does not work, when MSB is set in 
> the address(es).
>
>   


[-- Attachment #2: xtensa-call0-MSB.diff --]
[-- Type: text/x-patch, Size: 682 bytes --]

2009-09-18  Maxim Grigoriev  <maxim2405@gmail.com>

        * xtensa-tdep.c (call0_analyze_prologue): Replace INT_MAX by UNIT_MAX.

Index: gdb/xtensa-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/xtensa-tdep.c,v
retrieving revision 1.39
diff -u -r1.39 xtensa-tdep.c
--- gdb/xtensa-tdep.c	14 Aug 2009 00:32:32 -0000	1.39
+++ gdb/xtensa-tdep.c	19 Sep 2009 03:48:44 -0000
@@ -2125,7 +2125,7 @@
      Assume we may be in the prologue until we hit a flow control instr.  */
 
   rtmp = NULL;
-  body_pc = INT_MAX;
+  body_pc = UINT_MAX;
   end_pc = 0;
 
   /* Find out, if we have an information about the prologue from DWARF.  */

Re: [commit] Fix Xtensa Call0 ABI prologue analysis

[Joel Brobecker]

> Can somebody, please, give me an instruction on how to check this patch  
> in to GDB 7.0 branch ?
> Or, just apply the patch.

The information on how to checkout a copy of the branch is on the main
web page: http://www.sourceware.org/gdb/.  To have write priviledges,
you'll have to use your sourceware.org account rather than the pserver
access.  After that, commit should have as usual.  Let me know if you
have any problem.

-- 
Joel

[commit] gdb_7_0-branch -- Fix Xtensa Call0 ABI prologue analysis

[Maxim Grigoriev]

[-- Attachment #1: Type: text/plain, Size: 169 bytes --]

Same patch committed onto GDB 7.0 branch.

Maxim Grigoriev wrote:
> On Xtensa CALL0 ABI, prologue analysis does not work, when MSB is set in 
> the address(es).
>
>   


[-- Attachment #2: xtensa-call0-MSB.diff --]
[-- Type: text/x-patch, Size: 682 bytes --]

2009-09-18  Maxim Grigoriev  <maxim2405@gmail.com>

        * xtensa-tdep.c (call0_analyze_prologue): Replace INT_MAX by UNIT_MAX.

Index: gdb/xtensa-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/xtensa-tdep.c,v
retrieving revision 1.39
diff -u -r1.39 xtensa-tdep.c
--- gdb/xtensa-tdep.c	14 Aug 2009 00:32:32 -0000	1.39
+++ gdb/xtensa-tdep.c	19 Sep 2009 03:48:44 -0000
@@ -2125,7 +2125,7 @@
      Assume we may be in the prologue until we hit a flow control instr.  */
 
   rtmp = NULL;
-  body_pc = INT_MAX;
+  body_pc = UINT_MAX;
   end_pc = 0;
 
   /* Find out, if we have an information about the prologue from DWARF.  */