re PR c++/36852 (Two dimensional array in template method argument list incorrectly interpreted.)

	PR c++/36852
	* tree.c (cplus_array_hash, build_cplus_array_type_1): Hash on
	TYPE_UID instead of pointers.

	* g++.dg/pch/array-1.C: New test.
	* g++.dg/pch/array-1.Hs: New file.

From-SVN: r138250

gcc/cp/ChangeLog

+2008-07-29  Jakub Jelinek  <jakub@redhat.com>
+
+	PR c++/36852
+	* tree.c (cplus_array_hash, build_cplus_array_type_1): Hash on
+	TYPE_UID instead of pointers.
+
 2008-07-29  Jan Hubicka  <jh@suse.cz>
 
  	* optimize.c (maybe_clone_body): Remove DECL_INLINE.

gcc/cp/tree.c

@@ -504,9 +504,9 @@ cplus_array_hash (const void* k)
   hashval_t hash;
   const_tree const t = (const_tree) k;
 
-  hash = (htab_hash_pointer (TREE_TYPE (t))
-	  ^ htab_hash_pointer (TYPE_DOMAIN (t)));
-
+  hash = TYPE_UID (TREE_TYPE (t));
+  if (TYPE_DOMAIN (t))
+    hash ^= TYPE_UID (TYPE_DOMAIN (t));
   return hash;
 }
 
@@ -553,8 +553,9 @@ build_cplus_array_type_1 (tree elt_type, tree index_type)
 	cplus_array_htab = htab_create_ggc (61, &cplus_array_hash,
 					    &cplus_array_compare, NULL);
       
-      hash = (htab_hash_pointer (elt_type)
-	      ^ htab_hash_pointer (index_type));
+      hash = TYPE_UID (elt_type);
+      if (index_type)
+	hash ^= TYPE_UID (index_type);
       cai.type = elt_type;
       cai.domain = index_type;
 

gcc/testsuite/ChangeLog

+2008-07-29  Jakub Jelinek  <jakub@redhat.com>
+
+	PR c++/36852
+	* g++.dg/pch/array-1.C: New test.
+	* g++.dg/pch/array-1.Hs: New file.
+
 2008-07-29  Jan Hubicka  <jh@suse.cz>
 
 	* gcc.dg/20040206-1.c: Expect frontend warning now.

gcc/testsuite/g++.dg/pch/array-1.C

+// PR c++/36852
+
+#include "array-1.H"
+
+template <class T>
+T
+A::foo (T t[3][3])
+{
+  return t[0][1];
+}
+
+int
+main ()
+{
+}

gcc/testsuite/g++.dg/pch/array-1.Hs

+struct A
+{
+  template <class T> static T foo (T[3][3]);
+};