Use allocate_optimized_out_value instead of set_value_optimized_out.

Allocate the value as optimized out from the start rather than allocating
a value with contents, and then marking it optimized out.

gdb/
2013-07-04  Pedro Alves  <palves@redhat.com>

	* findvar.c (value_of_register): Use allocate_optimized_out_value
	if the register has been optimized out, instead of
	set_value_optimized_out.
	* frame-unwind.c (frame_unwind_got_optimized): Use
	allocate_optimized_out_value.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 2353e71e9ebdf4df0c7a16e07996c4666fa7b1f1..eb0eb4bd30622f008e111471aabb3aa9de841ef4 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,11 @@
+2013-07-04  Pedro Alves  <palves@redhat.com>
+
+       * findvar.c (value_of_register): Use allocate_optimized_out_value
+       if the register has been optimized out, instead of
+       set_value_optimized_out.
+       * frame-unwind.c (frame_unwind_got_optimized): Use
+       allocate_optimized_out_value.
+
 2013-07-04  Pedro Alves  <palves@redhat.com>
 
        * value.c (value_bits_valid): If the value is not lval_computed,

diff --git a/gdb/findvar.c b/gdb/findvar.c
index fdce436baac214f2ed2fddd06cbe3c904e481472..f586ce24efeb8d9b8ea725f47d15e3d7150c6734 100644 (file)
--- a/gdb/findvar.c
+++ b/gdb/findvar.c
@@ -266,6 +266,7 @@ value_of_register (int regnum, struct frame_info *frame)
   int optim;
   int unavail;
   struct value *reg_val;
+  struct type *reg_type;
   int realnum;
   gdb_byte raw_buffer[MAX_REGISTER_SIZE];
   enum lval_type lval;
@@ -279,7 +280,11 @@ value_of_register (int regnum, struct frame_info *frame)
   frame_register (frame, regnum, &optim, &unavail,
                  &lval, &addr, &realnum, raw_buffer);
 
-  reg_val = allocate_value (register_type (gdbarch, regnum));
+  reg_type = register_type (gdbarch, regnum);
+  if (optim)
+    reg_val = allocate_optimized_out_value (reg_type);
+  else
+    reg_val = allocate_value (reg_type);
 
   if (!optim && !unavail)
     memcpy (value_contents_raw (reg_val), raw_buffer,
@@ -291,7 +296,6 @@ value_of_register (int regnum, struct frame_info *frame)
   VALUE_LVAL (reg_val) = lval;
   set_value_address (reg_val, addr);
   VALUE_REGNUM (reg_val) = regnum;
-  set_value_optimized_out (reg_val, optim);
   if (unavail)
     mark_value_bytes_unavailable (reg_val, 0, register_size (gdbarch, regnum));
   VALUE_FRAME_ID (reg_val) = get_frame_id (frame);

diff --git a/gdb/frame-unwind.c b/gdb/frame-unwind.c
index b66febff63966b48a0a00337414a1b2e45366a29..ce2f6da831348ed5111d155b4f113fed0f1716fc 100644 (file)
--- a/gdb/frame-unwind.c
+++ b/gdb/frame-unwind.c
@@ -162,11 +162,9 @@ struct value *
 frame_unwind_got_optimized (struct frame_info *frame, int regnum)
 {
   struct gdbarch *gdbarch = frame_unwind_arch (frame);
-  struct value *reg_val;
+  struct type *reg_type = register_type (gdbarch, regnum);
 
-  reg_val = value_zero (register_type (gdbarch, regnum), not_lval);
-  set_value_optimized_out (reg_val, 1);
-  return reg_val;
+  return allocate_optimized_out_value (reg_type);
 }
 
 /* Return a value which indicates that FRAME copied REGNUM into