From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 10357 invoked by alias); 28 Jun 2007 06:11:29 -0000 Received: (qmail 10347 invoked by uid 22791); 28 Jun 2007 06:11:28 -0000 X-Spam-Check-By: sourceware.org Received: from mail.codesourcery.com (HELO mail.codesourcery.com) (65.74.133.4) by sourceware.org (qpsmtpd/0.31) with ESMTP; Thu, 28 Jun 2007 06:11:22 +0000 Received: (qmail 4229 invoked from network); 28 Jun 2007 06:11:20 -0000 Received: from unknown (HELO gateway) (10.0.0.100) by mail.codesourcery.com with SMTP; 28 Jun 2007 06:11:20 -0000 Received: by gateway (Postfix, from userid 1010) id 799FD6C117; Wed, 27 Jun 2007 23:11:20 -0700 (PDT) From: Richard Sandiford To: Joel Brobecker Mail-Followup-To: Joel Brobecker ,gdb-patches@sourceware.org, richard@codesourcery.com Cc: gdb-patches@sourceware.org Subject: Re: Make all four mipsisa* simulators support mipsisa{32,64}{,r2} References: <878xa81c99.fsf@firetop.home> <20070626185001.GG3700@adacore.com> Date: Thu, 28 Jun 2007 09:41:00 -0000 In-Reply-To: <20070626185001.GG3700@adacore.com> (Joel Brobecker's message of "Tue, 26 Jun 2007 14:50:01 -0400") Message-ID: <87tzsslibc.fsf@firetop.home> User-Agent: Gnus/5.110006 (No Gnus v0.6) Emacs/21.4 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii 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-06/txt/msg00487.txt.bz2 Joel Brobecker writes: >> Although most mipsisa* GNU tools support all four of MIPS32, MIP32r2, >> MIPS64 and MIPS64r2, the gdb simulator doesn't. So while the gcc >> configurations build multilibs for all four targets, it usually isn't >> possible to test them all using one configuration's simulator. > > [..] > >> sim/mips/ >> * configure.ac (sim_mipsisa3264_configs): New variable. >> (mipsis32*-*-, mipsisa32r2*-*-*, mips64*-*-*, mips64r2*-*-*): Make >> every configuration support all four targets, using the triplet to >> determine the default. >> * configure: Regenerate. > > I think this deserves an entry in gdb/NEWS. Events have proven otherwise. I can't have tested the patch as well as I'd thought: I think the only mips32 code I tested on mipsisa64 must have been soft-float. It turns out that mips.igen's implementation of ldc1 does not work when WITH_TARGET_WORD_SIZE==64. The different W_T_W_S probably causes other (perhaps more subtle) problems in the same line, so I've decided to revert the patch. I'd still like to make the change in principle, but it clearly needs more work, and I don't have time for that at the moment. Sorry for screwing this up. Richard