* frame.c (get_frame_register_bytes): Avoid excessive function calls.
authorAndreas Schwab <schwab@linux-m68k.org>
Wed, 24 Sep 2008 12:59:49 +0000 (12:59 +0000)
committerAndreas Schwab <schwab@linux-m68k.org>
Wed, 24 Sep 2008 12:59:49 +0000 (12:59 +0000)
gdb/ChangeLog
gdb/frame.c

index 155c48319a909188b6096f96b53b66e3faa5462e..fbc6f108a7bbf598fdec565e9bf863c6739b584b 100644 (file)
@@ -1,7 +1,7 @@
 2008-09-24  Andreas Schwab  <schwab@suse.de>
 
        * frame.c (get_frame_register_bytes): Take pseudo registers into
-       account.
+       account.  Avoid excessive function calls.
 
 2008-09-23  Doug Evans  <dje@google.com>
 
index 0fd21e4eabed67b79547de05ff27fb14a7fc4e76..a877494307624b03bdde5b6e1d62a57a33d43cc2 100644 (file)
@@ -798,6 +798,7 @@ get_frame_register_bytes (struct frame_info *frame, int regnum,
   struct gdbarch *gdbarch = get_frame_arch (frame);
   int i;
   int maxsize;
+  int numregs;
 
   /* Skip registers wholly inside of OFFSET.  */
   while (offset >= register_size (gdbarch, regnum))
@@ -809,8 +810,8 @@ get_frame_register_bytes (struct frame_info *frame, int regnum,
   /* Ensure that we will not read beyond the end of the register file.
      This can only ever happen if the debug information is bad.  */
   maxsize = -offset;
-  for (i = regnum;
-       i < gdbarch_num_regs (gdbarch) + gdbarch_num_pseudo_regs (gdbarch); i++)
+  numregs = gdbarch_num_regs (gdbarch) + gdbarch_num_pseudo_regs (gdbarch);
+  for (i = regnum; i < numregs; i++)
     {
       int thissize = register_size (gdbarch, i);
       if (thissize == 0)