Rewrite find_active_thread() and find_tcb() to use ptid_t, instead of
[deliverable/binutils-gdb.git] / gdb / gdbarch.sh
index 75480900d9663e2c56efe536259b70d7459a31e0..d18a5f292ead83609bf46d76104f175ab119c23e 100755 (executable)
@@ -85,7 +85,7 @@ EOF
            case "${invalid_p}" in
                0 ) valid_p=1 ;;
                "" )
-                   if [ "${predefault}" ]
+                   if [ -n "${predefault}" ]
                    then
                        #invalid_p="gdbarch->${function} == ${predefault}"
                        valid_p="gdbarch->${function} != ${predefault}"
@@ -104,10 +104,10 @@ EOF
            # always a valid definition of MEMBER as this again
            # ensures consistency.
 
-           if [ "${postdefault}" != "" ]
+           if [ -n "${postdefault}" ]
            then
                fallbackdefault="${postdefault}"
-           elif [ "${predefault}" != "" ]
+           elif [ -n "${predefault}" ]
            then
                fallbackdefault="${predefault}"
            else
@@ -120,7 +120,7 @@ EOF
            break
        fi
     done
-    if [ "${class}" ]
+    if [ -n "${class}" ]
     then
        true
     else
@@ -131,8 +131,8 @@ EOF
 
 fallback_default_p ()
 {
-    [ "${postdefault}" != "" -a "${invalid_p}" != "0" ] \
-       || [ "${predefault}" != "" -a "${invalid_p}" = "0" ]
+    [ -n "${postdefault}" -a "x${invalid_p}" != "x0" ] \
+       || [ -n "${predefault}" -a "x${invalid_p}" = "x0" ]
 }
 
 class_is_variable_p ()
@@ -248,17 +248,19 @@ do
 
     predefault ) : ;;
 
-       # A initial value to assign to MEMBER of the freshly
-       # malloc()ed gdbarch object.  After the gdbarch object has
-       # been initialized using PREDEFAULT, it is passed to the
-       # target code for further updates.
+       # An initial value to assign to MEMBER of the freshly
+       # malloc()ed gdbarch object.  After initialization, the
+       # freshly malloc()ed object is passed to the target
+       # architecture code for further updates.
 
        # If PREDEFAULT is empty, zero is used.
 
-       # When POSTDEFAULT is empty, a non-empty PREDEFAULT and a zero
-       # INVALID_P will be used as default values when when
-       # multi-arch is disabled.  Specify a zero PREDEFAULT function
-       # to make that fallback call internal_error().
+       # A non-empty PREDEFAULT, an empty POSTDEFAULT and a zero
+       # INVALID_P are specified, PREDEFAULT will be used as the
+       # default for the non- multi-arch target.
+
+       # A zero PREDEFAULT function will force the fallback to call
+       # internal_error().
 
        # Variable declarations can refer to ``gdbarch'' which will
        # contain the current architecture.  Care should be taken.
@@ -266,9 +268,8 @@ do
     postdefault ) : ;;
 
        # A value to assign to MEMBER of the new gdbarch object should
-       # the target code fail to change the PREDEFAULT value.  Also
-       # use POSTDEFAULT as the fallback value for the non-
-       # multi-arch case.
+       # the target architecture code fail to change the PREDEFAULT
+       # value.
 
        # If POSTDEFAULT is empty, no post update is performed.
 
@@ -276,6 +277,11 @@ do
        # INVALID_P will be used to determine if MEMBER should be
        # changed to POSTDEFAULT.
 
+       # If a non-empty POSTDEFAULT and a zero INVALID_P are
+       # specified, POSTDEFAULT will be used as the default for the
+       # non- multi-arch target (regardless of the value of
+       # PREDEFAULT).
+
        # You cannot specify both a zero INVALID_P and a POSTDEFAULT.
 
        # Variable declarations can refer to ``gdbarch'' which will
@@ -382,6 +388,10 @@ f::TARGET_READ_FP:CORE_ADDR:read_fp:void:::0:generic_target_read_fp::0
 f::TARGET_WRITE_FP:void:write_fp:CORE_ADDR val:val::0:generic_target_write_fp::0
 f::TARGET_READ_SP:CORE_ADDR:read_sp:void:::0:generic_target_read_sp::0
 f::TARGET_WRITE_SP:void:write_sp:CORE_ADDR val:val::0:generic_target_write_sp::0
