From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 26647 invoked by alias); 15 Jan 2014 11:41:40 -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 26631 invoked by uid 89); 15 Jan 2014 11:41:39 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.3 required=5.0 tests=AWL,BAYES_00 autolearn=ham version=3.3.2 X-HELO: rock.gnat.com Received: from rock.gnat.com (HELO rock.gnat.com) (205.232.38.15) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA encrypted) ESMTPS; Wed, 15 Jan 2014 11:41:39 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id 6FBAD116585; Wed, 15 Jan 2014 06:41:37 -0500 (EST) Received: from rock.gnat.com ([127.0.0.1]) by localhost (rock.gnat.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id Jubo7vdwOMid; Wed, 15 Jan 2014 06:41:37 -0500 (EST) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id D83F011653A; Wed, 15 Jan 2014 06:41:36 -0500 (EST) Received: by joel.gnat.com (Postfix, from userid 1000) id D3B12E0816; Wed, 15 Jan 2014 15:41:33 +0400 (RET) Date: Wed, 15 Jan 2014 11:41:00 -0000 From: Joel Brobecker To: Sanimir Agovic Cc: tromey@redhat.com, palves@redhat.com, xdje42@gmail.com, gdb-patches@sourceware.org, keven.boell@intel.com Subject: Re: [PATCH v3 00/13] C99 variable length array support Message-ID: <20140115114133.GA26593@adacore.com> References: <1386166785-28037-1-git-send-email-sanimir.agovic@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1386166785-28037-1-git-send-email-sanimir.agovic@intel.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-SW-Source: 2014-01/txt/msg00494.txt.bz2 Hello, > this patch series (v3) add C99 variable length support to gdb. > > It allows the user to evaluate a vla like an ordinary static array e.g. print > its elements instead of printing the pointer to the array. In addition > the size of a vla can be retrieved with gdbs builtin sizeof operator. We've started working internally at AdaCore on getting rid of some of the "GNAT encodings", with the goal of replacing them with pure dwarf constructs. One of the issues we've noticed is precisely the lack of support for dynamically-sized arrays, and I think this patch series would be a very good stepping stone towards that. I remember I had one patch that made me a little nervous (patch #5/13, adding type re-fecthing after some value_* calls), but I also said that I would be OK if the patch went in as is. Apart from that, it seems to me that most/all? comments have been addressed? Perhaps, all we're missing is just v4 of the patch series? Any chance we could make progress on those? If there is anything I can do to assist, please let me know as well! Thank you :) -- Joel