From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 11049 invoked by alias); 22 Feb 2007 17:28:49 -0000 Received: (qmail 11024 invoked by uid 22791); 22 Feb 2007 17:28:49 -0000 X-Spam-Check-By: sourceware.org Received: from mail.oarcorp.com (HELO mail.oarcorp.com) (216.186.189.5) by sourceware.org (qpsmtpd/0.31) with SMTP; Thu, 22 Feb 2007 17:28:42 +0000 Received: (qmail 25756 invoked by uid 507); 22 Feb 2007 17:28:39 -0000 Received: from 192.168.1.3 by mail.oarcorp.com (envelope-from , uid 501) with qmail-scanner-1.25st (spamassassin: 3.1.1. perlscan: 1.25st. Clear:RC:1(192.168.1.3):. Processed in 0.032904 secs); 22 Feb 2007 17:28:39 -0000 X-Qmail-Scanner-Mail-From: joel.sherrill@oarcorp.com via mail.oarcorp.com X-Qmail-Scanner: 1.25st (Clear:RC:1(192.168.1.3):. Processed in 0.032904 secs Process 25750) Received: from unknown (HELO ?192.168.1.3?) (192.168.1.3) by mail.oarcorp.com with SMTP; 22 Feb 2007 17:28:39 -0000 Message-ID: <45DDD2C6.4080606@oarcorp.com> Date: Thu, 22 Feb 2007 17:28:00 -0000 From: Joel Sherrill User-Agent: Thunderbird 1.5.0.9 (X11/20070102) MIME-Version: 1.0 To: drow@false.org, gdb-patches@sourceware.org, Ralf Corsepius Subject: Re: gdb 6.6 sim/erc32 does not parse -E option References: <45DDCCAE.9060801@oarcorp.com> <20070222171239.GA13379@caradoc.them.org> In-Reply-To: <20070222171239.GA13379@caradoc.them.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit 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: 2007-02/txt/msg00275.txt.bz2 Daniel Jacobowitz wrote: > On Thu, Feb 22, 2007 at 11:02:38AM -0600, Joel Sherrill wrote: > >> Hi, >> >> Apparently gdb 6.6 passes a -E endian option >> to the sim_open hook. The erc32 simulator >> does not parse this option and produces >> a error message like this: >> >> (gdb) tar sim >> unknown option -E >> open of big failed >> Connected to the simulator. >> >> The attached patch is in the style of the >> existing code and silently eats the -E option >> if the endian value is big or little. >> > > Could you try this instead? > > http://sourceware.org/ml/gdb-patches/2007-02/msg00078.html > > With this patch, gdb still passes a -E big to sim_open in the erc32 simulator code and there is nothing to parse it at that level. AFAIK the erc32 simulator is big endian only and should just eat the option for at least endian == big. My patch just ate the option if it was either big or little and gave an error on any other value. --joel