+# Function for getting target's idea of a frame pointer.  FIXME: GDB's
+# whole scheme for dealing with "frames" and "frame pointers" needs a
+# serious shakedown.
+f::TARGET_VIRTUAL_FRAME_POINTER:void:virtual_frame_pointer:CORE_ADDR pc, int *frame_regnum, LONGEST *frame_offset:pc, frame_regnum, frame_offset::0:legacy_virtual_frame_pointer::0
 #
 M:::void:register_read:int regnum, char *buf:regnum, buf:
 M:::void:register_write:int regnum, char *buf:regnum, buf:
@@ -423,6 +433,8 @@ f:2:DO_REGISTERS_INFO:void:do_registers_info:int reg_nr, int fpregs:reg_nr, fpre
 # also include/...-sim.h.
 f:2:REGISTER_SIM_REGNO:int:register_sim_regno:int reg_nr:reg_nr:::default_register_sim_regno::0
 F:2:REGISTER_BYTES_OK:int:register_bytes_ok:long nr_bytes:nr_bytes::0:0
+f:2:CANNOT_FETCH_REGISTER:int:cannot_fetch_register:int regnum:regnum:::cannot_register_not::0
+f:2:CANNOT_STORE_REGISTER:int:cannot_store_register:int regnum:regnum:::cannot_register_not::0
 #
 v:1:USE_GENERIC_DUMMY_FRAMES:int:use_generic_dummy_frames::::0:-1
 v:2:CALL_DUMMY_LOCATION:int:call_dummy_location::::0:0
@@ -438,6 +450,8 @@ v:2:SIZEOF_CALL_DUMMY_WORDS:int:sizeof_call_dummy_words::::0:legacy_sizeof_call_
 v:1:CALL_DUMMY_STACK_ADJUST_P:int:call_dummy_stack_adjust_p::::0:-1:::0x%08lx
 v:2:CALL_DUMMY_STACK_ADJUST:int:call_dummy_stack_adjust::::0:::gdbarch->call_dummy_stack_adjust_p && gdbarch->call_dummy_stack_adjust == 0:0x%08lx::CALL_DUMMY_STACK_ADJUST_P
 f:2:FIX_CALL_DUMMY:void:fix_call_dummy:char *dummy, CORE_ADDR pc, CORE_ADDR fun, int nargs, struct value **args, struct type *type, int gcc_p:dummy, pc, fun, nargs, args, type, gcc_p:::0
+f:2:INIT_FRAME_PC_FIRST:void:init_frame_pc_first:int fromleaf, struct frame_info *prev:fromleaf, prev:::init_frame_pc_noop::0
+f:2:INIT_FRAME_PC:void:init_frame_pc:int fromleaf, struct frame_info *prev:fromleaf, prev:::init_frame_pc_default::0
 #
 v:2:BELIEVE_PCC_PROMOTION:int:believe_pcc_promotion:::::::
 v:2:BELIEVE_PCC_PROMOTION_TYPE:int:believe_pcc_promotion_type:::::::
@@ -466,17 +480,9 @@ f:2:PUSH_DUMMY_FRAME:void:push_dummy_frame:void:-:::0
 f:1:PUSH_RETURN_ADDRESS:CORE_ADDR:push_return_address:CORE_ADDR pc, CORE_ADDR sp:pc, sp:::0
 f:2:POP_FRAME:void:pop_frame:void:-:::0
 #
-# I wish that these would just go away....
-f:2:D10V_MAKE_DADDR:CORE_ADDR:d10v_make_daddr:CORE_ADDR x:x:::0::0
-f:2:D10V_MAKE_IADDR:CORE_ADDR:d10v_make_iaddr:CORE_ADDR x:x:::0::0
-f:2:D10V_DADDR_P:int:d10v_daddr_p:CORE_ADDR x:x:::0::0
-f:2:D10V_IADDR_P:int:d10v_iaddr_p:CORE_ADDR x:x:::0::0
-f:2:D10V_CONVERT_DADDR_TO_RAW:CORE_ADDR:d10v_convert_daddr_to_raw:CORE_ADDR x:x:::0::0
-f:2:D10V_CONVERT_IADDR_TO_RAW:CORE_ADDR:d10v_convert_iaddr_to_raw:CORE_ADDR x:x:::0::0
-#
 f:2:STORE_STRUCT_RETURN:void:store_struct_return:CORE_ADDR addr, CORE_ADDR sp:addr, sp:::0
 f:2:STORE_RETURN_VALUE:void:store_return_value:struct type *type, char *valbuf:type, valbuf:::0
