diff mbox series

Add wide_int version of inchash::hash::add_wide_int

Message ID 87wp3zrvlz.fsf@linaro.org
State New
Headers show
Series Add wide_int version of inchash::hash::add_wide_int | expand

Commit Message

Richard Sandiford Oct. 13, 2017, 2:05 p.m. UTC
This patch adds an inchash hasher for wide_int-based types.
It means that hash_tree no longer hashes TREE_INT_CST_EXT_NUNITS,
but that was redundant with hashing the type.

Tested on aarch64-linux-gnu, x86_64-linux-gnu and powerpc64-linux-gnu.
OK to install?

Richard


2017-10-13  Richard Sandiford  <richard.sandiford@linaro.org>

gcc/
	* inchash.h (inchash::hash::add_wide_int): New function.
	* lto-streamer-out.c (hash_tree): Use it.

Comments

Jeff Law Oct. 13, 2017, 2:17 p.m. UTC | #1
On 10/13/2017 08:05 AM, Richard Sandiford wrote:
> This patch adds an inchash hasher for wide_int-based types.

> It means that hash_tree no longer hashes TREE_INT_CST_EXT_NUNITS,

> but that was redundant with hashing the type.

> 

> Tested on aarch64-linux-gnu, x86_64-linux-gnu and powerpc64-linux-gnu.

> OK to install?

> 

> Richard

> 

> 

> 2017-10-13  Richard Sandiford  <richard.sandiford@linaro.org>

> 

> gcc/

> 	* inchash.h (inchash::hash::add_wide_int): New function.

> 	* lto-streamer-out.c (hash_tree): Use it.

OK.
jeff
diff mbox series

Patch

Index: gcc/inchash.h
===================================================================
--- gcc/inchash.h	2017-10-13 14:59:35.120146199 +0100
+++ gcc/inchash.h	2017-10-13 15:00:00.743308010 +0100
@@ -63,6 +63,15 @@  hashval_t iterative_hash_hashval_t (hash
     val = iterative_hash_host_wide_int (v, val);
   }
 
+  /* Add wide_int-based value V.  */
+  template<typename T>
+  void add_wide_int (const generic_wide_int<T> &x)
+  {
+    add_int (x.get_len ());
+    for (unsigned i = 0; i < x.get_len (); i++)
+      add_hwi (x.elt (i));
+  }
+
   /* Hash in pointer PTR.  */
   void add_ptr (const void *ptr)
   {
Index: gcc/lto-streamer-out.c
===================================================================
--- gcc/lto-streamer-out.c	2017-10-13 14:59:35.121978207 +0100
+++ gcc/lto-streamer-out.c	2017-10-13 15:00:00.743308010 +0100
@@ -1028,13 +1028,7 @@  #define visit(SIBLING) \
   hstate.commit_flag ();
 
   if (CODE_CONTAINS_STRUCT (code, TS_INT_CST))
-    {
-      int i;
-      hstate.add_hwi (TREE_INT_CST_NUNITS (t));
-      hstate.add_hwi (TREE_INT_CST_EXT_NUNITS (t));
-      for (i = 0; i < TREE_INT_CST_NUNITS (t); i++)
-	hstate.add_hwi (TREE_INT_CST_ELT (t, i));
-    }
+    hstate.add_wide_int (wi::to_widest (t));
 
   if (CODE_CONTAINS_STRUCT (code, TS_REAL_CST))
     {