From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id po7DEjHxpWA9TgAAWB0awg (envelope-from ) for ; Thu, 20 May 2021 01:18:41 -0400 Received: by simark.ca (Postfix, from userid 112) id 3D1891F11C; Thu, 20 May 2021 01:18:41 -0400 (EDT) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on simark.ca X-Spam-Level: X-Spam-Status: No, score=-1.1 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,MAILING_LIST_MULTI,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.2 Received: from sourceware.org (server2.sourceware.org [8.43.85.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id 6ACD61E54D for ; Thu, 20 May 2021 01:18:40 -0400 (EDT) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 8C353393F872; Thu, 20 May 2021 05:18:39 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 8C353393F872 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1621487919; bh=xkAJxpze1bHRCIAn84fGFUmvGZaAPCkIqGYlSesi0hM=; h=Date:To:Subject:References:In-Reply-To:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc: From; b=gWJnkEwKrRdXrTADF93moIbfwNHyvJOJKeQ25AN9+EIVELrMllFdEz/sAXuBgyF/W KZduPCXRg/tnmIGUT1DWGjqkDI37FhH69LcFf8xmvsclu8GSzpvEFGqMhqWfFaktMa tn52UEG/Q4KaeRExBjeliZd31VYN6PFigm9Cf3F0= Received: from smtp.gentoo.org (woodpecker.gentoo.org [IPv6:2001:470:ea4a:1:5054:ff:fec7:86e4]) by sourceware.org (Postfix) with ESMTP id DFCEF3857026; Thu, 20 May 2021 05:18:34 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org DFCEF3857026 Received: from vapier (localhost [127.0.0.1]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id 947BB340D58; Thu, 20 May 2021 05:18:32 +0000 (UTC) Date: Thu, 20 May 2021 01:18:32 -0400 To: Hans-Peter Nilsson Subject: Re: [PATCH] opcodes: cris: move desc & opc files from sim/ Message-ID: Mail-Followup-To: Hans-Peter Nilsson , binutils@sourceware.org, gdb-patches@sourceware.org References: <20210519212626.58A6A203C3@pchp3.se.axis.com> <20210519213237.8C6CA203C1@pchp3.se.axis.com> <20210519234827.86E7C203C6@pchp3.se.axis.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20210519234827.86E7C203C6@pchp3.se.axis.com> X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Mike Frysinger via Gdb-patches Reply-To: Mike Frysinger Cc: binutils@sourceware.org, gdb-patches@sourceware.org Errors-To: gdb-patches-bounces@sourceware.org Sender: "Gdb-patches" On 20 May 2021 01:48, Hans-Peter Nilsson wrote: > From: Mike Frysinger > > On 19 May 2021 23:32, Hans-Peter Nilsson wrote: > > > From: Hans-Peter Nilsson > > > > From: Mike Frysinger via Binutils > > > > > All other cgen ports keep their desc & opc files under opcodes/, so > > > > > move the cris files over too. The cris-opc.c file is already here. > > > > > > > > Unline (most) other CGEN ports, the assembler and > > > > disassembler isn't generated from the CGEN description. But > > > > that's not a reason to keep it different in this regard. > > > > > > > > Ok with the appropriate opcodes/ChangeLog entry. Thanks! > > > > > > No, wait, NOT ok: cris-opc.c isn't CGEN-generated. Please > > > fix so it isn't "regenerated". > > > > you want me to delete the cris cgen rule entirely ? > > "No" if you mean removing the regeneration rule for the > CGEN-generated CRIS files of course. > > But, if you want to move the CRIS CGEN files to opcodes/, > there must be precautions to not overwrite the cris-opc.c > there. from what i can tell, the opcodes/ script has no support for only generating a single file. it wants to do them all. so you want me to pull the desc logic out of sim/common/cgen.sh and into opcodes/cgen.sh so that cris can run that logic ? it's hard to test as the cris cgen rules fail for me even now with syntax errors in the scm files. -mike