[RFA] Implement Cell BE SPU-specific vector type

[Markus Deuling <deuling at de dot ibm dot com>]

Hello,

there were problems with whitespaces in the last patch I posted, so here 
it is again.

I reworked the patch that adds a SPU-specific vector type to spu-tdep.c 
(the order of the union fields is changed).

Ok to apply ?

ChangeLog:

    * spu-tdep.c (spu_init_vector_type): New function.
    (spu_builtin_type_vec128.): New static variable.
    (spu_register_type): Return spu_builtin_type_vec128 as default.
    (_initialize_spu_tdep): Call spu_init_vector_type.

=======================================
diff -urN src/gdb/spu-tdep.c dev/gdb/spu-tdep.c
--- src/gdb/spu-tdep.c  2006-11-22 14:49:53.000000000 +0100
+++ dev/gdb/spu-tdep.c  2006-12-12 17:05:24.000000000 +0100
@@ -45,6 +45,8 @@

 #include "spu-tdep.h"

+/* SPU-specific vector type.  */
+struct type *spu_builtin_type_vec128;

 /* Registers.  */

@@ -84,7 +86,7 @@
 spu_register_type (struct gdbarch *gdbarch, int reg_nr)
 {
   if (reg_nr < SPU_NUM_GPRS)
-    return builtin_type_vec128;
+    return spu_builtin_type_vec128;

   switch (reg_nr)
     {
@@ -1092,8 +1094,31 @@
   return gdbarch;
 }

+/* Implement a SPU-specific vector type as replacement
+   for __gdb_builtin_type_vec128.  */
+static void
+spu_init_vector_type (void)
+{
+  struct type *type;
+
+  type = init_composite_type ("__spu_builtin_type_vec128", 
TYPE_CODE_UNION);
+  append_composite_type_field (type, "uint128", builtin_type_int128);
+  append_composite_type_field (type, "v2_int64", builtin_type_v2_int64);
+  append_composite_type_field (type, "v4_int32", builtin_type_v4_int32);
+  append_composite_type_field (type, "v8_int16", builtin_type_v8_int16);
+  append_composite_type_field (type, "v16_int8", builtin_type_v16_int8);
+  append_composite_type_field (type, "v2_double", builtin_type_v2_double);
+  append_composite_type_field (type, "v4_float", builtin_type_v4_float);
+
+  TYPE_FLAGS (type) |= TYPE_FLAG_VECTOR;
+  TYPE_NAME (type) = "spu_builtin_type_vec128";
+  spu_builtin_type_vec128 = type;
+}
+
 void
 _initialize_spu_tdep (void)
 {
   register_gdbarch_init (bfd_arch_spu, spu_gdbarch_init);
+
+  spu_init_vector_type ();
 }


Regards,
Markus

--
Markus Deuling
GNU Toolchain for Linux on Cell BE
deuling@de.ibm.com