From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 25667 invoked by alias); 6 Feb 2014 16:28:00 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 25629 invoked by uid 89); 6 Feb 2014 16:28:00 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=0.4 required=5.0 tests=AWL,BAYES_00,KAM_STOCKGEN,RP_MATCHES_RCVD,UNWANTED_LANGUAGE_BODY autolearn=no version=3.3.2 X-HELO: mga09.intel.com Received: from mga09.intel.com (HELO mga09.intel.com) (134.134.136.24) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 06 Feb 2014 16:27:59 +0000 Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga102.jf.intel.com with ESMTP; 06 Feb 2014 08:23:44 -0800 X-ExtLoop1: 1 Received: from irvmail001.ir.intel.com ([163.33.26.43]) by orsmga001.jf.intel.com with ESMTP; 06 Feb 2014 08:27:40 -0800 Received: from ullecvh004g04.iul.intel.com (ullecvh004g04.iul.intel.com [172.28.50.210]) by irvmail001.ir.intel.com (8.14.3/8.13.6/MailSET/Hub) with ESMTP id s16GRdof031070; Thu, 6 Feb 2014 16:27:39 GMT Received: from ullecvh004g04.iul.intel.com (ullecvh004g04.iul.intel.com [127.0.0.1]) by ullecvh004g04.iul.intel.com (8.13.8/8.13.8) with ESMTP id s16GRdjH025298; Thu, 6 Feb 2014 17:27:39 +0100 Received: (from sagovic@localhost) by ullecvh004g04.iul.intel.com (8.13.8/8.13.8/Submit) id s16GRcsm025297; Thu, 6 Feb 2014 17:27:38 +0100 From: Sanimir Agovic To: tromey@redhat.com, brobecker@adacore.com Cc: keven.boell@intel.com, gdb-patches@sourceware.org Subject: [PATCH v5 01/15] refactoring: rename create_range_type to create_static_range_type Date: Thu, 06 Feb 2014 16:28:00 -0000 Message-Id: <1391704056-25246-2-git-send-email-sanimir.agovic@intel.com> In-Reply-To: <1391704056-25246-1-git-send-email-sanimir.agovic@intel.com> References: <1391704056-25246-1-git-send-email-sanimir.agovic@intel.com> X-IsSubscribed: yes X-SW-Source: 2014-02/txt/msg00101.txt.bz2 2013-12-19 Sanimir Agovic Keven Boell * gdbtypes.c (create_static_range_type): Renamed from create_range_type. * gdbtypes.h (create_static_range_type): Renamed from create_range_type. * ada-lang.c (*): All uses of create_range_type updated. * coffread.c (*): All uses of create_range_type updated. * dwarf2read.c (*): All uses of create_range_type updated. * f-exp.y (*): All uses of create_range_type updated. * m2-valprint.c (*): All uses of create_range_type updated. * mdebugread.c (*): All uses of create_range_type updated. * stabsread.c (*): All uses of create_range_type updated. * valops.c (*): All uses of create_range_type updated. * valprint.c (*): All uses of create_range_type updated. Signed-off-by: Sanimir Agovic --- gdb/ada-lang.c | 22 +++++++++++----------- gdb/coffread.c | 4 ++-- gdb/dwarf2read.c | 6 +++--- gdb/f-exp.y | 9 ++++++--- gdb/gdbtypes.c | 6 +++--- gdb/gdbtypes.h | 4 ++-- gdb/m2-valprint.c | 2 +- gdb/mdebugread.c | 4 ++-- gdb/stabsread.c | 11 ++++++----- gdb/valops.c | 16 ++++++++-------- gdb/valprint.c | 2 +- 11 files changed, 45 insertions(+), 41 deletions(-) diff --git a/gdb/ada-lang.c b/gdb/ada-lang.c index 786ca7a..0d93267 100644 --- a/gdb/ada-lang.c +++ b/gdb/ada-lang.c @@ -1875,8 +1875,8 @@ ada_type_of_array (struct value *arr, int bounds) struct value *high = desc_one_bound (descriptor, arity, 1); arity -= 1; - create_range_type (range_type, value_type (low), - longest_to_int (value_as_long (low)), + create_static_range_type (range_type, value_type (low), + longest_to_int (value_as_long (low)), longest_to_int (value_as_long (high))); elt_type = create_array_type (array_type, elt_type, range_type); @@ -2649,8 +2649,8 @@ ada_value_slice_from_ptr (struct value *array_ptr, struct type *type, + ((low - ada_discrete_type_low_bound (TYPE_INDEX_TYPE (type0))) * TYPE_LENGTH (TYPE_TARGET_TYPE (type0))); struct type *index_type = - create_range_type (NULL, TYPE_TARGET_TYPE (TYPE_INDEX_TYPE (type0)), - low, high); + create_static_range_type (NULL, TYPE_TARGET_TYPE (TYPE_INDEX_TYPE (type0)), + low, high); struct type *slice_type = create_array_type (NULL, TYPE_TARGET_TYPE (type0), index_type); @@ -2663,7 +2663,7 @@ ada_value_slice (struct value *array, int low, int high) { struct type *type = ada_check_typedef (value_type (array)); struct type *index_type = - create_range_type (NULL, TYPE_INDEX_TYPE (type), low, high); + create_static_range_type (NULL, TYPE_INDEX_TYPE (type), low, high); struct type *slice_type = create_array_type (NULL, TYPE_TARGET_TYPE (type), index_type); @@ -2872,8 +2872,8 @@ empty_array (struct type *arr_type, int low) { struct type *arr_type0 = ada_check_typedef (arr_type); struct type *index_type = - create_range_type (NULL, TYPE_TARGET_TYPE (TYPE_INDEX_TYPE (arr_type0)), - low, low - 1); + create_static_range_type + (NULL, TYPE_TARGET_TYPE (TYPE_INDEX_TYPE (arr_type0)), low, low - 1); struct type *elt_type = ada_array_element_type (arr_type0, 1); return allocate_value (create_array_type (NULL, elt_type, index_type)); @@ -10853,9 +10853,8 @@ to_fixed_range_type (struct type *raw_type, struct value *dval) if (L < INT_MIN || U > INT_MAX) return raw_type; else - return create_range_type (alloc_type_copy (raw_type), raw_type, - ada_discrete_type_low_bound (raw_type), - ada_discrete_type_high_bound (raw_type)); + return create_static_range_type (alloc_type_copy (raw_type), raw_type, + L, U); } else { @@ -10918,7 +10917,8 @@ to_fixed_range_type (struct type *raw_type, struct value *dval) } } - type = create_range_type (alloc_type_copy (raw_type), base_type, L, U); + type = create_static_range_type (alloc_type_copy (raw_type), + base_type, L, U); TYPE_NAME (type) = name; return type; } diff --git a/gdb/coffread.c b/gdb/coffread.c index b49e30c..0e6cd88 100644 --- a/gdb/coffread.c +++ b/gdb/coffread.c @@ -1842,8 +1842,8 @@ decode_type (struct coff_symbol *cs, unsigned int c_type, base_type = decode_type (cs, new_c_type, aux, objfile); index_type = objfile_type (objfile)->builtin_int; range_type = - create_range_type ((struct type *) NULL, - index_type, 0, n - 1); + create_static_range_type ((struct type *) NULL, + index_type, 0, n - 1); type = create_array_type ((struct type *) NULL, base_type, range_type); diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c index 1c7dfc5..4ba25ff 100644 --- a/gdb/dwarf2read.c +++ b/gdb/dwarf2read.c @@ -13231,7 +13231,7 @@ read_array_type (struct die_info *die, struct dwarf2_cu *cu) if (die->child == NULL) { index_type = objfile_type (objfile)->builtin_int; - range_type = create_range_type (NULL, index_type, 0, -1); + range_type = create_static_range_type (NULL, index_type, 0, -1); type = create_array_type (NULL, element_type, range_type); return set_die_type (die, type, cu); } @@ -13935,7 +13935,7 @@ read_tag_string_type (struct die_info *die, struct dwarf2_cu *cu) } index_type = objfile_type (objfile)->builtin_int; - range_type = create_range_type (NULL, index_type, 1, length); + range_type = create_static_range_type (NULL, index_type, 1, length); char_type = language_string_char_type (cu->language_defn, gdbarch); type = create_string_type (NULL, char_type, range_type); @@ -14400,7 +14400,7 @@ read_subrange_type (struct die_info *die, struct dwarf2_cu *cu) if (!TYPE_UNSIGNED (base_type) && (high & negative_mask)) high |= negative_mask; - range_type = create_range_type (NULL, orig_base_type, low, high); + range_type = create_static_range_type (NULL, orig_base_type, low, high); /* Mark arrays with dynamic length at least as an array of unspecified length. GDB could check the boundary but before it gets implemented at diff --git a/gdb/f-exp.y b/gdb/f-exp.y index 567cd00..af8de68 100644 --- a/gdb/f-exp.y +++ b/gdb/f-exp.y @@ -558,10 +558,13 @@ ptype : typebase array_size = pop_type_int (); if (array_size != -1) { + struct type *index_type; + + index_type = parse_f_type->builtin_integer; range_type = - create_range_type ((struct type *) NULL, - parse_f_type->builtin_integer, - 0, array_size - 1); + create_static_range_type((struct type *) NULL, + index_type, + 0, array_size - 1); follow_type = create_array_type ((struct type *) NULL, follow_type, range_type); diff --git a/gdb/gdbtypes.c b/gdb/gdbtypes.c index 2470304..807e18e 100644 --- a/gdb/gdbtypes.c +++ b/gdb/gdbtypes.c @@ -809,8 +809,8 @@ allocate_stub_method (struct type *type) sure it is TYPE_CODE_UNDEF before we bash it into a range type? */ struct type * -create_range_type (struct type *result_type, struct type *index_type, - LONGEST low_bound, LONGEST high_bound) +create_static_range_type (struct type *result_type, struct type *index_type, + LONGEST low_bound, LONGEST high_bound) { if (result_type == NULL) result_type = alloc_type_copy (index_type); @@ -998,7 +998,7 @@ lookup_array_range_type (struct type *element_type, struct gdbarch *gdbarch = get_type_arch (element_type); struct type *index_type = builtin_type (gdbarch)->builtin_int; struct type *range_type - = create_range_type (NULL, index_type, low_bound, high_bound); + = create_static_range_type (NULL, index_type, low_bound, high_bound); return create_array_type (NULL, element_type, range_type); } diff --git a/gdb/gdbtypes.h b/gdb/gdbtypes.h index c7bef5f..0396078 100644 --- a/gdb/gdbtypes.h +++ b/gdb/gdbtypes.h @@ -1526,8 +1526,8 @@ extern struct type *lookup_function_type_with_arguments (struct type *, int, struct type **); -extern struct type *create_range_type (struct type *, struct type *, LONGEST, - LONGEST); +extern struct type *create_static_range_type (struct type *, struct type *, + LONGEST, LONGEST); extern struct type *create_array_type (struct type *, struct type *, struct type *); diff --git a/gdb/m2-valprint.c b/gdb/m2-valprint.c index f8a8f75..12c38a5 100644 --- a/gdb/m2-valprint.c +++ b/gdb/m2-valprint.c @@ -475,7 +475,7 @@ m2_val_print (struct type *type, const gdb_byte *valaddr, int embedded_offset, address, stream, recurse, original_value, options); break; } - /* FIXME: create_range_type does not set the unsigned bit in a + /* FIXME: create_static_range_type does not set the unsigned bit in a range type (I think it probably should copy it from the target type), so we won't print values which are too large to fit in a signed integer correctly. */ diff --git a/gdb/mdebugread.c b/gdb/mdebugread.c index bbeea12..b0ba3ee 100644 --- a/gdb/mdebugread.c +++ b/gdb/mdebugread.c @@ -1869,8 +1869,8 @@ upgrade_type (int fd, struct type **tpp, int tq, union aux_ext *ax, int bigend, ax++; rf = AUX_GET_WIDTH (bigend, ax); /* bit size of array element */ - range = create_range_type ((struct type *) NULL, indx, - lower, upper); + range = create_static_range_type ((struct type *) NULL, indx, + lower, upper); t = create_array_type ((struct type *) NULL, *tpp, range); diff --git a/gdb/stabsread.c b/gdb/stabsread.c index 0046772..b8e36d4 100644 --- a/gdb/stabsread.c +++ b/gdb/stabsread.c @@ -863,9 +863,9 @@ define_symbol (CORE_ADDR valu, char *string, int desc, int type, /* NULL terminate the string. */ string_local[ind] = 0; range_type - = create_range_type (NULL, - objfile_type (objfile)->builtin_int, - 0, ind); + = create_static_range_type (NULL, + objfile_type (objfile)->builtin_int, + 0, ind); SYMBOL_TYPE (sym) = create_array_type (NULL, objfile_type (objfile)->builtin_char, range_type); @@ -3615,7 +3615,7 @@ read_array_type (char **pp, struct type *type, } range_type = - create_range_type ((struct type *) NULL, index_type, lower, upper); + create_static_range_type ((struct type *) NULL, index_type, lower, upper); type = create_array_type (type, element_type, range_type); return type; @@ -4245,7 +4245,8 @@ handle_true_range: index_type = objfile_type (objfile)->builtin_int; } - result_type = create_range_type ((struct type *) NULL, index_type, n2, n3); + result_type + = create_static_range_type ((struct type *) NULL, index_type, n2, n3); return (result_type); } diff --git a/gdb/valops.c b/gdb/valops.c index d43c758..5c7bb89 100644 --- a/gdb/valops.c +++ b/gdb/valops.c @@ -415,10 +415,10 @@ value_cast (struct type *type, struct value *arg2) "divide object size in cast")); /* FIXME-type-allocation: need a way to free this type when we are done with it. */ - range_type = create_range_type ((struct type *) NULL, - TYPE_TARGET_TYPE (range_type), - low_bound, - new_length + low_bound - 1); + range_type = create_static_range_type ((struct type *) NULL, + TYPE_TARGET_TYPE (range_type), + low_bound, + new_length + low_bound - 1); deprecated_set_value_type (arg2, create_array_type ((struct type *) NULL, element_type, @@ -3569,10 +3569,10 @@ value_slice (struct value *array, int lowbound, int length) /* FIXME-type-allocation: need a way to free this type when we are done with it. */ - slice_range_type = create_range_type ((struct type *) NULL, - TYPE_TARGET_TYPE (range_type), - lowbound, - lowbound + length - 1); + slice_range_type = create_static_range_type ((struct type *) NULL, + TYPE_TARGET_TYPE (range_type), + lowbound, + lowbound + length - 1); { struct type *element_type = TYPE_TARGET_TYPE (array_type); diff --git a/gdb/valprint.c b/gdb/valprint.c index 7ebcdfd..b178109 100644 --- a/gdb/valprint.c +++ b/gdb/valprint.c @@ -597,7 +597,7 @@ generic_val_print (struct type *type, const gdb_byte *valaddr, break; case TYPE_CODE_RANGE: - /* FIXME: create_range_type does not set the unsigned bit in a + /* FIXME: create_static_range_type does not set the unsigned bit in a range type (I think it probably should copy it from the target type), so we won't print values which are too large to fit in a signed integer correctly. */ -- 1.8.4.2