From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 95113 invoked by alias); 22 Jan 2016 12:40:59 -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 95029 invoked by uid 89); 22 Jan 2016 12:40:58 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-0.8 required=5.0 tests=AWL,BAYES_00,KAM_LAZY_DOMAIN_SECURITY,RCVD_IN_DNSWL_NONE autolearn=no version=3.3.2 spammy=inconvenience, bode, Hx-languages-length:3153, dozen 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; Fri, 22 Jan 2016 12:40:57 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id 24F2D11688D; Fri, 22 Jan 2016 07:40:55 -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 0fkMG7twlz9Q; Fri, 22 Jan 2016 07:40:55 -0500 (EST) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id A2A6D11687F; Fri, 22 Jan 2016 07:40:54 -0500 (EST) Received: by joel.gnat.com (Postfix, from userid 1000) id 9720140C2A; Fri, 22 Jan 2016 16:40:50 +0400 (RET) Date: Fri, 22 Jan 2016 12:40:00 -0000 From: Joel Brobecker To: Keven Boell Cc: Yao Qi , gdb-patches@sourceware.org Subject: Re: [PATCH 1/2] fort_dyn_array: add basic fortran dyn array support Message-ID: <20160122124050.GG5146@adacore.com> References: <1435754532-17922-1-git-send-email-keven.boell@intel.com> <1435754532-17922-2-git-send-email-keven.boell@intel.com> <20150721180502.GN7406@adacore.com> <55C213C7.7070202@linux.intel.com> <20150805202301.GB14992@adacore.com> <51130.172.28.205.135.1438861308.squirrel@linux.intel.com> <20150820125159.GD4571@adacore.com> <5617A6FB.4050407@linux.intel.com> <86oacgsgdx.fsf@gmail.com> <56A1D8CD.3040905@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <56A1D8CD.3040905@linux.intel.com> User-Agent: Mutt/1.5.23 (2014-03-12) X-SW-Source: 2016-01/txt/msg00558.txt.bz2 On Fri, Jan 22, 2016 at 08:22:53AM +0100, Keven Boell wrote: > On 20.01.2016 11:18, Yao Qi wrote: > > Keven Boell writes: > > > >> Fortran provide types whose values may be dynamically allocated > >> or associated with a variable under explicit program control. > >> The purpose of this commit is > >> * to read allocated/associated DWARF tags and store them in > >> the dynamic property list of main_type. > >> * enable GDB to print the value of a dynamic array in Fortran > >> in case the type is allocated or associated (pointer to > >> dynamic array). > >> > >> Examples: > >> (gdb) p vla_not_allocated > >> $1 = > >> > >> (gdb) p vla_allocated > >> $1 = (1, 2, 3) > >> > >> (gdb) p vla_ptr_not_associated > >> $1 = > >> > >> (gdb) p vla_ptr_associated > >> $1 = (1, 2, 3) > >> > >> Add basic test coverage for most dynamic array use-cases > >> in Fortran. > >> The commit contains the following tests: > >> * Ensure that values of Fortran dynamic arrays > >> can be evaluated correctly in various ways and states. > >> * Ensure that Fortran primitives can be evaluated > >> correctly when used as a dynamic array. > >> * Dynamic arrays passed to subroutines and handled > >> in different ways inside the routine. > >> * Ensure that the ptype of dynamic arrays in > >> Fortran can be printed in GDB correctly. > >> * Ensure that dynamic arrays in different states > >> (allocated/associated) can be evaluated. > >> * Dynamic arrays passed to functions and returned from > >> functions. > >> * History values of dynamic arrays can be accessed and > >> printed again with the correct values. > >> * Dynamic array evaluations using MI protocol. > >> * Sizeof output of dynamic arrays in various states. > >> > >> The patch was tested using the test suite on Ubuntu 12.04 64bit. > > > > Hi Keven, > > The test cases added by this commit fail on some other OS and targets, > > see this thread, https://sourceware.org/ml/gdb-testers/2015-q4/msg02136.html > > can you take a look? > > > > Hi Yao, Joel, > > I don't think I will be able to fix the failures on the mentioned > hosts/targets before you create the branch, as I need to replicate the > environment on my end first to start investigating. Therefore I > suggest to revert the change for now. Sorry if this caused any > inconvenience. I don't think that reverting without more investigation is a good idea. I quickly looked at a good dozen of those reports, and, as far as I know, these are new FAILs, so we don't know that they are regressions. I could also be compiler issues. There is also the fact that reviewing these patches took me not just hours, but actually days in total (I don't know how much time Keven et al also spent answering all my comments). I would prefer re-doing all that work only if we have confirmation that this is causing a critical problem that we can't fix without Keven's help. That being said - it doesn't bode well for the future of this feature if the authors don't have time to look into issues they create... Thanks, -- Joel