-f:2:EXTRACT_STRUCT_VALUE_ADDRESS:CORE_ADDR:extract_struct_value_address:char *regbuf:regbuf:::0
+F:2:EXTRACT_STRUCT_VALUE_ADDRESS:CORE_ADDR:extract_struct_value_address:char *regbuf:regbuf:::0
 f:2:USE_STRUCT_CONVENTION:int:use_struct_convention:int gcc_p, struct type *value_type:gcc_p, value_type:::0
 #
 f:2:FRAME_INIT_SAVED_REGS:void:frame_init_saved_regs:struct frame_info *frame:frame::0:0
@@ -513,7 +519,17 @@ v:2:PARM_BOUNDARY:int:parm_boundary
 v:2:TARGET_FLOAT_FORMAT:const struct floatformat *:float_format::::::default_float_format (gdbarch)
 v:2:TARGET_DOUBLE_FORMAT:const struct floatformat *:double_format::::::default_double_format (gdbarch)
 v:2:TARGET_LONG_DOUBLE_FORMAT:const struct floatformat *:long_double_format::::::&floatformat_unknown
-f:2:CONVERT_FROM_FUNC_PTR_ADDR:CORE_ADDR:convert_from_func_ptr_addr:CORE_ADDR addr:addr:::default_convert_from_func_ptr_addr::0
+f:2:CONVERT_FROM_FUNC_PTR_ADDR:CORE_ADDR:convert_from_func_ptr_addr:CORE_ADDR addr:addr:::core_addr_identity::0
+# On some machines there are bits in addresses which are not really
+# part of the address, but are used by the kernel, the hardware, etc.
+# for special purposes.  ADDR_BITS_REMOVE takes out any such bits so
+# we get a "real" address such as one would find in a symbol table.
+# This is used only for addresses of instructions, and even then I'm
+# not sure it's used in all contexts.  It exists to deal with there
+# being a few stray bits in the PC which would mislead us, not as some
+# sort of generic thing to handle alignment or segmentation (it's
+# possible it should be in TARGET_READ_PC instead).
+f:2:ADDR_BITS_REMOVE:CORE_ADDR:addr_bits_remove:CORE_ADDR addr:addr:::core_addr_identity::0
 # FIXME/cagney/2001-01-18: This should be split in two.  A target method that indicates if
 # the target needs software single step.  An ISA method to implement it.
 #
@@ -523,6 +539,8 @@ f:2:CONVERT_FROM_FUNC_PTR_ADDR:CORE_ADDR:convert_from_func_ptr_addr:CORE_ADDR ad
 # FIXME/cagney/2001-01-18: The logic is backwards.  It should be asking if the target can
 # single step.  If not, then implement single step using breakpoints.
 F:2:SOFTWARE_SINGLE_STEP:void:software_single_step:enum target_signal sig, int insert_breakpoints_p:sig, insert_breakpoints_p::0:0
+f:2:TARGET_PRINT_INSN:int:print_insn:bfd_vma vma, disassemble_info *info:vma, info:::legacy_print_insn::0
+f:2:SKIP_TRAMPOLINE_CODE:CORE_ADDR:skip_trampoline_code:CORE_ADDR pc:pc:::generic_skip_trampoline_code::0
 EOF
 }
 
@@ -549,7 +567,7 @@ EOF
        kill $$
        exit 1
     fi
-    if [ "${invalid_p}" = "0" -a "${postdefault}" != "" ]
+    if [ "x${invalid_p}" = "x0" -a -n "${postdefault}" ]
     then
        echo "Error: postdefault is useless when invalid_p=0" 1>&2
        kill $$
@@ -612,6 +630,8 @@ cat <<EOF
 #ifndef GDBARCH_H
 #define GDBARCH_H
 
+#include "dis-asm.h" /* Get defs for disassemble_info, which unfortunately is a typedef. */
+
 struct frame_info;
 struct value;
 
@@ -633,6 +653,10 @@ extern struct gdbarch *current_gdbarch;
 #error "FRAME_FIND_SAVED_REGS: replaced by FRAME_INIT_SAVED_REGS"
 #endif
 #endif
+
+#if (GDB_MULTI_ARCH >= GDB_MULTI_ARCH_PURE) && defined (GDB_TM_FILE)
+#error "GDB_TM_FILE: Pure multi-arch targets do not have a tm.h file."
+#endif
 EOF
 
 # function typedef's
@@ -646,6 +670,9 @@ do
        printf "\n"
        printf "extern ${returntype} gdbarch_${function} (struct gdbarch *gdbarch);\n"
        printf "/* set_gdbarch_${function}() - not applicable - pre-initialized. */\n"
