[PUSHED,OB/PATCH] Garbage collect value_contents_equal

Message ID 1405360333-24295-1-git-send-email-palves@redhat.com
State Committed
Headers

Commit Message

Pedro Alves July 14, 2014, 5:52 p.m. UTC
  Hasn't been used in years.

gdb/
2014-07-14  Pedro Alves  <palves@redhat.com>

	* value.c (value_contents_equal): Delete function.
	* value.h (value_contents_equal): Delete declaration.
---
 gdb/ChangeLog |  5 +++++
 gdb/value.c   | 19 -------------------
 gdb/value.h   |  1 -
 3 files changed, 5 insertions(+), 20 deletions(-)
  

Patch

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 6d4d2b9..dd4b2eb 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@ 
+2014-07-14  Pedro Alves  <palves@redhat.com>
+
+	* value.c (value_contents_equal): Delete function.
+	* value.h (value_contents_equal): Delete declaration.
+
 2014-07-14  Tom Tromey  <tromey@redhat.com>
 
 	PR exp/17106:
diff --git a/gdb/value.c b/gdb/value.c
index 557056f..29abe5f 100644
--- a/gdb/value.c
+++ b/gdb/value.c
@@ -1210,25 +1210,6 @@  value_contents_writeable (struct value *value)
   return value_contents_raw (value);
 }
 
-/* Return non-zero if VAL1 and VAL2 have the same contents.  Note that
-   this function is different from value_equal; in C the operator ==
-   can return 0 even if the two values being compared are equal.  */
-
-int
-value_contents_equal (struct value *val1, struct value *val2)
-{
-  struct type *type1;
-  struct type *type2;
-
-  type1 = check_typedef (value_type (val1));
-  type2 = check_typedef (value_type (val2));
-  if (TYPE_LENGTH (type1) != TYPE_LENGTH (type2))
-    return 0;
-
-  return (memcmp (value_contents (val1), value_contents (val2),
-		  TYPE_LENGTH (type1)) == 0);
-}
-
 int
 value_optimized_out (struct value *value)
 {
diff --git a/gdb/value.h b/gdb/value.h
index c2e9faf..86ebd70 100644
--- a/gdb/value.h
+++ b/gdb/value.h
@@ -322,7 +322,6 @@  extern const gdb_byte *
   value_contents_for_printing_const (const struct value *value);
 
 extern int value_fetch_lazy (struct value *val);
-extern int value_contents_equal (struct value *val1, struct value *val2);
 
 /* If nonzero, this is the value of a variable which does not actually
    exist in the program, at least partially.  If the value is lazy,