From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-25791-listarch-gdb-patches=sources.redhat.com@sources.redhat.com>
Received: (qmail 12102 invoked by alias); 8 Jul 2003 02:26:15 -0000
Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm
Precedence: bulk
List-Subscribe: <mailto:gdb-patches-subscribe@sources.redhat.com>
List-Archive: <http://sources.redhat.com/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sources.redhat.com>
List-Help: <mailto:gdb-patches-help@sources.redhat.com>, <http://sources.redhat.com/ml/#faqs>
Sender: gdb-patches-owner@sources.redhat.com
Received: (qmail 11887 invoked from network); 8 Jul 2003 02:26:14 -0000
Received: from unknown (HELO neon-gw.transmeta.com) (63.209.4.196)
  by sources.redhat.com with SMTP; 8 Jul 2003 02:26:14 -0000
Received: (from root@localhost)
	by neon-gw.transmeta.com (8.9.3/8.9.3) id TAA07692;
	Mon, 7 Jul 2003 19:26:07 -0700
Received: from mailhost.transmeta.com(10.1.1.15) by neon-gw.transmeta.com via smap (V2.1)
	id xma007686; Mon, 7 Jul 03 19:25:57 -0700
Received: from casey.transmeta.com (casey.transmeta.com [10.10.25.22])
	by deepthought.transmeta.com (8.11.6/8.11.6) with ESMTP id h682Q2F10877;
	Mon, 7 Jul 2003 19:26:02 -0700 (PDT)
Received: (from dje@localhost) by casey.transmeta.com (8.9.3/8.7.3) id TAA16147; Mon, 7 Jul 2003 19:26:01 -0700
From: Doug Evans <dje@transmeta.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Message-ID: <16138.11193.836930.926051@casey.transmeta.com>
Date: Tue, 08 Jul 2003 02:26:00 -0000
To: "Kei Sakamoto" <sakamoto.kei@renesas.com>
Cc: "Daniel Jacobowitz" <drow@mvista.com>, <gdb-patches@sources.redhat.com>
Subject: Re: [patch/testcase] fix gdb.objc/basicclass.exp
In-Reply-To: <005101c344ec$5bfbb8e0$5169910a@KEI>
References: <013e01c341d9$b364eb30$5169910a@KEI>
	<20030707141226.GB27227@nevyn.them.org>
	<005101c344ec$5bfbb8e0$5169910a@KEI>
X-SW-Source: 2003-07/txt/msg00148.txt.bz2

Kei Sakamoto writes:
 > > On Fri, Jul 04, 2003 at 12:09:36PM +0900, Kei Sakamoto wrote:
 > > > The attached patch fixes gdb.objc.basicclass.exp.
 > > >
 > > > The current test always fail with sim because of libobjc's error.
 > > >
 > > > (gdb) PASS: gdb.objc/basicclass.exp: continue until method breakpoint
 > > > run
 > > > The program being debugged has been started already.
 > > > Start it from the beginning? (y or n) y
 > > > Starting program:
 > > > /project/m32r-tools/insight-20030703/build/gdb/testsuite/gdb.objc/basicclass
 > > > Virtual memory exhausted
 > > >
 > > > *** EXIT code 4242
 > > >
 > > > Program exited with code 0157255.
 > > > [Switching to process 0]
 > > > (gdb) FAIL: gdb.objc/basicclass.exp: resetting breakpoints when rerunning
 > > >
 > > > I assume this is not libobjc's bug. Please tell me if my
 > > > assumption is wrong.
 > > >
 > > > Anyway, I inserted load command before run command. I checked
 > > > this patch with m32r-elf, arm-elf, and sh-elf.
 > >
 > > You missed the patch - would you mind posting it?
 > 
 > I'm sorry about that. This is the patch.

I'm curious.  Why is this now necessary?
Way back when run did an implicit load, among other things
(at least for sims I remember).

One can certainly debate what semantics "run" should have.
I like "run" behaving the same across targets (unix, sim, whatever).

"run" might not make much sense to target boards
(including sims emulating target boards).
In such cases I'd argue we shouldn't be using it, rather than
trying to make it work.