+       printf "#if (GDB_MULTI_ARCH > GDB_MULTI_ARCH_PARTIAL) && defined (${macro})\n"
+       printf "#error \"Non multi-arch definition of ${macro}\"\n"
+       printf "#endif\n"
        printf "#if GDB_MULTI_ARCH\n"
        printf "#if (GDB_MULTI_ARCH > GDB_MULTI_ARCH_PARTIAL) || !defined (${macro})\n"
        printf "#define ${macro} (gdbarch_${function} (current_gdbarch))\n"
@@ -660,7 +687,7 @@ printf "\n"
 printf "/* The following are initialized by the target dependent code. */\n"
 function_list | while do_read
 do
-    if [ "${comment}" ]
+    if [ -n "${comment}" ]
     then
        echo "${comment}" | sed \
            -e '2 s,#,/*,' \
@@ -692,6 +719,9 @@ do
            printf "#endif\n"
            printf "\n"
            printf "extern int gdbarch_${function}_p (struct gdbarch *gdbarch);\n"
+           printf "#if (GDB_MULTI_ARCH > GDB_MULTI_ARCH_PARTIAL) && defined (${macro}_P)\n"
+           printf "#error \"Non multi-arch definition of ${macro}\"\n"
+           printf "#endif\n"
            printf "#if (GDB_MULTI_ARCH > GDB_MULTI_ARCH_PARTIAL) || !defined (${macro}_P)\n"
            printf "#define ${macro}_P() (gdbarch_${function}_p (current_gdbarch))\n"
            printf "#endif\n"
@@ -711,6 +741,9 @@ do
        printf "\n"
        printf "extern ${returntype} gdbarch_${function} (struct gdbarch *gdbarch);\n"
        printf "extern void set_gdbarch_${function} (struct gdbarch *gdbarch, ${returntype} ${function});\n"
+       printf "#if (GDB_MULTI_ARCH > GDB_MULTI_ARCH_PARTIAL) && defined (${macro})\n"
+       printf "#error \"Non multi-arch definition of ${macro}\"\n"
+       printf "#endif\n"
        printf "#if GDB_MULTI_ARCH\n"
        printf "#if (GDB_MULTI_ARCH > GDB_MULTI_ARCH_PARTIAL) || !defined (${macro})\n"
        printf "#define ${macro} (gdbarch_${function} (current_gdbarch))\n"
@@ -725,7 +758,7 @@ do
            printf "\n"
            printf "/* Default (function) for non- multi-arch platforms. */\n"
            printf "#if (!GDB_MULTI_ARCH) && !defined (${macro})\n"
-           if [ "${fallbackdefault}" = "0" ]
+           if [ "x${fallbackdefault}" = "x0" ]
            then
                printf "#define ${macro}(${actual}) (internal_error (__FILE__, __LINE__, \"${macro}\"), 0)\n"
            else
@@ -736,7 +769,7 @@ do
            printf "#endif\n"
        fi
        printf "\n"
-       if [ "${formal}" = "void" ] && class_is_multiarch_p
+       if [ "x${formal}" = "xvoid" ] && class_is_multiarch_p
        then
            printf "typedef ${returntype} (gdbarch_${function}_ftype) (struct gdbarch *gdbarch);\n"
        elif class_is_multiarch_p
@@ -745,7 +778,7 @@ do
        else
            printf "typedef ${returntype} (gdbarch_${function}_ftype) (${formal});\n"
        fi
-       if [ "${formal}" = "void" ]
+       if [ "x${formal}" = "xvoid" ]
        then
          printf "extern ${returntype} gdbarch_${function} (struct gdbarch *gdbarch);\n"
        else
@@ -754,12 +787,15 @@ do
        printf "extern void set_gdbarch_${function} (struct gdbarch *gdbarch, gdbarch_${function}_ftype *${function});\n"
        if class_is_multiarch_p ; then :
        else
+           printf "#if (GDB_MULTI_ARCH > GDB_MULTI_ARCH_PARTIAL) && defined (${macro})\n"
+           printf "#error \"Non multi-arch definition of ${macro}\"\n"
+           printf "#endif\n"
            printf "#if GDB_MULTI_ARCH\n"
            printf "#if (GDB_MULTI_ARCH > GDB_MULTI_ARCH_PARTIAL) || !defined (${macro})\n"
-           if [ "${actual}" = "" ]
+           if [ "x${actual}" = "x" ]
            then
                printf "#define ${macro}() (gdbarch_${function} (current_gdbarch))\n"
