[RFA] remove unused(?) code in tramp_frame_cache

[RFA] remove unused(?) code in tramp_frame_cache

[Doug Evans]

Hi.

I was going to check this in as obvious, but I couldn't
discount the possibility that the call to get_frame_pc was
being done for possible side effects.
I don't have a target to test this on.

Does this look ok?

2010-03-04  Doug Evans  <dje@google.com>

	* tramp-frame.c (tramp_frame_cache): Delete unused code.

Index: tramp-frame.c
===================================================================
RCS file: /cvs/src/src/gdb/tramp-frame.c,v
retrieving revision 1.18
diff -u -p -r1.18 tramp-frame.c
--- tramp-frame.c	1 Jan 2010 07:31:43 -0000	1.18
+++ tramp-frame.c	4 Mar 2010 18:05:38 -0000
@@ -44,7 +44,6 @@ static struct trad_frame_cache *
 tramp_frame_cache (struct frame_info *this_frame,
 		   void **this_cache)
 {
-  CORE_ADDR pc = get_frame_pc (this_frame);
   struct tramp_frame_cache *tramp_cache = (*this_cache);
   if (tramp_cache->trad_cache == NULL)
     {

Re: [RFA] remove unused(?) code in tramp_frame_cache

[Daniel Jacobowitz]

On Thu, Mar 04, 2010 at 10:13:57AM -0800, Doug Evans wrote:
> Hi.
> 
> I was going to check this in as obvious, but I couldn't
> discount the possibility that the call to get_frame_pc was
> being done for possible side effects.
> I don't have a target to test this on.
> 
> Does this look ok?

Yes, this is fine.

-- 
Daniel Jacobowitz
CodeSourcery