From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 14932 invoked by alias); 22 Jul 2012 08:28:12 -0000 Received: (qmail 14924 invoked by uid 22791); 22 Jul 2012 08:28:11 -0000 X-SWARE-Spam-Status: No, hits=-7.6 required=5.0 tests=AWL,BAYES_00,KHOP_RCVD_UNTRUST,KHOP_THREADED,RCVD_IN_DNSWL_HI,RCVD_IN_HOSTKARMA_W,SPF_HELO_PASS,TW_BJ,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Sun, 22 Jul 2012 08:27:55 +0000 Received: from int-mx12.intmail.prod.int.phx2.redhat.com (int-mx12.intmail.prod.int.phx2.redhat.com [10.5.11.25]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id q6M8RtCw025922 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Sun, 22 Jul 2012 04:27:55 -0400 Received: from spoyarek (vpn-236-237.phx2.redhat.com [10.3.236.237]) by int-mx12.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id q6M8RrgI031651 for ; Sun, 22 Jul 2012 04:27:54 -0400 Date: Sun, 22 Jul 2012 08:28:00 -0000 From: Siddhesh Poyarekar To: gdb-patches@sourceware.org Subject: Re: [PATCH] Make SIZE element for dwarf_block as size_t Message-ID: <20120722135746.1e864ddc@spoyarek> In-Reply-To: <20120722131658.2809309e@spoyarek> References: <20120722131658.2809309e@spoyarek> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="MP_/6.VgNlTMlbmXgaTWZ6zgApL" 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 X-SW-Source: 2012-07/txt/msg00423.txt.bz2 --MP_/6.VgNlTMlbmXgaTWZ6zgApL Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Content-Disposition: inline Content-length: 915 On Sun, 22 Jul 2012 13:16:58 +0530, Siddhesh wrote: > 2012-07-22 Siddhesh Poyarekar > > * dwarf2loc.h (struct dwarf2_locexpr_baton): Make SIZE as > size_t. > (struct dwarf2_loclist_baton): Likewise. > * dwarf2read.c (struct dwarf_block): Likewise. > (dump_die_shallow): Use pulongest to print dwarf_block.size. > > [1] http://sourceware.org/ml/gdb-patches/2012-06/msg00851.html A small adjustment to this patch to adjust for the expansion of dwarf_block.size -- a couple of local variables in decode_locdesc needed expansion as well. Regards, Siddhesh 2012-07-22 Siddhesh Poyarekar * dwarf2loc.h (struct dwarf2_locexpr_baton): Make SIZE as size_t. (struct dwarf2_loclist_baton): Likewise. * dwarf2read.c (struct dwarf_block): Likewise. (dump_die_shallow): Use pulongest to print dwarf_block.size. (decode_locdesc): Expand SIZE and I to size_t. --MP_/6.VgNlTMlbmXgaTWZ6zgApL Content-Type: text/x-patch Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename=dwblock-size.patch Content-length: 1948 diff --git a/gdb/dwarf2loc.h b/gdb/dwarf2loc.h index e9d06a3..0f2af3c 100644 --- a/gdb/dwarf2loc.h +++ b/gdb/dwarf2loc.h @@ -97,7 +97,7 @@ struct dwarf2_locexpr_baton /* Length of the location expression. For optimized out expressions it is zero. */ - unsigned long size; + size_t size; /* The compilation unit containing the symbol whose location we're computing. */ @@ -114,7 +114,7 @@ struct dwarf2_loclist_baton const gdb_byte *data; /* Length of the location list. */ - unsigned long size; + size_t size; /* The compilation unit containing the symbol whose location we're computing. */ diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c index 187d1e8..1d499ce 100644 --- a/gdb/dwarf2read.c +++ b/gdb/dwarf2read.c @@ -887,7 +887,7 @@ struct die_info /* Blocks are a bunch of untyped bytes. */ struct dwarf_block { - unsigned int size; + size_t size; /* Valid only if SIZE is not zero. */ gdb_byte *data; @@ -15074,12 +15074,12 @@ dump_die_shallow (struct ui_file *f, int indent, struct die_info *die) case DW_FORM_block4: case DW_FORM_block: case DW_FORM_block1: - fprintf_unfiltered (f, "block: size %d", - DW_BLOCK (&die->attrs[i])->size); + fprintf_unfiltered (f, "block: size %s", + pulongest (DW_BLOCK (&die->attrs[i])->size)); break; case DW_FORM_exprloc: - fprintf_unfiltered (f, "expression: size %u", - DW_BLOCK (&die->attrs[i])->size); + fprintf_unfiltered (f, "expression: size %s", + pulongest (DW_BLOCK (&die->attrs[i])->size)); break; case DW_FORM_ref_addr: fprintf_unfiltered (f, "ref address: "); @@ -15616,8 +15616,8 @@ static CORE_ADDR decode_locdesc (struct dwarf_block *blk, struct dwarf2_cu *cu) { struct objfile *objfile = cu->objfile; - int i; - int size = blk->size; + size_t i; + size_t size = blk->size; gdb_byte *data = blk->data; CORE_ADDR stack[64]; int stacki; --MP_/6.VgNlTMlbmXgaTWZ6zgApL--