-           elif [ "${actual}" = "-" ]
+           elif [ "x${actual}" = "x-" ]
            then
                printf "#define ${macro} (gdbarch_${function} (current_gdbarch))\n"
            else
@@ -1015,8 +1051,6 @@ extern const struct bfd_arch_info *target_architecture;
 
 /* The target-system-dependent disassembler is semi-dynamic */
 
-#include "dis-asm.h"           /* Get defs for disassemble_info */
-
 extern int dis_asm_read_memory (bfd_vma memaddr, bfd_byte *myaddr,
                                unsigned int len, disassemble_info *info);
 
@@ -1028,32 +1062,12 @@ extern void dis_asm_print_address (bfd_vma addr,
 
 extern int (*tm_print_insn) (bfd_vma, disassemble_info*);
 extern disassemble_info tm_print_insn_info;
-#ifndef TARGET_PRINT_INSN
-#define TARGET_PRINT_INSN(vma, info) (*tm_print_insn) (vma, info)
-#endif
 #ifndef TARGET_PRINT_INSN_INFO
 #define TARGET_PRINT_INSN_INFO (&tm_print_insn_info)
 #endif
 
 
 
-/* Explicit test for D10V architecture.
-   USE of these macro's is *STRONGLY* discouraged. */
-
-#define GDB_TARGET_IS_D10V (TARGET_ARCHITECTURE->arch == bfd_arch_d10v)
-
-
-/* Fallback definition for EXTRACT_STRUCT_VALUE_ADDRESS */
-#ifndef EXTRACT_STRUCT_VALUE_ADDRESS
-#define EXTRACT_STRUCT_VALUE_ADDRESS_P (0)
-#define EXTRACT_STRUCT_VALUE_ADDRESS(X) (internal_error (__FILE__, __LINE__, "gdbarch: EXTRACT_STRUCT_VALUE_ADDRESS"), 0)
-#else
-#ifndef EXTRACT_STRUCT_VALUE_ADDRESS_P
-#define EXTRACT_STRUCT_VALUE_ADDRESS_P (1)
-#endif
-#endif
-
-
 /* Set the dynamic target-system-dependent parameters (architecture,
    byte-order, ...) using information found in the BFD */
 
@@ -1065,6 +1079,10 @@ extern void set_gdbarch_from_file (bfd *);
 
 extern void initialize_current_architecture (void);
 
+/* For non-multiarched targets, do any initialization of the default
+   gdbarch object necessary after the _initialize_MODULE functions
+   have run.  */
+extern void initialize_non_multiarch ();
 
 /* gdbarch trace variable */
 extern int gdbarch_debug;
@@ -1114,11 +1132,13 @@ cat <<EOF
 #include "floatformat.h"
 
 #include "gdb_assert.h"
+#include "gdb-events.h"
 
 /* Static function declarations */
 
 static void verify_gdbarch (struct gdbarch *gdbarch);
-static void check_gdbarch_data (struct gdbarch *);
+static void alloc_gdbarch_data (struct gdbarch *);
+static void init_gdbarch_data (struct gdbarch *);
 static void free_gdbarch_data (struct gdbarch *);
 static void init_gdbarch_swap (struct gdbarch *);
 static void swapout_gdbarch_swap (struct gdbarch *);
@@ -1245,6 +1265,15 @@ cat <<EOF
 };
 
 struct gdbarch *current_gdbarch = &startup_gdbarch;
+
+/* Do any initialization needed for a non-multiarch configuration
+   after the _initialize_MODULE functions have been run.  */
+void
+initialize_non_multiarch ()
+{
+  alloc_gdbarch_data (&startup_gdbarch);
+  init_gdbarch_data (&startup_gdbarch);
+}
 EOF
 
 # Create a new gdbarch struct
