X-Git-Url: https://git.libre-soc.org/?a=blobdiff_plain;f=gdb%2Fobjc-lang.c;h=d5897c036b6f2d2d6ebd33cec2d0b3e20a15b298;hb=91158a569dc571a9916dfad98c6c95ce789ad18d;hp=6fbbaa7fd2d8073c47b806096946ae63f1c62cfc;hpb=9a3d7dfd2e56889dce422be422767465d3f0f06b;p=binutils-gdb.git diff --git a/gdb/objc-lang.c b/gdb/objc-lang.c index 6fbbaa7fd2d..d5897c036b6 100644 --- a/gdb/objc-lang.c +++ b/gdb/objc-lang.c @@ -1,6 +1,7 @@ /* Objective-C language support routines for GDB, the GNU debugger. - Copyright 2002, 2003 Free Software Foundation, Inc. + Copyright (C) 2002, 2003, 2004, 2005, 2007, 2008, 2009, 2010 + Free Software Foundation, Inc. Contributed by Apple Computer, Inc. Written by Michael Snyder. @@ -9,7 +10,7 @@ This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by - the Free Software Foundation; either version 2 of the License, or + the Free Software Foundation; either version 3 of the License, or (at your option) any later version. This program is distributed in the hope that it will be useful, @@ -18,9 +19,7 @@ GNU General Public License for more details. You should have received a copy of the GNU General Public License - along with this program; if not, write to the Free Software - Foundation, Inc., 59 Temple Place - Suite 330, - Boston, MA 02111-1307, USA. */ + along with this program. If not, see . */ #include "defs.h" #include "symtab.h" @@ -30,6 +29,7 @@ #include "language.h" #include "c-lang.h" #include "objc-lang.h" +#include "exceptions.h" #include "complaints.h" #include "value.h" #include "symfile.h" @@ -42,6 +42,9 @@ #include "gdb_regex.h" #include "regcache.h" #include "block.h" +#include "infcall.h" +#include "valprint.h" +#include "gdb_assert.h" #include @@ -73,37 +76,7 @@ struct objc_method { CORE_ADDR imp; }; -/* Complaints about ObjC classes, selectors, etc. */ - -#if (!defined __GNUC__ || __GNUC__ < 2 || __GNUC_MINOR__ < (defined __cplusplus ? 6 : 4)) -#define __CHECK_FUNCTION ((__const char *) 0) -#else -#define __CHECK_FUNCTION __PRETTY_FUNCTION__ -#endif - -#define CHECK(expression) \ - ((void) ((expression) ? 0 : gdb_check (#expression, __FILE__, __LINE__, \ - __CHECK_FUNCTION))) - -#define CHECK_FATAL(expression) \ - ((void) ((expression) ? 0 : gdb_check_fatal (#expression, __FILE__, \ - __LINE__, __CHECK_FUNCTION))) - -static void -gdb_check (const char *str, const char *file, - unsigned int line, const char *func) -{ - error ("assertion failure on line %u of \"%s\" in function \"%s\": %s\n", - line, file, func, str); -} - -static void -gdb_check_fatal (const char *str, const char *file, - unsigned int line, const char *func) -{ - internal_error (file, line, - "assertion failure in function \"%s\": %s\n", func, str); -} +static const struct objfile_data *objc_objfile_data; /* Lookup a structure type named "struct NAME", visible in lexical block BLOCK. If NOERR is nonzero, return zero if NAME is not @@ -112,32 +85,32 @@ gdb_check_fatal (const char *str, const char *file, struct symbol * lookup_struct_typedef (char *name, struct block *block, int noerr) { - register struct symbol *sym; + struct symbol *sym; - sym = lookup_symbol (name, block, STRUCT_NAMESPACE, 0, - (struct symtab **) NULL); + sym = lookup_symbol (name, block, STRUCT_DOMAIN, 0); if (sym == NULL) { if (noerr) return 0; else - error ("No struct type named %s.", name); + error (_("No struct type named %s."), name); } if (TYPE_CODE (SYMBOL_TYPE (sym)) != TYPE_CODE_STRUCT) { if (noerr) return 0; else - error ("This context has class, union or enum %s, not a struct.", + error (_("This context has class, union or enum %s, not a struct."), name); } return sym; } CORE_ADDR -lookup_objc_class (char *classname) +lookup_objc_class (struct gdbarch *gdbarch, char *classname) { + struct type *char_type = builtin_type (gdbarch)->builtin_char; struct value * function, *classval; if (! target_has_execution) @@ -147,24 +120,25 @@ lookup_objc_class (char *classname) } if (lookup_minimal_symbol("objc_lookUpClass", 0, 0)) - function = find_function_in_inferior("objc_lookUpClass"); + function = find_function_in_inferior("objc_lookUpClass", NULL); else if (lookup_minimal_symbol ("objc_lookup_class", 0, 0)) - function = find_function_in_inferior("objc_lookup_class"); + function = find_function_in_inferior("objc_lookup_class", NULL); else { - complaint (&symfile_complaints, "no way to lookup Objective-C classes"); + complaint (&symfile_complaints, _("no way to lookup Objective-C classes")); return 0; } - classval = value_string (classname, strlen (classname) + 1); + classval = value_string (classname, strlen (classname) + 1, char_type); classval = value_coerce_array (classval); return (CORE_ADDR) value_as_long (call_function_by_hand (function, 1, &classval)); } -int -lookup_child_selector (char *selname) +CORE_ADDR +lookup_child_selector (struct gdbarch *gdbarch, char *selname) { + struct type *char_type = builtin_type (gdbarch)->builtin_char; struct value * function, *selstring; if (! target_has_execution) @@ -174,23 +148,24 @@ lookup_child_selector (char *selname) } if (lookup_minimal_symbol("sel_getUid", 0, 0)) - function = find_function_in_inferior("sel_getUid"); + function = find_function_in_inferior("sel_getUid", NULL); else if (lookup_minimal_symbol ("sel_get_any_uid", 0, 0)) - function = find_function_in_inferior("sel_get_any_uid"); + function = find_function_in_inferior("sel_get_any_uid", NULL); else { - complaint (&symfile_complaints, "no way to lookup Objective-C selectors"); + complaint (&symfile_complaints, _("no way to lookup Objective-C selectors")); return 0; } selstring = value_coerce_array (value_string (selname, - strlen (selname) + 1)); + strlen (selname) + 1, char_type)); return value_as_long (call_function_by_hand (function, 1, &selstring)); } struct value * -value_nsstring (char *ptr, int len) +value_nsstring (struct gdbarch *gdbarch, char *ptr, int len) { + struct type *char_type = builtin_type (gdbarch)->builtin_char; struct value *stringValue[3]; struct value *function, *nsstringValue; struct symbol *sym; @@ -199,38 +174,43 @@ value_nsstring (char *ptr, int len) if (!target_has_execution) return 0; /* Can't call into inferior to create NSString. */ - if (!(sym = lookup_struct_typedef("NSString", 0, 1)) && - !(sym = lookup_struct_typedef("NXString", 0, 1))) - type = lookup_pointer_type(builtin_type_void); - else - type = lookup_pointer_type(SYMBOL_TYPE (sym)); - - stringValue[2] = value_string(ptr, len); + stringValue[2] = value_string(ptr, len, char_type); stringValue[2] = value_coerce_array(stringValue[2]); /* _NSNewStringFromCString replaces "istr" after Lantern2A. */ if (lookup_minimal_symbol("_NSNewStringFromCString", 0, 0)) { - function = find_function_in_inferior("_NSNewStringFromCString"); + function = find_function_in_inferior("_NSNewStringFromCString", NULL); nsstringValue = call_function_by_hand(function, 1, &stringValue[2]); } else if (lookup_minimal_symbol("istr", 0, 0)) { - function = find_function_in_inferior("istr"); + function = find_function_in_inferior("istr", NULL); nsstringValue = call_function_by_hand(function, 1, &stringValue[2]); } else if (lookup_minimal_symbol("+[NSString stringWithCString:]", 0, 0)) { - function = find_function_in_inferior("+[NSString stringWithCString:]"); + function + = find_function_in_inferior("+[NSString stringWithCString:]", NULL); + type = builtin_type (gdbarch)->builtin_long; + stringValue[0] = value_from_longest - (builtin_type_long, lookup_objc_class ("NSString")); + (type, lookup_objc_class (gdbarch, "NSString")); stringValue[1] = value_from_longest - (builtin_type_long, lookup_child_selector ("stringWithCString:")); + (type, lookup_child_selector (gdbarch, "stringWithCString:")); nsstringValue = call_function_by_hand(function, 3, &stringValue[0]); } else - error ("NSString: internal error -- no way to create new NSString"); + error (_("NSString: internal error -- no way to create new NSString")); - VALUE_TYPE(nsstringValue) = type; + sym = lookup_struct_typedef("NSString", 0, 1); + if (sym == NULL) + sym = lookup_struct_typedef("NXString", 0, 1); + if (sym == NULL) + type = builtin_type (gdbarch)->builtin_data_ptr; + else + type = lookup_pointer_type(SYMBOL_TYPE (sym)); + + deprecated_set_value_type (nsstringValue, type); return nsstringValue; } @@ -264,22 +244,24 @@ objc_demangle (const char *mangled, int options) xfree(demangled); /* not mangled name */ return NULL; } - if (cp[1] == '_') { /* easy case: no category name */ - *cp++ = ' '; /* replace two '_' with one ' ' */ - strcpy(cp, mangled + (cp - demangled) + 2); - } - else { - *cp++ = '('; /* less easy case: category name */ - cp = strchr(cp, '_'); - if (!cp) - { - xfree(demangled); /* not mangled name */ - return NULL; - } - *cp++ = ')'; - *cp++ = ' '; /* overwriting 1st char of method name... */ - strcpy(cp, mangled + (cp - demangled)); /* get it back */ - } + if (cp[1] == '_') /* easy case: no category name */ + { + *cp++ = ' '; /* replace two '_' with one ' ' */ + strcpy(cp, mangled + (cp - demangled) + 2); + } + else + { + *cp++ = '('; /* less easy case: category name */ + cp = strchr(cp, '_'); + if (!cp) + { + xfree(demangled); /* not mangled name */ + return NULL; + } + *cp++ = ')'; + *cp++ = ' '; /* overwriting 1st char of method name... */ + strcpy(cp, mangled + (cp - demangled)); /* get it back */ + } while (*cp && *cp == '_') cp++; /* skip any initial underbars in method name */ @@ -301,9 +283,8 @@ objc_demangle (const char *mangled, int options) for printing characters and strings is language specific. */ static void -objc_emit_char (register int c, struct ui_file *stream, int quoter) +objc_emit_char (int c, struct type *type, struct ui_file *stream, int quoter) { - c &= 0xFF; /* Avoid sign bit follies. */ if (PRINT_LITERAL_FORM (c)) @@ -347,10 +328,10 @@ objc_emit_char (register int c, struct ui_file *stream, int quoter) } static void -objc_printchar (int c, struct ui_file *stream) +objc_printchar (int c, struct type *type, struct ui_file *stream) { fputs_filtered ("'", stream); - objc_emit_char (c, stream, '\''); + objc_emit_char (c, type, stream, '\''); fputs_filtered ("'", stream); } @@ -361,16 +342,15 @@ objc_printchar (int c, struct ui_file *stream) FORCE_ELLIPSES. */ static void -objc_printstr (struct ui_file *stream, char *string, - unsigned int length, int width, int force_ellipses) +objc_printstr (struct ui_file *stream, struct type *type, + const gdb_byte *string, unsigned int length, + const char *encoding, int force_ellipses, + const struct value_print_options *options) { - register unsigned int i; + unsigned int i; unsigned int things_printed = 0; int in_quotes = 0; int need_comma = 0; - extern int inspect_it; - extern int repeat_count_threshold; - extern int print_max; /* If the string was not truncated due to `set print elements', and the last byte of it is a null, we don't print that, in @@ -384,7 +364,7 @@ objc_printstr (struct ui_file *stream, char *string, return; } - for (i = 0; i < length && things_printed < print_max; ++i) + for (i = 0; i < length && things_printed < options->print_max; ++i) { /* Position of the character we are examining to see whether it is repeated. */ @@ -408,33 +388,33 @@ objc_printstr (struct ui_file *stream, char *string, ++reps; } - if (reps > repeat_count_threshold) + if (reps > options->repeat_count_threshold) { if (in_quotes) { - if (inspect_it) + if (options->inspect_it) fputs_filtered ("\\\", ", stream); else fputs_filtered ("\", ", stream); in_quotes = 0; } - objc_printchar (string[i], stream); + objc_printchar (string[i], type, stream); fprintf_filtered (stream, " ", reps); i = rep1 - 1; - things_printed += repeat_count_threshold; + things_printed += options->repeat_count_threshold; need_comma = 1; } else { if (!in_quotes) { - if (inspect_it) + if (options->inspect_it) fputs_filtered ("\\\"", stream); else fputs_filtered ("\"", stream); in_quotes = 1; } - objc_emit_char (string[i], stream, '"'); + objc_emit_char (string[i], type, stream, '"'); ++things_printed; } } @@ -442,7 +422,7 @@ objc_printstr (struct ui_file *stream, char *string, /* Terminate the quotes if necessary. */ if (in_quotes) { - if (inspect_it) + if (options->inspect_it) fputs_filtered ("\\\"", stream); else fputs_filtered ("\"", stream); @@ -452,147 +432,6 @@ objc_printstr (struct ui_file *stream, char *string, fputs_filtered ("...", stream); } -/* Create a fundamental C type using default reasonable for the - current target. - - Some object/debugging file formats (DWARF version 1, COFF, etc) do - not define fundamental types such as "int" or "double". Others - (stabs or DWARF version 2, etc) do define fundamental types. For - the formats which don't provide fundamental types, gdb can create - such types using this function. - - FIXME: Some compilers distinguish explicitly signed integral types - (signed short, signed int, signed long) from "regular" integral - types (short, int, long) in the debugging information. There is - some disagreement as to how useful this feature is. In particular, - gcc does not support this. Also, only some debugging formats allow - the distinction to be passed on to a debugger. For now, we always - just use "short", "int", or "long" as the type name, for both the - implicit and explicitly signed types. This also makes life easier - for the gdb test suite since we don't have to account for the - differences in output depending upon what the compiler and - debugging format support. We will probably have to re-examine the - issue when gdb starts taking it's fundamental type information - directly from the debugging information supplied by the compiler. - fnf@cygnus.com */ - -static struct type * -objc_create_fundamental_type (struct objfile *objfile, int typeid) -{ - register struct type *type = NULL; - - switch (typeid) - { - default: - /* FIXME: For now, if we are asked to produce a type not in - this language, create the equivalent of a C integer type - with the name "". When all the dust settles from - the type reconstruction work, this should probably become - an error. */ - type = init_type (TYPE_CODE_INT, - TARGET_INT_BIT / TARGET_CHAR_BIT, - 0, "", objfile); - warning ("internal error: no C/C++ fundamental type %d", typeid); - break; - case FT_VOID: - type = init_type (TYPE_CODE_VOID, - TARGET_CHAR_BIT / TARGET_CHAR_BIT, - 0, "void", objfile); - break; - case FT_CHAR: - type = init_type (TYPE_CODE_INT, - TARGET_CHAR_BIT / TARGET_CHAR_BIT, - 0, "char", objfile); - break; - case FT_SIGNED_CHAR: - type = init_type (TYPE_CODE_INT, - TARGET_CHAR_BIT / TARGET_CHAR_BIT, - 0, "signed char", objfile); - break; - case FT_UNSIGNED_CHAR: - type = init_type (TYPE_CODE_INT, - TARGET_CHAR_BIT / TARGET_CHAR_BIT, - TYPE_FLAG_UNSIGNED, "unsigned char", objfile); - break; - case FT_SHORT: - type = init_type (TYPE_CODE_INT, - TARGET_SHORT_BIT / TARGET_CHAR_BIT, - 0, "short", objfile); - break; - case FT_SIGNED_SHORT: - type = init_type (TYPE_CODE_INT, - TARGET_SHORT_BIT / TARGET_CHAR_BIT, - 0, "short", objfile); /* FIXME-fnf */ - break; - case FT_UNSIGNED_SHORT: - type = init_type (TYPE_CODE_INT, - TARGET_SHORT_BIT / TARGET_CHAR_BIT, - TYPE_FLAG_UNSIGNED, "unsigned short", objfile); - break; - case FT_INTEGER: - type = init_type (TYPE_CODE_INT, - TARGET_INT_BIT / TARGET_CHAR_BIT, - 0, "int", objfile); - break; - case FT_SIGNED_INTEGER: - type = init_type (TYPE_CODE_INT, - TARGET_INT_BIT / TARGET_CHAR_BIT, - 0, "int", objfile); /* FIXME -fnf */ - break; - case FT_UNSIGNED_INTEGER: - type = init_type (TYPE_CODE_INT, - TARGET_INT_BIT / TARGET_CHAR_BIT, - TYPE_FLAG_UNSIGNED, "unsigned int", objfile); - break; - case FT_LONG: - type = init_type (TYPE_CODE_INT, - TARGET_LONG_BIT / TARGET_CHAR_BIT, - 0, "long", objfile); - break; - case FT_SIGNED_LONG: - type = init_type (TYPE_CODE_INT, - TARGET_LONG_BIT / TARGET_CHAR_BIT, - 0, "long", objfile); /* FIXME -fnf */ - break; - case FT_UNSIGNED_LONG: - type = init_type (TYPE_CODE_INT, - TARGET_LONG_BIT / TARGET_CHAR_BIT, - TYPE_FLAG_UNSIGNED, "unsigned long", objfile); - break; - case FT_LONG_LONG: - type = init_type (TYPE_CODE_INT, - TARGET_LONG_LONG_BIT / TARGET_CHAR_BIT, - 0, "long long", objfile); - break; - case FT_SIGNED_LONG_LONG: - type = init_type (TYPE_CODE_INT, - TARGET_LONG_LONG_BIT / TARGET_CHAR_BIT, - 0, "signed long long", objfile); - break; - case FT_UNSIGNED_LONG_LONG: - type = init_type (TYPE_CODE_INT, - TARGET_LONG_LONG_BIT / TARGET_CHAR_BIT, - TYPE_FLAG_UNSIGNED, "unsigned long long", objfile); - break; - case FT_FLOAT: - type = init_type (TYPE_CODE_FLT, - TARGET_FLOAT_BIT / TARGET_CHAR_BIT, - 0, "float", objfile); - break; - case FT_DBL_PREC_FLOAT: - type = init_type (TYPE_CODE_FLT, - TARGET_DOUBLE_BIT / TARGET_CHAR_BIT, - 0, "double", objfile); - break; - case FT_EXT_PREC_FLOAT: - type = init_type (TYPE_CODE_FLT, - TARGET_LONG_DOUBLE_BIT / TARGET_CHAR_BIT, - 0, "long double", objfile); - break; - } - return (type); -} - /* Determine if we are currently in the Objective-C dispatch function. If so, get the address of the method function that the dispatcher would call and use that as the function to step into instead. Also @@ -600,12 +439,13 @@ objc_create_fundamental_type (struct objfile *objfile, int typeid) for the user since they are only interested in stepping into the method function anyway. */ static CORE_ADDR -objc_skip_trampoline (CORE_ADDR stop_pc) +objc_skip_trampoline (struct frame_info *frame, CORE_ADDR stop_pc) { + struct gdbarch *gdbarch = get_frame_arch (frame); CORE_ADDR real_stop_pc; CORE_ADDR method_stop_pc; - real_stop_pc = SKIP_TRAMPOLINE_CODE (stop_pc); + real_stop_pc = gdbarch_skip_trampoline_code (gdbarch, frame, stop_pc); if (real_stop_pc != 0) find_objc_msgcall (real_stop_pc, &method_stop_pc); @@ -614,7 +454,8 @@ objc_skip_trampoline (CORE_ADDR stop_pc) if (method_stop_pc) { - real_stop_pc = SKIP_TRAMPOLINE_CODE (method_stop_pc); + real_stop_pc = gdbarch_skip_trampoline_code + (gdbarch, frame, method_stop_pc); if (real_stop_pc == 0) real_stop_pc = method_stop_pc; } @@ -657,58 +498,43 @@ static const struct op_print objc_op_print_tab[] = {"sizeof ", UNOP_SIZEOF, PREC_PREFIX, 0}, {"++", UNOP_PREINCREMENT, PREC_PREFIX, 0}, {"--", UNOP_PREDECREMENT, PREC_PREFIX, 0}, - {NULL, 0, 0, 0} -}; - -struct type ** const (objc_builtin_types[]) = -{ - &builtin_type_int, - &builtin_type_long, - &builtin_type_short, - &builtin_type_char, - &builtin_type_float, - &builtin_type_double, - &builtin_type_void, - &builtin_type_long_long, - &builtin_type_signed_char, - &builtin_type_unsigned_char, - &builtin_type_unsigned_short, - &builtin_type_unsigned_int, - &builtin_type_unsigned_long, - &builtin_type_unsigned_long_long, - &builtin_type_long_double, - &builtin_type_complex, - &builtin_type_double_complex, - 0 + {NULL, OP_NULL, PREC_NULL, 0} }; const struct language_defn objc_language_defn = { "objective-c", /* Language name */ language_objc, - objc_builtin_types, range_check_off, type_check_off, case_sensitive_on, + array_row_major, + macro_expansion_c, + &exp_descriptor_standard, objc_parse, objc_error, - evaluate_subexp_standard, + null_post_parser, objc_printchar, /* Print a character constant */ objc_printstr, /* Function to print string constant */ objc_emit_char, - objc_create_fundamental_type, /* Create fundamental type in this language */ c_print_type, /* Print a type using appropriate syntax */ + c_print_typedef, /* Print a typedef using appropriate syntax */ c_val_print, /* Print a value using appropriate syntax */ c_value_print, /* Print a top-level value */ objc_skip_trampoline, /* Language specific skip_trampoline */ + "self", /* name_of_this */ + basic_lookup_symbol_nonlocal, /* lookup_symbol_nonlocal */ + basic_lookup_transparent_type,/* lookup_transparent_type */ objc_demangle, /* Language specific symbol demangler */ - {"", "", "", ""}, /* Binary format info */ - {"0%lo", "0", "o", ""}, /* Octal format info */ - {"%ld", "", "d", ""}, /* Decimal format info */ - {"0x%lx", "0x", "x", ""}, /* Hex format info */ + NULL, /* Language specific class_name_from_physname */ objc_op_print_tab, /* Expression operators for printing */ 1, /* C-style arrays */ 0, /* String lower bound */ - &builtin_type_char, /* Type of string elements */ + default_word_break_characters, + default_make_symbol_completion_list, + c_language_arch_info, + default_print_array_index, + default_pass_by_reference, + default_get_string, LANG_MAGIC }; @@ -731,7 +557,7 @@ static char *msglist_sel; void start_msglist(void) { - register struct selname *new = + struct selname *new = (struct selname *) xmalloc (sizeof (struct selname)); new->next = selname_chain; @@ -749,45 +575,51 @@ add_msglist(struct stoken *str, int addcolon) char *s, *p; int len, plen; - if (str == 0) { /* Unnamed arg, or... */ - if (addcolon == 0) { /* variable number of args. */ - msglist_len++; - return; + if (str == 0) /* Unnamed arg, or... */ + { + if (addcolon == 0) /* variable number of args. */ + { + msglist_len++; + return; + } + p = ""; + plen = 0; + } + else + { + p = str->ptr; + plen = str->length; } - p = ""; - plen = 0; - } else { - p = str->ptr; - plen = str->length; - } len = plen + strlen(msglist_sel) + 2; s = (char *)xmalloc(len); strcpy(s, msglist_sel); strncat(s, p, plen); xfree(msglist_sel); msglist_sel = s; - if (addcolon) { - s[len-2] = ':'; - s[len-1] = 0; - msglist_len++; - } else + if (addcolon) + { + s[len-2] = ':'; + s[len-1] = 0; + msglist_len++; + } + else s[len-2] = '\0'; } int end_msglist(void) { - register int val = msglist_len; - register struct selname *sel = selname_chain; - register char *p = msglist_sel; - int selid; + int val = msglist_len; + struct selname *sel = selname_chain; + char *p = msglist_sel; + CORE_ADDR selid; selname_chain = sel->next; msglist_len = sel->msglist_len; msglist_sel = sel->msglist_sel; - selid = lookup_child_selector(p); + selid = lookup_child_selector (parse_gdbarch, p); if (!selid) - error("Can't find selector \"%s\"", p); + error (_("Can't find selector \"%s\""), p); write_exp_elt_longcst (selid); xfree(p); write_exp_elt_longcst (val); /* Number of args */ @@ -803,7 +635,8 @@ end_msglist(void) * Used for qsorting lists of objc methods (either by class or selector). */ -int specialcmp(char *a, char *b) +static int +specialcmp (char *a, char *b) { while (*a && *a != ' ' && *a != ']' && *b && *b != ' ' && *b != ']') { @@ -833,12 +666,12 @@ compare_selectors (const void *a, const void *b) aname = SYMBOL_PRINT_NAME (*(struct symbol **) a); bname = SYMBOL_PRINT_NAME (*(struct symbol **) b); if (aname == NULL || bname == NULL) - error ("internal: compare_selectors(1)"); + error (_("internal: compare_selectors(1)")); aname = strchr(aname, ' '); bname = strchr(bname, ' '); if (aname == NULL || bname == NULL) - error ("internal: compare_selectors(2)"); + error (_("internal: compare_selectors(2)")); return specialcmp (aname+1, bname+1); } @@ -890,16 +723,17 @@ selectors_info (char *regexp, int from_tty) } if (regexp != NULL) - if (0 != (val = re_comp (myregexp))) - error ("Invalid regexp (%s): %s", val, regexp); + { + val = re_comp (myregexp); + if (val != 0) + error (_("Invalid regexp (%s): %s"), val, regexp); + } /* First time thru is JUST to get max length and count. */ ALL_MSYMBOLS (objfile, msymbol) { QUIT; - name = SYMBOL_DEMANGLED_NAME (msymbol); - if (name == NULL) - name = DEPRECATED_SYMBOL_NAME (msymbol); + name = SYMBOL_NATURAL_NAME (msymbol); if (name && (name[0] == '-' || name[0] == '+') && name[1] == '[') /* Got a method name. */ @@ -922,7 +756,7 @@ selectors_info (char *regexp, int from_tty) } if (matches) { - printf_filtered ("Selectors matching \"%s\":\n\n", + printf_filtered (_("Selectors matching \"%s\":\n\n"), regexp ? regexp : "*"); sym_arr = alloca (matches * sizeof (struct symbol *)); @@ -930,9 +764,7 @@ selectors_info (char *regexp, int from_tty) ALL_MSYMBOLS (objfile, msymbol) { QUIT; - name = SYMBOL_DEMANGLED_NAME (msymbol); - if (name == NULL) - name = DEPRECATED_SYMBOL_NAME (msymbol); + name = SYMBOL_NATURAL_NAME (msymbol); if (name && (name[0] == '-' || name[0] == '+') && name[1] == '[') /* Got a method name. */ @@ -956,9 +788,7 @@ selectors_info (char *regexp, int from_tty) char *p = asel; QUIT; - name = SYMBOL_DEMANGLED_NAME (sym_arr[ix]); - if (name == NULL) - name = DEPRECATED_SYMBOL_NAME (sym_arr[ix]); + name = SYMBOL_NATURAL_NAME (sym_arr[ix]); name = strchr (name, ' ') + 1; if (p[0] && specialcmp(name, p) == 0) continue; /* Seen this one already (not unique). */ @@ -973,7 +803,7 @@ selectors_info (char *regexp, int from_tty) begin_line(); } else - printf_filtered ("No selectors matching \"%s\"\n", regexp ? regexp : "*"); + printf_filtered (_("No selectors matching \"%s\"\n"), regexp ? regexp : "*"); } /* @@ -991,7 +821,7 @@ compare_classes (const void *a, const void *b) aname = SYMBOL_PRINT_NAME (*(struct symbol **) a); bname = SYMBOL_PRINT_NAME (*(struct symbol **) b); if (aname == NULL || bname == NULL) - error ("internal: compare_classes(1)"); + error (_("internal: compare_classes(1)")); return specialcmp (aname+1, bname+1); } @@ -1033,16 +863,17 @@ classes_info (char *regexp, int from_tty) } if (regexp != NULL) - if (0 != (val = re_comp (myregexp))) - error ("Invalid regexp (%s): %s", val, regexp); + { + val = re_comp (myregexp); + if (val != 0) + error (_("Invalid regexp (%s): %s"), val, regexp); + } /* First time thru is JUST to get max length and count. */ ALL_MSYMBOLS (objfile, msymbol) { QUIT; - name = SYMBOL_DEMANGLED_NAME (msymbol); - if (name == NULL) - name = DEPRECATED_SYMBOL_NAME (msymbol); + name = SYMBOL_NATURAL_NAME (msymbol); if (name && (name[0] == '-' || name[0] == '+') && name[1] == '[') /* Got a method name. */ @@ -1059,16 +890,14 @@ classes_info (char *regexp, int from_tty) } if (matches) { - printf_filtered ("Classes matching \"%s\":\n\n", + printf_filtered (_("Classes matching \"%s\":\n\n"), regexp ? regexp : "*"); sym_arr = alloca (matches * sizeof (struct symbol *)); matches = 0; ALL_MSYMBOLS (objfile, msymbol) { QUIT; - name = SYMBOL_DEMANGLED_NAME (msymbol); - if (name == NULL) - name = DEPRECATED_SYMBOL_NAME (msymbol); + name = SYMBOL_NATURAL_NAME (msymbol); if (name && (name[0] == '-' || name[0] == '+') && name[1] == '[') /* Got a method name. */ @@ -1085,9 +914,7 @@ classes_info (char *regexp, int from_tty) char *p = aclass; QUIT; - name = SYMBOL_DEMANGLED_NAME (sym_arr[ix]); - if (name == NULL) - name = DEPRECATED_SYMBOL_NAME (sym_arr[ix]); + name = SYMBOL_NATURAL_NAME (sym_arr[ix]); name += 2; if (p[0] && specialcmp(name, p) == 0) continue; /* Seen this one already (not unique). */ @@ -1102,7 +929,7 @@ classes_info (char *regexp, int from_tty) begin_line(); } else - printf_filtered ("No classes matching \"%s\"\n", regexp ? regexp : "*"); + printf_filtered (_("No classes matching \"%s\"\n"), regexp ? regexp : "*"); } /* @@ -1156,7 +983,7 @@ parse_selector (char *method, char **selector) char *nselector = NULL; - CHECK (selector != NULL); + gdb_assert (selector != NULL); s1 = method; @@ -1173,17 +1000,18 @@ parse_selector (char *method, char **selector) nselector = s1; s2 = s1; - for (;;) { - if (isalnum (*s2) || (*s2 == '_') || (*s2 == ':')) - *s1++ = *s2; - else if (isspace (*s2)) - ; - else if ((*s2 == '\0') || (*s2 == '\'')) - break; - else - return NULL; - s2++; - } + for (;;) + { + if (isalnum (*s2) || (*s2 == '_') || (*s2 == ':')) + *s1++ = *s2; + else if (isspace (*s2)) + ; + else if ((*s2 == '\0') || (*s2 == '\'')) + break; + else + return NULL; + s2++; + } *s1++ = '\0'; while (isspace (*s2)) @@ -1215,10 +1043,10 @@ parse_method (char *method, char *type, char **class, char *ncategory = NULL; char *nselector = NULL; - CHECK (type != NULL); - CHECK (class != NULL); - CHECK (category != NULL); - CHECK (selector != NULL); + gdb_assert (type != NULL); + gdb_assert (class != NULL); + gdb_assert (category != NULL); + gdb_assert (selector != NULL); s1 = method; @@ -1267,17 +1095,18 @@ parse_method (char *method, char *type, char **class, nselector = s2; s1 = s2; - for (;;) { - if (isalnum (*s2) || (*s2 == '_') || (*s2 == ':')) - *s1++ = *s2; - else if (isspace (*s2)) - ; - else if (*s2 == ']') - break; - else - return NULL; - s2++; - } + for (;;) + { + if (isalnum (*s2) || (*s2 == '_') || (*s2 == ':')) + *s1++ = *s2; + else if (isspace (*s2)) + ; + else if (*s2 == ']') + break; + else + return NULL; + s2++; + } *s1++ = '\0'; s2++; @@ -1328,97 +1157,128 @@ find_methods (struct symtab *symtab, char type, static char *tmp = NULL; static unsigned int tmplen = 0; - CHECK (nsym != NULL); - CHECK (ndebug != NULL); + gdb_assert (nsym != NULL); + gdb_assert (ndebug != NULL); if (symtab) block = BLOCKVECTOR_BLOCK (BLOCKVECTOR (symtab), STATIC_BLOCK); - ALL_MSYMBOLS (objfile, msymbol) + ALL_OBJFILES (objfile) { - QUIT; + unsigned int *objc_csym; - if ((msymbol->type != mst_text) && (msymbol->type != mst_file_text)) - /* Not a function or method. */ - continue; + /* The objfile_csym variable counts the number of ObjC methods + that this objfile defines. We save that count as a private + objfile data. If we have already determined that this objfile + provides no ObjC methods, we can skip it entirely. */ - if (symtab) - if ((SYMBOL_VALUE_ADDRESS (msymbol) < BLOCK_START (block)) || - (SYMBOL_VALUE_ADDRESS (msymbol) >= BLOCK_END (block))) - /* Not in the specified symtab. */ - continue; + unsigned int objfile_csym = 0; - symname = SYMBOL_DEMANGLED_NAME (msymbol); - if (symname == NULL) - symname = DEPRECATED_SYMBOL_NAME (msymbol); - if (symname == NULL) + objc_csym = objfile_data (objfile, objc_objfile_data); + if (objc_csym != NULL && *objc_csym == 0) + /* There are no ObjC symbols in this objfile. Skip it entirely. */ continue; - if ((symname[0] != '-' && symname[0] != '+') || (symname[1] != '[')) - /* Not a method name. */ - continue; - - while ((strlen (symname) + 1) >= tmplen) + ALL_OBJFILE_MSYMBOLS (objfile, msymbol) { - tmplen = (tmplen == 0) ? 1024 : tmplen * 2; - tmp = xrealloc (tmp, tmplen); - } - strcpy (tmp, symname); + struct gdbarch *gdbarch = get_objfile_arch (objfile); + CORE_ADDR pc = SYMBOL_VALUE_ADDRESS (msymbol); - if (parse_method (tmp, &ntype, &nclass, &ncategory, &nselector) == NULL) - continue; + QUIT; + + /* Check the symbol name first as this can be done entirely without + sending any query to the target. */ + symname = SYMBOL_NATURAL_NAME (msymbol); + if (symname == NULL) + continue; + + if ((symname[0] != '-' && symname[0] != '+') || (symname[1] != '[')) + /* Not a method name. */ + continue; + + /* The minimal symbol might point to a function descriptor; + resolve it to the actual code address instead. */ + pc = gdbarch_convert_from_func_ptr_addr (gdbarch, pc, + ¤t_target); + + if (symtab) + if (pc < BLOCK_START (block) || pc >= BLOCK_END (block)) + /* Not in the specified symtab. */ + continue; + + /* Now that thinks are a bit sane, clean up the symname. */ + while ((strlen (symname) + 1) >= tmplen) + { + tmplen = (tmplen == 0) ? 1024 : tmplen * 2; + tmp = xrealloc (tmp, tmplen); + } + strcpy (tmp, symname); + + if (parse_method (tmp, &ntype, &nclass, &ncategory, &nselector) == NULL) + continue; - if ((type != '\0') && (ntype != type)) - continue; + objfile_csym++; - if ((class != NULL) - && ((nclass == NULL) || (strcmp (class, nclass) != 0))) - continue; + if ((type != '\0') && (ntype != type)) + continue; - if ((category != NULL) && - ((ncategory == NULL) || (strcmp (category, ncategory) != 0))) - continue; + if ((class != NULL) + && ((nclass == NULL) || (strcmp (class, nclass) != 0))) + continue; - if ((selector != NULL) && - ((nselector == NULL) || (strcmp (selector, nselector) != 0))) - continue; + if ((category != NULL) && + ((ncategory == NULL) || (strcmp (category, ncategory) != 0))) + continue; + + if ((selector != NULL) && + ((nselector == NULL) || (strcmp (selector, nselector) != 0))) + continue; - sym = find_pc_function (SYMBOL_VALUE_ADDRESS (msymbol)); - if (sym != NULL) - { - const char *newsymname = SYMBOL_DEMANGLED_NAME (sym); + sym = find_pc_function (pc); + if (sym != NULL) + { + const char *newsymname = SYMBOL_NATURAL_NAME (sym); - if (newsymname == NULL) - newsymname = DEPRECATED_SYMBOL_NAME (sym); - if (strcmp (symname, newsymname) == 0) - { - /* Found a high-level method sym: swap it into the - lower part of sym_arr (below num_debuggable). */ - if (syms != NULL) - { - syms[csym] = syms[cdebug]; - syms[cdebug] = sym; - } - csym++; - cdebug++; - } - else - { - warning ( + if (strcmp (symname, newsymname) == 0) + { + /* Found a high-level method sym: swap it into the + lower part of sym_arr (below num_debuggable). */ + if (syms != NULL) + { + syms[csym] = syms[cdebug]; + syms[cdebug] = sym; + } + csym++; + cdebug++; + } + else + { + warning ( "debugging symbol \"%s\" does not match minimal symbol (\"%s\"); ignoring", - newsymname, symname); - if (syms != NULL) - syms[csym] = (struct symbol *) msymbol; - csym++; - } - } - else + newsymname, symname); + if (syms != NULL) + syms[csym] = (struct symbol *) msymbol; + csym++; + } + } + else + { + /* Found a non-debuggable method symbol. */ + if (syms != NULL) + syms[csym] = (struct symbol *) msymbol; + csym++; + } + } + if (objc_csym == NULL) { - /* Found a non-debuggable method symbol. */ - if (syms != NULL) - syms[csym] = (struct symbol *) msymbol; - csym++; + objc_csym = obstack_alloc (&objfile->objfile_obstack, + sizeof (*objc_csym)); + *objc_csym = objfile_csym; + set_objfile_data (objfile, objc_objfile_data, objc_csym); } + else + /* Count of ObjC methods in this objfile should be constant. */ + gdb_assert (*objc_csym == objfile_csym); } if (nsym != NULL) @@ -1445,8 +1305,8 @@ char *find_imps (struct symtab *symtab, struct block *block, char *buf = NULL; char *tmp = NULL; - CHECK (nsym != NULL); - CHECK (ndebug != NULL); + gdb_assert (nsym != NULL); + gdb_assert (ndebug != NULL); if (nsym != NULL) *nsym = 0; @@ -1457,37 +1317,36 @@ char *find_imps (struct symtab *symtab, struct block *block, strcpy (buf, method); tmp = parse_method (buf, &type, &class, &category, &selector); - if (tmp == NULL) { - - struct symtab *sym_symtab = NULL; - struct symbol *sym = NULL; - struct minimal_symbol *msym = NULL; - - strcpy (buf, method); - tmp = parse_selector (buf, &selector); - - if (tmp == NULL) - return NULL; - - sym = lookup_symbol (selector, block, VAR_NAMESPACE, 0, &sym_symtab); - if (sym != NULL) - { - if (syms) - syms[csym] = sym; - csym++; - cdebug++; - } + if (tmp == NULL) + { + struct symbol *sym = NULL; + struct minimal_symbol *msym = NULL; - if (sym == NULL) - msym = lookup_minimal_symbol (selector, 0, 0); + strcpy (buf, method); + tmp = parse_selector (buf, &selector); - if (msym != NULL) - { - if (syms) - syms[csym] = (struct symbol *)msym; - csym++; - } - } + if (tmp == NULL) + return NULL; + + sym = lookup_symbol (selector, block, VAR_DOMAIN, 0); + if (sym != NULL) + { + if (syms) + syms[csym] = sym; + csym++; + cdebug++; + } + + if (sym == NULL) + msym = lookup_minimal_symbol (selector, 0, 0); + + if (msym != NULL) + { + if (syms) + syms[csym] = (struct symbol *)msym; + csym++; + } + } if (syms != NULL) find_methods (symtab, type, class, category, selector, @@ -1557,13 +1416,13 @@ char *find_imps (struct symtab *symtab, struct block *block, return method + (tmp - buf); } -void +static void print_object_command (char *args, int from_tty) { struct value *object, *function, *description; CORE_ADDR string_addr, object_addr; int i = 0; - char c = -1; + gdb_byte c = 0; if (!args || !*args) error ( @@ -1571,12 +1430,12 @@ print_object_command (char *args, int from_tty) { struct expression *expr = parse_expression (args); - register struct cleanup *old_chain = + struct cleanup *old_chain = make_cleanup (free_current_contents, &expr); int pc = 0; - object = expr->language_defn->evaluate_exp (builtin_type_void_data_ptr, - expr, &pc, EVAL_NORMAL); + object = evaluate_subexp (builtin_type (expr->gdbarch)->builtin_data_ptr, + expr, &pc, EVAL_NORMAL); do_cleanups (old_chain); } @@ -1584,18 +1443,18 @@ print_object_command (char *args, int from_tty) object_addr = value_as_long (object); read_memory (object_addr, &c, 1); - function = find_function_in_inferior ("_NSPrintForDebugger"); + function = find_function_in_inferior ("_NSPrintForDebugger", NULL); if (function == NULL) - error ("Unable to locate _NSPrintForDebugger in child process"); + error (_("Unable to locate _NSPrintForDebugger in child process")); description = call_function_by_hand (function, 1, &object); string_addr = value_as_long (description); if (string_addr == 0) - error ("object returns null description"); + error (_("object returns null description")); read_memory (string_addr + i++, &c, 1); - if (c != '\0') + if (c != 0) do { /* Read and print characters up to EOS. */ QUIT; @@ -1603,7 +1462,7 @@ print_object_command (char *args, int from_tty) read_memory (string_addr + i++, &c, 1); } while (c != 0); else - printf_filtered(""); + printf_filtered(_("")); printf_filtered ("\n"); } @@ -1645,30 +1504,33 @@ static struct objc_methcall methcalls[] = { * case the functions have moved for some reason. */ -void +static void find_objc_msgsend (void) { unsigned int i; - for (i = 0; i < nmethcalls; i++) { - struct minimal_symbol *func; + for (i = 0; i < nmethcalls; i++) + { + struct minimal_symbol *func; - /* Try both with and without underscore. */ - func = lookup_minimal_symbol (methcalls[i].name, NULL, NULL); - if ((func == NULL) && (methcalls[i].name[0] == '_')) { - func = lookup_minimal_symbol (methcalls[i].name + 1, NULL, NULL); - } - if (func == NULL) { - methcalls[i].begin = 0; - methcalls[i].end = 0; - continue; + /* Try both with and without underscore. */ + func = lookup_minimal_symbol (methcalls[i].name, NULL, NULL); + if ((func == NULL) && (methcalls[i].name[0] == '_')) + { + func = lookup_minimal_symbol (methcalls[i].name + 1, NULL, NULL); + } + if (func == NULL) + { + methcalls[i].begin = 0; + methcalls[i].end = 0; + continue; + } + + methcalls[i].begin = SYMBOL_VALUE_ADDRESS (func); + do { + methcalls[i].end = SYMBOL_VALUE_ADDRESS (++func); + } while (methcalls[i].begin == methcalls[i].end); } - - methcalls[i].begin = SYMBOL_VALUE_ADDRESS (func); - do { - methcalls[i].end = SYMBOL_VALUE_ADDRESS (++func); - } while (methcalls[i].begin == methcalls[i].end); - } } /* find_objc_msgcall (replaces pc_off_limits) @@ -1681,7 +1543,7 @@ find_objc_msgsend (void) * The old function "pc_off_limits" used to do a lot of other things * in addition, such as detecting shared library jump stubs and * returning the address of the shlib function that would be called. - * That functionality has been moved into the SKIP_TRAMPOLINE_CODE and + * That functionality has been moved into the gdbarch_skip_trampoline_code and * IN_SOLIB_TRAMPOLINE macros, which are resolved in the target- * dependent modules. */ @@ -1692,7 +1554,7 @@ struct objc_submethod_helper_data { CORE_ADDR *new_pc; }; -int +static int find_objc_msgcall_submethod_helper (void * arg) { struct objc_submethod_helper_data *s = @@ -1704,7 +1566,7 @@ find_objc_msgcall_submethod_helper (void * arg) return 0; } -int +static int find_objc_msgcall_submethod (int (*f) (CORE_ADDR, CORE_ADDR *), CORE_ADDR pc, CORE_ADDR *new_pc) @@ -1730,7 +1592,10 @@ find_objc_msgcall (CORE_ADDR pc, CORE_ADDR *new_pc) unsigned int i; find_objc_msgsend (); - if (new_pc != NULL) { *new_pc = 0; } + if (new_pc != NULL) + { + *new_pc = 0; + } for (i = 0; i < nmethcalls; i++) if ((pc >= methcalls[i].begin) && (pc < methcalls[i].end)) @@ -1745,112 +1610,90 @@ find_objc_msgcall (CORE_ADDR pc, CORE_ADDR *new_pc) return 0; } +extern initialize_file_ftype _initialize_objc_language; /* -Wmissing-prototypes */ + void _initialize_objc_language (void) { add_language (&objc_language_defn); add_info ("selectors", selectors_info, /* INFO SELECTORS command. */ - "All Objective-C selectors, or those matching REGEXP."); + _("All Objective-C selectors, or those matching REGEXP.")); add_info ("classes", classes_info, /* INFO CLASSES command. */ - "All Objective-C classes, or those matching REGEXP."); + _("All Objective-C classes, or those matching REGEXP.")); add_com ("print-object", class_vars, print_object_command, - "Ask an Objective-C object to print itself."); + _("Ask an Objective-C object to print itself.")); add_com_alias ("po", "print-object", class_vars, 1); } -#if defined (__powerpc__) || defined (__ppc__) -static unsigned long FETCH_ARGUMENT (int i) -{ - return read_register (3 + i); -} -#elif defined (__i386__) -static unsigned long FETCH_ARGUMENT (int i) -{ - CORE_ADDR stack = read_register (SP_REGNUM); - return read_memory_unsigned_integer (stack + (4 * (i + 1)), 4); -} -#elif defined (__sparc__) -static unsigned long FETCH_ARGUMENT (int i) -{ - return read_register (O0_REGNUM + i); -} -#elif defined (__hppa__) || defined (__hppa) -static unsigned long FETCH_ARGUMENT (int i) -{ - return read_register (R0_REGNUM + 26 - i); -} -#else -#error unknown architecture -#endif - -#if defined (__hppa__) || defined (__hppa) -static CORE_ADDR CONVERT_FUNCPTR (CORE_ADDR pc) +static void +read_objc_method (struct gdbarch *gdbarch, CORE_ADDR addr, + struct objc_method *method) { - if (pc & 0x2) - pc = (CORE_ADDR) read_memory_integer (pc & ~0x3, 4); + enum bfd_endian byte_order = gdbarch_byte_order (gdbarch); - return pc; + method->name = read_memory_unsigned_integer (addr + 0, 4, byte_order); + method->types = read_memory_unsigned_integer (addr + 4, 4, byte_order); + method->imp = read_memory_unsigned_integer (addr + 8, 4, byte_order); } -#else -static CORE_ADDR CONVERT_FUNCPTR (CORE_ADDR pc) -{ - return pc; -} -#endif -static void -read_objc_method (CORE_ADDR addr, struct objc_method *method) +static unsigned long +read_objc_methlist_nmethods (struct gdbarch *gdbarch, CORE_ADDR addr) { - method->name = read_memory_unsigned_integer (addr + 0, 4); - method->types = read_memory_unsigned_integer (addr + 4, 4); - method->imp = read_memory_unsigned_integer (addr + 8, 4); -} + enum bfd_endian byte_order = gdbarch_byte_order (gdbarch); -static -unsigned long read_objc_methlist_nmethods (CORE_ADDR addr) -{ - return read_memory_unsigned_integer (addr + 4, 4); + return read_memory_unsigned_integer (addr + 4, 4, byte_order); } static void -read_objc_methlist_method (CORE_ADDR addr, unsigned long num, - struct objc_method *method) +read_objc_methlist_method (struct gdbarch *gdbarch, CORE_ADDR addr, + unsigned long num, struct objc_method *method) { - CHECK_FATAL (num < read_objc_methlist_nmethods (addr)); - read_objc_method (addr + 8 + (12 * num), method); + gdb_assert (num < read_objc_methlist_nmethods (gdbarch, addr)); + read_objc_method (gdbarch, addr + 8 + (12 * num), method); } static void -read_objc_object (CORE_ADDR addr, struct objc_object *object) +read_objc_object (struct gdbarch *gdbarch, CORE_ADDR addr, + struct objc_object *object) { - object->isa = read_memory_unsigned_integer (addr, 4); + enum bfd_endian byte_order = gdbarch_byte_order (gdbarch); + + object->isa = read_memory_unsigned_integer (addr, 4, byte_order); } static void -read_objc_super (CORE_ADDR addr, struct objc_super *super) +read_objc_super (struct gdbarch *gdbarch, CORE_ADDR addr, + struct objc_super *super) { - super->receiver = read_memory_unsigned_integer (addr, 4); - super->class = read_memory_unsigned_integer (addr + 4, 4); + enum bfd_endian byte_order = gdbarch_byte_order (gdbarch); + + super->receiver = read_memory_unsigned_integer (addr, 4, byte_order); + super->class = read_memory_unsigned_integer (addr + 4, 4, byte_order); }; static void -read_objc_class (CORE_ADDR addr, struct objc_class *class) +read_objc_class (struct gdbarch *gdbarch, CORE_ADDR addr, + struct objc_class *class) { - class->isa = read_memory_unsigned_integer (addr, 4); - class->super_class = read_memory_unsigned_integer (addr + 4, 4); - class->name = read_memory_unsigned_integer (addr + 8, 4); - class->version = read_memory_unsigned_integer (addr + 12, 4); - class->info = read_memory_unsigned_integer (addr + 16, 4); - class->instance_size = read_memory_unsigned_integer (addr + 18, 4); - class->ivars = read_memory_unsigned_integer (addr + 24, 4); - class->methods = read_memory_unsigned_integer (addr + 28, 4); - class->cache = read_memory_unsigned_integer (addr + 32, 4); - class->protocols = read_memory_unsigned_integer (addr + 36, 4); + enum bfd_endian byte_order = gdbarch_byte_order (gdbarch); + + class->isa = read_memory_unsigned_integer (addr, 4, byte_order); + class->super_class = read_memory_unsigned_integer (addr + 4, 4, byte_order); + class->name = read_memory_unsigned_integer (addr + 8, 4, byte_order); + class->version = read_memory_unsigned_integer (addr + 12, 4, byte_order); + class->info = read_memory_unsigned_integer (addr + 16, 4, byte_order); + class->instance_size = read_memory_unsigned_integer (addr + 18, 4, byte_order); + class->ivars = read_memory_unsigned_integer (addr + 24, 4, byte_order); + class->methods = read_memory_unsigned_integer (addr + 28, 4, byte_order); + class->cache = read_memory_unsigned_integer (addr + 32, 4, byte_order); + class->protocols = read_memory_unsigned_integer (addr + 36, 4, byte_order); } -CORE_ADDR -find_implementation_from_class (CORE_ADDR class, CORE_ADDR sel) +static CORE_ADDR +find_implementation_from_class (struct gdbarch *gdbarch, + CORE_ADDR class, CORE_ADDR sel) { + enum bfd_endian byte_order = gdbarch_byte_order (gdbarch); CORE_ADDR subclass = class; while (subclass != 0) @@ -1859,7 +1702,7 @@ find_implementation_from_class (CORE_ADDR class, CORE_ADDR sel) struct objc_class class_str; unsigned mlistnum = 0; - read_objc_class (subclass, &class_str); + read_objc_class (gdbarch, subclass, &class_str); for (;;) { @@ -1868,17 +1711,18 @@ find_implementation_from_class (CORE_ADDR class, CORE_ADDR sel) unsigned long i; mlist = read_memory_unsigned_integer (class_str.methods + - (4 * mlistnum), 4); + (4 * mlistnum), + 4, byte_order); if (mlist == 0) break; - nmethods = read_objc_methlist_nmethods (mlist); + nmethods = read_objc_methlist_nmethods (gdbarch, mlist); for (i = 0; i < nmethods; i++) { struct objc_method meth_str; - read_objc_methlist_method (mlist, i, &meth_str); + read_objc_methlist_method (gdbarch, mlist, i, &meth_str); #if 0 fprintf (stderr, "checking method 0x%lx against selector 0x%lx\n", @@ -1886,7 +1730,9 @@ find_implementation_from_class (CORE_ADDR class, CORE_ADDR sel) #endif if (meth_str.name == sel) - return CONVERT_FUNCPTR (meth_str.imp); + /* FIXME: hppa arch was doing a pointer dereference + here. There needs to be a better way to do that. */ + return meth_str.imp; } mlistnum++; } @@ -1896,31 +1742,36 @@ find_implementation_from_class (CORE_ADDR class, CORE_ADDR sel) return 0; } -CORE_ADDR -find_implementation (CORE_ADDR object, CORE_ADDR sel) +static CORE_ADDR +find_implementation (struct gdbarch *gdbarch, + CORE_ADDR object, CORE_ADDR sel) { struct objc_object ostr; if (object == 0) return 0; - read_objc_object (object, &ostr); + read_objc_object (gdbarch, object, &ostr); if (ostr.isa == 0) return 0; - return find_implementation_from_class (ostr.isa, sel); + return find_implementation_from_class (gdbarch, ostr.isa, sel); } static int resolve_msgsend (CORE_ADDR pc, CORE_ADDR *new_pc) { + struct frame_info *frame = get_current_frame (); + struct gdbarch *gdbarch = get_frame_arch (frame); + struct type *ptr_type = builtin_type (gdbarch)->builtin_func_ptr; + CORE_ADDR object; CORE_ADDR sel; CORE_ADDR res; - object = FETCH_ARGUMENT (0); - sel = FETCH_ARGUMENT (1); + object = gdbarch_fetch_pointer_argument (gdbarch, frame, 0, ptr_type); + sel = gdbarch_fetch_pointer_argument (gdbarch, frame, 1, ptr_type); - res = find_implementation (object, sel); + res = find_implementation (gdbarch, object, sel); if (new_pc != 0) *new_pc = res; if (res == 0) @@ -1931,14 +1782,18 @@ resolve_msgsend (CORE_ADDR pc, CORE_ADDR *new_pc) static int resolve_msgsend_stret (CORE_ADDR pc, CORE_ADDR *new_pc) { + struct frame_info *frame = get_current_frame (); + struct gdbarch *gdbarch = get_frame_arch (frame); + struct type *ptr_type = builtin_type (gdbarch)->builtin_func_ptr; + CORE_ADDR object; CORE_ADDR sel; CORE_ADDR res; - object = FETCH_ARGUMENT (1); - sel = FETCH_ARGUMENT (2); + object = gdbarch_fetch_pointer_argument (gdbarch, frame, 1, ptr_type); + sel = gdbarch_fetch_pointer_argument (gdbarch, frame, 2, ptr_type); - res = find_implementation (object, sel); + res = find_implementation (gdbarch, object, sel); if (new_pc != 0) *new_pc = res; if (res == 0) @@ -1949,20 +1804,24 @@ resolve_msgsend_stret (CORE_ADDR pc, CORE_ADDR *new_pc) static int resolve_msgsend_super (CORE_ADDR pc, CORE_ADDR *new_pc) { + struct frame_info *frame = get_current_frame (); + struct gdbarch *gdbarch = get_frame_arch (frame); + struct type *ptr_type = builtin_type (gdbarch)->builtin_func_ptr; + struct objc_super sstr; CORE_ADDR super; CORE_ADDR sel; CORE_ADDR res; - super = FETCH_ARGUMENT (0); - sel = FETCH_ARGUMENT (1); + super = gdbarch_fetch_pointer_argument (gdbarch, frame, 0, ptr_type); + sel = gdbarch_fetch_pointer_argument (gdbarch, frame, 1, ptr_type); - read_objc_super (super, &sstr); + read_objc_super (gdbarch, super, &sstr); if (sstr.class == 0) return 0; - res = find_implementation_from_class (sstr.class, sel); + res = find_implementation_from_class (gdbarch, sstr.class, sel); if (new_pc != 0) *new_pc = res; if (res == 0) @@ -1973,23 +1832,33 @@ resolve_msgsend_super (CORE_ADDR pc, CORE_ADDR *new_pc) static int resolve_msgsend_super_stret (CORE_ADDR pc, CORE_ADDR *new_pc) { + struct frame_info *frame = get_current_frame (); + struct gdbarch *gdbarch = get_frame_arch (frame); + struct type *ptr_type = builtin_type (gdbarch)->builtin_func_ptr; + struct objc_super sstr; CORE_ADDR super; CORE_ADDR sel; CORE_ADDR res; - super = FETCH_ARGUMENT (1); - sel = FETCH_ARGUMENT (2); + super = gdbarch_fetch_pointer_argument (gdbarch, frame, 1, ptr_type); + sel = gdbarch_fetch_pointer_argument (gdbarch, frame, 2, ptr_type); - read_objc_super (super, &sstr); + read_objc_super (gdbarch, super, &sstr); if (sstr.class == 0) return 0; - res = find_implementation_from_class (sstr.class, sel); + res = find_implementation_from_class (gdbarch, sstr.class, sel); if (new_pc != 0) *new_pc = res; if (res == 0) return 1; return 0; } + +void +_initialize_objc_lang (void) +{ + objc_objfile_data = register_objfile_data (); +}