* [PATCH] sim: add arch prefix to sim_machs
@ 2021-06-29 1:37 Mike Frysinger
2021-09-10 21:59 ` Mike Frysinger
0 siblings, 1 reply; 3+ messages in thread
From: Mike Frysinger @ 2021-06-29 1:37 UTC (permalink / raw)
To: cgen
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
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] sim: add arch prefix to sim_machs
2021-06-29 1:37 [PATCH] sim: add arch prefix to sim_machs Mike Frysinger
@ 2021-09-10 21:59 ` Mike Frysinger
2021-09-14 19:01 ` Jose E. Marchesi
0 siblings, 1 reply; 3+ messages in thread
From: Mike Frysinger @ 2021-09-10 21:59 UTC (permalink / raw)
To: cgen
[-- Attachment #1: Type: text/plain, Size: 709 bytes --]
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
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] sim: add arch prefix to sim_machs
2021-09-10 21:59 ` Mike Frysinger
@ 2021-09-14 19:01 ` Jose E. Marchesi
0 siblings, 0 replies; 3+ messages in thread
From: Jose E. Marchesi @ 2021-09-14 19:01 UTC (permalink / raw)
To: cgen
> 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.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-09-14 19:01 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-29 1:37 [PATCH] sim: add arch prefix to sim_machs Mike Frysinger
2021-09-10 21:59 ` Mike Frysinger
2021-09-14 19:01 ` Jose E. Marchesi
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).