@@ -1260,17 +1289,25 @@ struct gdbarch *
 gdbarch_alloc (const struct gdbarch_info *info,
                struct gdbarch_tdep *tdep)
 {
-  struct gdbarch *gdbarch = XMALLOC (struct gdbarch);
-  memset (gdbarch, 0, sizeof (*gdbarch));
-
-  gdbarch->tdep = tdep;
+  /* NOTE: The new architecture variable is named \`\`current_gdbarch''
+     so that macros such as TARGET_DOUBLE_BIT, when expanded, refer to
+     the current local architecture and not the previous global
+     architecture.  This ensures that the new architectures initial
+     values are not influenced by the previous architecture.  Once
+     everything is parameterised with gdbarch, this will go away.  */
+  struct gdbarch *current_gdbarch = XMALLOC (struct gdbarch);
+  memset (current_gdbarch, 0, sizeof (*current_gdbarch));
+
+  alloc_gdbarch_data (current_gdbarch);
+
+  current_gdbarch->tdep = tdep;
 EOF
 printf "\n"
 function_list | while do_read
 do
     if class_is_info_p
     then
-       printf "  gdbarch->${function} = info->${function};\n"
+       printf "  current_gdbarch->${function} = info->${function};\n"
     fi
 done
 printf "\n"
@@ -1279,16 +1316,16 @@ function_list | while do_read
 do
     if class_is_function_p || class_is_variable_p
     then
-       if [ "${predefault}" != "" -a "${predefault}" != "0" ]
+       if [ -n "${predefault}" -a "x${predefault}" != "x0" ]
        then
-         printf "  gdbarch->${function} = ${predefault};\n"
+         printf "  current_gdbarch->${function} = ${predefault};\n"
        fi
     fi
 done
 cat <<EOF
   /* gdbarch_alloc() */
 
-  return gdbarch;
+  return current_gdbarch;
 }
 EOF
 
@@ -1336,32 +1373,32 @@ function_list | while do_read
 do
     if class_is_function_p || class_is_variable_p
     then
-       if [ "${invalid_p}" = "0" ]
+       if [ "x${invalid_p}" = "x0" ]
        then
            printf "  /* Skip verify of ${function}, invalid_p == 0 */\n"
        elif class_is_predicate_p
        then
            printf "  /* Skip verify of ${function}, has predicate */\n"
        # FIXME: See do_read for potential simplification
-       elif [ "${invalid_p}" -a "${postdefault}" ]
+       elif [ -n "${invalid_p}" -a -n "${postdefault}" ]
        then
            printf "  if (${invalid_p})\n"
            printf "    gdbarch->${function} = ${postdefault};\n"
-       elif [ "${predefault}" -a "${postdefault}" ]
+       elif [ -n "${predefault}" -a -n "${postdefault}" ]
        then
            printf "  if (gdbarch->${function} == ${predefault})\n"
            printf "    gdbarch->${function} = ${postdefault};\n"
-       elif [ "${postdefault}" ]
+       elif [ -n "${postdefault}" ]
        then
            printf "  if (gdbarch->${function} == 0)\n"
            printf "    gdbarch->${function} = ${postdefault};\n"
-       elif [ "${invalid_p}" ]
+       elif [ -n "${invalid_p}" ]
        then
            printf "  if ((GDB_MULTI_ARCH >= ${level})\n"
            printf "      && (${invalid_p}))\n"
            printf "    internal_error (__FILE__, __LINE__,\n"
            printf "                    \"gdbarch: verify_gdbarch: ${function} invalid\");\n"
-       elif [ "${predefault}" ]
+       elif [ -n "${predefault}" ]
        then
            printf "  if ((GDB_MULTI_ARCH >= ${level})\n"
            printf "      && (gdbarch->${function} == ${predefault}))\n"
@@ -1393,16 +1430,22 @@ gdbarch_dump (struct gdbarch *gdbarch, struct ui_file *file)
                       "gdbarch_dump: GDB_MULTI_ARCH = %d\\n",
                       GDB_MULTI_ARCH);
 EOF
-function_list | while do_read
+function_list | sort -t: +2 | while do_read
 do
     # multiarch functions don't have macros.
-    class_is_multiarch_p && continue
-    if [ "${returntype}" = "void" ]
+    if class_is_multiarch_p
     then
-       printf "#if defined (${macro}) && GDB_MULTI_ARCH\n"
+       printf "  if (GDB_MULTI_ARCH)\n"
+       printf "    fprintf_unfiltered (file,\n"
+       printf "                        \"gdbarch_dump: ${function} = 0x%%08lx\\\\n\",\n"
+       printf "                        (long) current_gdbarch->${function});\n"
+       continue
+    fi
+    printf "#ifdef ${macro}\n"
+    if [ "x${returntype}" = "xvoid" ]
+    then
+       printf "#if GDB_MULTI_ARCH\n"
        printf "  /* Macro might contain \`[{}]' when not multi-arch */\n"
-    else
-       printf "#ifdef ${macro}\n"
     fi
     if class_is_function_p
     then
@@ -1415,26 +1458,17 @@ do
        printf "                      \"gdbarch_dump: ${macro} # %%s\\\\n\",\n"
        printf "                      XSTRING (${macro}));\n"
     fi
