From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 5982 invoked by alias); 4 Oct 2003 18:13:23 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 5975 invoked from network); 4 Oct 2003 18:13:22 -0000 Received: from unknown (HELO mx1.redhat.com) (66.187.233.31) by sources.redhat.com with SMTP; 4 Oct 2003 18:13:22 -0000 Received: from int-mx1.corp.redhat.com (int-mx1.corp.redhat.com [172.16.52.254]) by mx1.redhat.com (8.11.6/8.11.6) with ESMTP id h94IDM109885 for ; Sat, 4 Oct 2003 14:13:22 -0400 Received: from pobox.corp.redhat.com (pobox.corp.redhat.com [172.16.52.156]) by int-mx1.corp.redhat.com (8.11.6/8.11.6) with ESMTP id h94IDMc16137 for ; Sat, 4 Oct 2003 14:13:22 -0400 Received: from localhost.localdomain (vpn50-46.rdu.redhat.com [172.16.50.46]) by pobox.corp.redhat.com (8.12.8/8.12.8) with ESMTP id h94IDLbe021342; Sat, 4 Oct 2003 14:13:21 -0400 Received: (from kev@localhost) by localhost.localdomain (8.11.6/8.11.6) id h94IDFd27597; Sat, 4 Oct 2003 11:13:15 -0700 Date: Sat, 04 Oct 2003 18:13:00 -0000 From: Kevin Buettner Message-Id: <1031004181315.ZM27596@localhost.localdomain> In-Reply-To: Andrew Cagney "Re: [RFA] sh-tdep.c (sh_use_struct_convention): Restructure and fix" (Oct 4, 1:35pm) References: <20031004113939.GK11435@cygbert.vinschen.de> <3F7EED21.1060902@redhat.com> <1031004170432.ZM27387@localhost.localdomain> <3F7F04D4.3060506@redhat.com> To: Andrew Cagney , Kevin Buettner Subject: Re: [RFA] sh-tdep.c (sh_use_struct_convention): Restructure and fix Cc: Corinna Vinschen , gdb-patches@sources.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-SW-Source: 2003-10/txt/msg00087.txt.bz2 On Oct 4, 1:35pm, Andrew Cagney wrote: > >> See: http://sources.redhat.com/ml/gdb-patches/2003-10/msg00033.html. > >> The ppc64_sysv_return_value code in ppc-sysv-tdep.c, has been written in > >> a way that allows a quick update to this new iterface. > > > > Andrew, > > > > There are pros and cons to the approach that you used in > > ppc64_sysv_abi_return_value(). > > > > On the pro side - and this is definitely a good thing - you keep the > > struct convention information together with the implementation of how > > to return a value. > > It is stronger than that, it moves all of the ABIs struct convention > logic to one place. Huh? What other parts are there? (I fail to see why it's stronger than what I stated.) > At present key parts of the logic are scattered > inconsistently across core parts of GDB Please explain. How did the (traditional) ppc struct return mechanisms get scattered across core parts of GDB? > The result is a longer but more correct function. More correct? There is absolutely nothing which prevents a traditional "use_struct_convention" function from being correct. What's "more correct" than "correct"? I'll grant you that it may be easier to verify correctness by having the struct convention code placed side by side with the value (extract/store) return code. And then again, it might not. I can certainly envision an architecture which has a very simple struct return convention, but for which it's immensely complicated to implement the mechanics of that convention. (E.g, maybe a lot of twiddling is extract/store parts of the code to get all the bits in the correct places.) In such a case, having an easy to read "use_struct_convention" function would make it easier to verify with respect to an ABI doc. > > But this is also a con because you've spread the definition of > > "use_struct_convention" out over a much larger number of lines. It > > isn't (IMO) as easy to comprehend when arranged in this way. > > As my spec for the interface points out, if this were an OO language it > would have a different interface. > > > The jury is still out (at least as far as I'm concerned) as to > > which approach is better. I do happen to think that your approach > > is better for ppc64 (and ppc too), but this may not necessarily be the > > case for other architectures. > > This is a technical problem. The return_value patch fixes the problem > of GDB not being able to handle all cases of: > > return return-value-in-register > > correctly. If Corinna instead implements use_struct_convention, then > that bug won't be fixed. You're referring to your WIP patch, right? (Or did I miss a commit?) You could certainly arrange to have you're WIP patch use the current interfaces, couldn't you? If you did, then that bug would be fixed even with the traditional use_struct_convention code. Kevin