From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <jemarch@gnu.org>
Received: from eggs.gnu.org (eggs.gnu.org [IPv6:2001:470:142:3::10])
 by sourceware.org (Postfix) with ESMTPS id AF3553858407
 for <cgen@sourceware.org>; Tue, 14 Sep 2021 19:01:42 +0000 (GMT)
DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org AF3553858407
Received: from fencepost.gnu.org ([2001:470:142:3::e]:35246)
 by eggs.gnu.org with esmtp (Exim 4.90_1)
 (envelope-from <jemarch@gnu.org>) id 1mQDgc-0006Xz-9W
 for cgen@sourceware.org; Tue, 14 Sep 2021 15:01:42 -0400
Received: from [141.143.193.79] (port=22230 helo=termi.gnu.org)
 by fencepost.gnu.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)
 (Exim 4.90_1) (envelope-from <jemarch@gnu.org>) id 1mQDgZ-0001DX-RJ
 for cgen@sourceware.org; Tue, 14 Sep 2021 15:01:42 -0400
From: "Jose E. Marchesi" <jemarch@gnu.org>
To: cgen@sourceware.org
Subject: Re: [PATCH] sim: add arch prefix to sim_machs
References: <20210629013755.5441-1-vapier@gentoo.org> <YTvVQ1femrUzEsCO@vapier>
Date: Tue, 14 Sep 2021 21:01:35 +0200
In-Reply-To: <YTvVQ1femrUzEsCO@vapier> (Mike Frysinger via Cgen's message of
 "Fri, 10 Sep 2021 17:59:31 -0400")
Message-ID: <87tuin56cw.fsf@gnu.org>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain
X-Spam-Status: No, score=-8.9 required=5.0 tests=BAYES_00, GIT_PATCH_0,
 KAM_DMARC_STATUS, SPF_HELO_PASS, SPF_PASS,
 TXREP autolearn=ham autolearn_force=no version=3.4.4
X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on
 server2.sourceware.org
X-BeenThere: cgen@sourceware.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Cgen mailing list <cgen.sourceware.org>
List-Unsubscribe: <https://sourceware.org/mailman/options/cgen>,
 <mailto:cgen-request@sourceware.org?subject=unsubscribe>
List-Archive: <https://sourceware.org/pipermail/cgen/>
List-Post: <mailto:cgen@sourceware.org>
List-Help: <mailto:cgen-request@sourceware.org?subject=help>
List-Subscribe: <https://sourceware.org/mailman/listinfo/cgen>,
 <mailto:cgen-request@sourceware.org?subject=subscribe>
X-List-Received-Date: Tue, 14 Sep 2021 19:01:43 -0000


> ping ...
>
> On 28 Jun 2021 21:37, Mike Frysinger via Cgen wrote:
>> We want to namespace the sim_machs structure to avoid conflicts when
>> linking multiple architectures into a single build.
>> ---
>>  sim.scm | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>> 
>> diff --git a/sim.scm b/sim.scm
>> index ae61a0939cf0..c7fe822767c3 100644
>> --- a/sim.scm
>> +++ b/sim.scm
>> @@ -1475,7 +1475,8 @@
>>  
>>  (define (/gen-mach-data)
>>    (string-append
>> -   "const SIM_MACH * const sim_machs[] =\n{\n"
>> +   "const SIM_MACH * const " (symbol->string (current-arch-name))
>> +   "_sim_machs[] =\n{\n"
>>     (string-map (lambda (mach)
>>  		 (gen-obj-sanitize
>>  		  mach
>> -- 
>> 2.31.1
>> 

This LGTM.