-    printf "#endif\n"
-done
-function_list | while do_read
-do
-    if class_is_multiarch_p
+    if [ "x${returntype}" = "xvoid" ]
     then
-       printf "  if (GDB_MULTI_ARCH)\n"
-       printf "    fprintf_unfiltered (file,\n"
-       printf "                        \"gdbarch_dump: ${function} = 0x%%08lx\\\\n\",\n"
-       printf "                        (long) current_gdbarch->${function});\n"
-       continue
+       printf "#endif\n"
     fi
-    printf "#ifdef ${macro}\n"
-    if [ "${print_p}" = "()" ]
+    if [ "x${print_p}" = "x()" ]
     then
         printf "  gdbarch_dump_${function} (current_gdbarch);\n"
-    elif [ "${print_p}" = "0" ]
+    elif [ "x${print_p}" = "x0" ]
     then
         printf "  /* skip print of ${macro}, print_p == 0. */\n"
-    elif [ "${print_p}" ]
+    elif [ -n "${print_p}" ]
     then
         printf "  if (${print_p})\n"
        printf "    fprintf_unfiltered (file,\n"
@@ -1481,7 +1515,7 @@ do
        printf "int\n"
        printf "gdbarch_${function}_p (struct gdbarch *gdbarch)\n"
        printf "{\n"
-       if [ "${valid_p}" ]
+       if [ -n "${valid_p}" ]
        then
            printf "  return ${valid_p};\n"
        else
@@ -1493,7 +1527,7 @@ do
     then
        printf "\n"
        printf "${returntype}\n"
-       if [ "${formal}" = "void" ]
+       if [ "x${formal}" = "xvoid" ]
        then
          printf "gdbarch_${function} (struct gdbarch *gdbarch)\n"
        else
@@ -1505,7 +1539,7 @@ do
        printf "                    \"gdbarch: gdbarch_${function} invalid\");\n"
        printf "  if (gdbarch_debug >= 2)\n"
        printf "    fprintf_unfiltered (gdb_stdlog, \"gdbarch_${function} called\\\\n\");\n"
-       if [ "${actual}" = "-" -o "${actual}" = "" ]
+       if [ "x${actual}" = "x-" -o "x${actual}" = "x" ]
        then
            if class_is_multiarch_p
            then
@@ -1521,7 +1555,7 @@ do
                params="${actual}"
            fi
         fi
-               if [ "${returntype}" = "void" ]
+               if [ "x${returntype}" = "xvoid" ]
        then
          printf "  gdbarch->${function} (${params});\n"
        else
@@ -1541,15 +1575,15 @@ do
        printf "${returntype}\n"
        printf "gdbarch_${function} (struct gdbarch *gdbarch)\n"
        printf "{\n"
-       if [ "${invalid_p}" = "0" ]
+       if [ "x${invalid_p}" = "x0" ]
        then
            printf "  /* Skip verify of ${function}, invalid_p == 0 */\n"
-       elif [ "${invalid_p}" ]
+       elif [ -n "${invalid_p}" ]
        then
          printf "  if (${invalid_p})\n"
          printf "    internal_error (__FILE__, __LINE__,\n"
          printf "                    \"gdbarch: gdbarch_${function} invalid\");\n"
-       elif [ "${predefault}" ]
+       elif [ -n "${predefault}" ]
        then
          printf "  if (gdbarch->${function} == ${predefault})\n"
          printf "    internal_error (__FILE__, __LINE__,\n"
@@ -1628,81 +1662,66 @@ register_gdbarch_data (gdbarch_data_init_ftype *init,
 }
 
 
-/* Delete GDBARCH's data vector. */
+/* Walk through all the registered users initializing each in turn. */
 
 static void
-free_gdbarch_data (struct gdbarch *gdbarch)
+init_gdbarch_data (struct gdbarch *gdbarch)
 {
-  if (gdbarch->data != NULL)
+  struct gdbarch_data_registration *rego;
+  for (rego = gdbarch_data_registry.registrations;
+       rego != NULL;
+       rego = rego->next)
     {
-      struct gdbarch_data_registration *rego;
-
-      for (rego = gdbarch_data_registry.registrations;
-           rego != NULL;
-           rego = rego->next)
+      struct gdbarch_data *data = rego->data;
+      gdb_assert (data->index < gdbarch->nr_data);
+      if (data->init != NULL)
         {
-          struct gdbarch_data *data = rego->data;
-      
-          if (data->index < gdbarch->nr_data
-              && data->free != NULL
-              && gdbarch->data[data->index] != NULL)
-            {
-              data->free (gdbarch, gdbarch->data[data->index]);
-              gdbarch->data[data->index] = NULL;
-            }
+          void *pointer = data->init (gdbarch);
+          set_gdbarch_data (gdbarch, data, pointer);
         }
-      xfree (gdbarch->data);
-      gdbarch->data = NULL;
     }
 }
 
+/* Create/delete the gdbarch data vector. */
 
-/* Make sure that GDBARCH has space for all registered per-
-   architecture data.  If not, expand the table and initialize the
-   data values.  */
 static void
-check_gdbarch_data (struct gdbarch *gdbarch)
+alloc_gdbarch_data (struct gdbarch *gdbarch)
 {
-  int nr_allocated = gdbarch->nr_data;
-
-  /* How many per-architecture data items are registered so far?  */
-  int nr_registered = gdbarch_data_registry.nr;
+  gdb_assert (gdbarch->data == NULL);
+  gdbarch->nr_data = gdbarch_data_registry.nr;
+  gdbarch->data = xcalloc (gdbarch->nr_data, sizeof (void*));
+}
 
-  if (nr_allocated < nr_registered)
+static void
+free_gdbarch_data (struct gdbarch *gdbarch)
+{
+  struct gdbarch_data_registration *rego;
+  gdb_assert (gdbarch->data != NULL);
+  for (rego = gdbarch_data_registry.registrations;
+       rego != NULL;
+       rego = rego->next)
     {
-      /* Get enough room for all registered items, not just DATA.  */
-      int new_size = sizeof (gdbarch->data[0]) * nr_registered;
-      struct gdbarch_data_registration *rego;
-      
-      /* Expand the array, or perhaps allocate it for the first time.  */
-      gdbarch->data = (void **) (gdbarch->data
-                                 ? xrealloc (gdbarch->data, new_size)
-                                 : xmalloc (new_size));
-
-      /* Record the size now allocated.  */
-      gdbarch->nr_data = nr_registered;
-
-      /* Initialize the elements we just added.  */
-      for (rego = gdbarch_data_registry.registrations;
-           rego != NULL;
-           rego = rego->next)
+      struct gdbarch_data *data = rego->data;
+      gdb_assert (data->index < gdbarch->nr_data);
+      if (data->free != NULL && gdbarch->data[data->index] != NULL)
         {
-          struct gdbarch_data *data = rego->data;
-          
-          if (data->index >= nr_allocated)
-            gdbarch->data[data->index]
-              = (data->init != NULL ? data->init (gdbarch) : NULL);
+          data->free (gdbarch, gdbarch->data[data->index]);
+          gdbarch->data[data->index] = NULL;
         }
     }
+  xfree (gdbarch->data);
+  gdbarch->data = NULL;
 }
 
 
+/* Initialize the current value of thee specified per-architecture
+   data-pointer. */
+
 void
 set_gdbarch_data (struct gdbarch *gdbarch,
                   struct gdbarch_data *data,
                   void *pointer)
 {
-  check_gdbarch_data (gdbarch);
   gdb_assert (data->index < gdbarch->nr_data);
   if (data->free != NULL && gdbarch->data[data->index] != NULL)
     data->free (gdbarch, gdbarch->data[data->index]);
@@ -1715,7 +1734,6 @@ set_gdbarch_data (struct gdbarch *gdbarch,
 void *
 gdbarch_data (struct gdbarch_data *data)
 {
-  check_gdbarch_data (current_gdbarch);
   gdb_assert (data->index < current_gdbarch->nr_data);
   return current_gdbarch->data[data->index];
 }
@@ -2055,6 +2073,7 @@ gdbarch_update_p (struct gdbarch_info info)
                                new_gdbarch->bfd_arch_info->printable_name);
          current_gdbarch = new_gdbarch;
          swapin_gdbarch_swap (new_gdbarch);
+         architecture_changed_event ();
          return 1;
        }
     }
@@ -2084,6 +2103,12 @@ gdbarch_update_p (struct gdbarch_info info)
      called. */
   init_gdbarch_swap (new_gdbarch);
   
+  /* Initialize the per-architecture data-pointer of all parties that
+     registered an interest in this architecture.  CURRENT_GDBARCH
+     must be updated before these modules are called. */
+  init_gdbarch_data (new_gdbarch);
+  architecture_changed_event ();
+
   if (gdbarch_debug)
     gdbarch_dump (current_gdbarch, gdb_stdlog);
 
This page took 0.032542 seconds and 4 git commands to generate.