* [PATCH 1/1] target/loongarch: move translate modules to tcg/
@ 2023-12-29 9:24 Song Gao
2023-12-29 10:08 ` Philippe Mathieu-Daudé
0 siblings, 1 reply; 3+ messages in thread
From: Song Gao @ 2023-12-29 9:24 UTC (permalink / raw)
To: qemu-devel
Cc: richard.henderson, peter.maydell, philmd, alex.bennee, maobibo,
zhaotianrui
Introduce the target/loongarch/tcg directory. Its purpose is to hold the TCG
code that is selected by CONFIG_TCG
Signed-off-by: Song Gao <gaosong@loongson.cn>
---
target/loongarch/{ => tcg}/constant_timer.c | 0
target/loongarch/{ => tcg}/csr_helper.c | 0
target/loongarch/{ => tcg}/fpu_helper.c | 0
target/loongarch/{ => tcg}/iocsr_helper.c | 0
target/loongarch/{ => tcg}/op_helper.c | 0
target/loongarch/{ => tcg}/tlb_helper.c | 0
target/loongarch/{ => tcg}/translate.c | 0
target/loongarch/{ => tcg}/vec_helper.c | 0
.../{ => tcg}/insn_trans/trans_arith.c.inc | 0
.../{ => tcg}/insn_trans/trans_atomic.c.inc | 0
.../{ => tcg}/insn_trans/trans_bit.c.inc | 0
.../{ => tcg}/insn_trans/trans_branch.c.inc | 0
| 0
.../{ => tcg}/insn_trans/trans_farith.c.inc | 0
.../{ => tcg}/insn_trans/trans_fcmp.c.inc | 0
.../{ => tcg}/insn_trans/trans_fcnv.c.inc | 0
.../{ => tcg}/insn_trans/trans_fmemory.c.inc | 0
.../{ => tcg}/insn_trans/trans_fmov.c.inc | 0
.../{ => tcg}/insn_trans/trans_memory.c.inc | 0
.../{ => tcg}/insn_trans/trans_privileged.c.inc | 0
.../{ => tcg}/insn_trans/trans_shift.c.inc | 0
.../{ => tcg}/insn_trans/trans_vec.c.inc | 0
target/loongarch/meson.build | 17 ++---------------
target/loongarch/tcg/meson.build | 15 +++++++++++++++
24 files changed, 17 insertions(+), 15 deletions(-)
rename target/loongarch/{ => tcg}/constant_timer.c (100%)
rename target/loongarch/{ => tcg}/csr_helper.c (100%)
rename target/loongarch/{ => tcg}/fpu_helper.c (100%)
rename target/loongarch/{ => tcg}/iocsr_helper.c (100%)
rename target/loongarch/{ => tcg}/op_helper.c (100%)
rename target/loongarch/{ => tcg}/tlb_helper.c (100%)
rename target/loongarch/{ => tcg}/translate.c (100%)
rename target/loongarch/{ => tcg}/vec_helper.c (100%)
rename target/loongarch/{ => tcg}/insn_trans/trans_arith.c.inc (100%)
rename target/loongarch/{ => tcg}/insn_trans/trans_atomic.c.inc (100%)
rename target/loongarch/{ => tcg}/insn_trans/trans_bit.c.inc (100%)
rename target/loongarch/{ => tcg}/insn_trans/trans_branch.c.inc (100%)
rename target/loongarch/{ => tcg}/insn_trans/trans_extra.c.inc (100%)
rename target/loongarch/{ => tcg}/insn_trans/trans_farith.c.inc (100%)
rename target/loongarch/{ => tcg}/insn_trans/trans_fcmp.c.inc (100%)
rename target/loongarch/{ => tcg}/insn_trans/trans_fcnv.c.inc (100%)
rename target/loongarch/{ => tcg}/insn_trans/trans_fmemory.c.inc (100%)
rename target/loongarch/{ => tcg}/insn_trans/trans_fmov.c.inc (100%)
rename target/loongarch/{ => tcg}/insn_trans/trans_memory.c.inc (100%)
rename target/loongarch/{ => tcg}/insn_trans/trans_privileged.c.inc (100%)
rename target/loongarch/{ => tcg}/insn_trans/trans_shift.c.inc (100%)
rename target/loongarch/{ => tcg}/insn_trans/trans_vec.c.inc (100%)
create mode 100644 target/loongarch/tcg/meson.build
diff --git a/target/loongarch/constant_timer.c b/target/loongarch/tcg/constant_timer.c
similarity index 100%
rename from target/loongarch/constant_timer.c
rename to target/loongarch/tcg/constant_timer.c
diff --git a/target/loongarch/csr_helper.c b/target/loongarch/tcg/csr_helper.c
similarity index 100%
rename from target/loongarch/csr_helper.c
rename to target/loongarch/tcg/csr_helper.c
diff --git a/target/loongarch/fpu_helper.c b/target/loongarch/tcg/fpu_helper.c
similarity index 100%
rename from target/loongarch/fpu_helper.c
rename to target/loongarch/tcg/fpu_helper.c
diff --git a/target/loongarch/iocsr_helper.c b/target/loongarch/tcg/iocsr_helper.c
similarity index 100%
rename from target/loongarch/iocsr_helper.c
rename to target/loongarch/tcg/iocsr_helper.c
diff --git a/target/loongarch/op_helper.c b/target/loongarch/tcg/op_helper.c
similarity index 100%
rename from target/loongarch/op_helper.c
rename to target/loongarch/tcg/op_helper.c
diff --git a/target/loongarch/tlb_helper.c b/target/loongarch/tcg/tlb_helper.c
similarity index 100%
rename from target/loongarch/tlb_helper.c
rename to target/loongarch/tcg/tlb_helper.c
diff --git a/target/loongarch/translate.c b/target/loongarch/tcg/translate.c
similarity index 100%
rename from target/loongarch/translate.c
rename to target/loongarch/tcg/translate.c
diff --git a/target/loongarch/vec_helper.c b/target/loongarch/tcg/vec_helper.c
similarity index 100%
rename from target/loongarch/vec_helper.c
rename to target/loongarch/tcg/vec_helper.c
diff --git a/target/loongarch/insn_trans/trans_arith.c.inc b/target/loongarch/tcg/insn_trans/trans_arith.c.inc
similarity index 100%
rename from target/loongarch/insn_trans/trans_arith.c.inc
rename to target/loongarch/tcg/insn_trans/trans_arith.c.inc
diff --git a/target/loongarch/insn_trans/trans_atomic.c.inc b/target/loongarch/tcg/insn_trans/trans_atomic.c.inc
similarity index 100%
rename from target/loongarch/insn_trans/trans_atomic.c.inc
rename to target/loongarch/tcg/insn_trans/trans_atomic.c.inc
diff --git a/target/loongarch/insn_trans/trans_bit.c.inc b/target/loongarch/tcg/insn_trans/trans_bit.c.inc
similarity index 100%
rename from target/loongarch/insn_trans/trans_bit.c.inc
rename to target/loongarch/tcg/insn_trans/trans_bit.c.inc
diff --git a/target/loongarch/insn_trans/trans_branch.c.inc b/target/loongarch/tcg/insn_trans/trans_branch.c.inc
similarity index 100%
rename from target/loongarch/insn_trans/trans_branch.c.inc
rename to target/loongarch/tcg/insn_trans/trans_branch.c.inc
diff --git a/target/loongarch/insn_trans/trans_extra.c.inc b/target/loongarch/tcg/insn_trans/trans_extra.c.inc
similarity index 100%
rename from target/loongarch/insn_trans/trans_extra.c.inc
rename to target/loongarch/tcg/insn_trans/trans_extra.c.inc
diff --git a/target/loongarch/insn_trans/trans_farith.c.inc b/target/loongarch/tcg/insn_trans/trans_farith.c.inc
similarity index 100%
rename from target/loongarch/insn_trans/trans_farith.c.inc
rename to target/loongarch/tcg/insn_trans/trans_farith.c.inc
diff --git a/target/loongarch/insn_trans/trans_fcmp.c.inc b/target/loongarch/tcg/insn_trans/trans_fcmp.c.inc
similarity index 100%
rename from target/loongarch/insn_trans/trans_fcmp.c.inc
rename to target/loongarch/tcg/insn_trans/trans_fcmp.c.inc
diff --git a/target/loongarch/insn_trans/trans_fcnv.c.inc b/target/loongarch/tcg/insn_trans/trans_fcnv.c.inc
similarity index 100%
rename from target/loongarch/insn_trans/trans_fcnv.c.inc
rename to target/loongarch/tcg/insn_trans/trans_fcnv.c.inc
diff --git a/target/loongarch/insn_trans/trans_fmemory.c.inc b/target/loongarch/tcg/insn_trans/trans_fmemory.c.inc
similarity index 100%
rename from target/loongarch/insn_trans/trans_fmemory.c.inc
rename to target/loongarch/tcg/insn_trans/trans_fmemory.c.inc
diff --git a/target/loongarch/insn_trans/trans_fmov.c.inc b/target/loongarch/tcg/insn_trans/trans_fmov.c.inc
similarity index 100%
rename from target/loongarch/insn_trans/trans_fmov.c.inc
rename to target/loongarch/tcg/insn_trans/trans_fmov.c.inc
diff --git a/target/loongarch/insn_trans/trans_memory.c.inc b/target/loongarch/tcg/insn_trans/trans_memory.c.inc
similarity index 100%
rename from target/loongarch/insn_trans/trans_memory.c.inc
rename to target/loongarch/tcg/insn_trans/trans_memory.c.inc
diff --git a/target/loongarch/insn_trans/trans_privileged.c.inc b/target/loongarch/tcg/insn_trans/trans_privileged.c.inc
similarity index 100%
rename from target/loongarch/insn_trans/trans_privileged.c.inc
rename to target/loongarch/tcg/insn_trans/trans_privileged.c.inc
diff --git a/target/loongarch/insn_trans/trans_shift.c.inc b/target/loongarch/tcg/insn_trans/trans_shift.c.inc
similarity index 100%
rename from target/loongarch/insn_trans/trans_shift.c.inc
rename to target/loongarch/tcg/insn_trans/trans_shift.c.inc
diff --git a/target/loongarch/insn_trans/trans_vec.c.inc b/target/loongarch/tcg/insn_trans/trans_vec.c.inc
similarity index 100%
rename from target/loongarch/insn_trans/trans_vec.c.inc
rename to target/loongarch/tcg/insn_trans/trans_vec.c.inc
diff --git a/target/loongarch/meson.build b/target/loongarch/meson.build
index 18e8191e2b..a004523439 100644
--- a/target/loongarch/meson.build
+++ b/target/loongarch/meson.build
@@ -3,31 +3,18 @@ gen = decodetree.process('insns.decode')
loongarch_ss = ss.source_set()
loongarch_ss.add(files(
'cpu.c',
+ 'gdbstub.c'
))
-loongarch_tcg_ss = ss.source_set()
-loongarch_tcg_ss.add(gen)
-loongarch_tcg_ss.add(files(
- 'fpu_helper.c',
- 'op_helper.c',
- 'translate.c',
- 'gdbstub.c',
- 'vec_helper.c',
-))
-loongarch_tcg_ss.add(zlib)
loongarch_system_ss = ss.source_set()
loongarch_system_ss.add(files(
'loongarch-qmp-cmds.c',
'machine.c',
- 'tlb_helper.c',
- 'constant_timer.c',
- 'csr_helper.c',
- 'iocsr_helper.c',
))
common_ss.add(when: 'CONFIG_LOONGARCH_DIS', if_true: [files('disas.c'), gen])
-loongarch_ss.add_all(when: 'CONFIG_TCG', if_true: [loongarch_tcg_ss])
+subdir('tcg')
target_arch += {'loongarch': loongarch_ss}
target_system_arch += {'loongarch': loongarch_system_ss}
diff --git a/target/loongarch/tcg/meson.build b/target/loongarch/tcg/meson.build
new file mode 100644
index 0000000000..bb7411e5e5
--- /dev/null
+++ b/target/loongarch/tcg/meson.build
@@ -0,0 +1,15 @@
+loongarch_ss.add([zlib, gen])
+
+loongarch_ss.add(files(
+ 'fpu_helper.c',
+ 'op_helper.c',
+ 'vec_helper.c',
+ 'translate.c',
+))
+
+loongarch_system_ss.add(files(
+ 'constant_timer.c',
+ 'iocsr_helper.c',
+ 'tlb_helper.c',
+ 'csr_helper.c',
+))
--
2.25.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 1/1] target/loongarch: move translate modules to tcg/
2023-12-29 9:24 [PATCH 1/1] target/loongarch: move translate modules to tcg/ Song Gao
@ 2023-12-29 10:08 ` Philippe Mathieu-Daudé
2024-01-02 1:25 ` gaosong
0 siblings, 1 reply; 3+ messages in thread
From: Philippe Mathieu-Daudé @ 2023-12-29 10:08 UTC (permalink / raw)
To: Song Gao, qemu-devel
Cc: richard.henderson, peter.maydell, alex.bennee, maobibo,
zhaotianrui
Hi,
On 29/12/23 10:24, Song Gao wrote:
> Introduce the target/loongarch/tcg directory. Its purpose is to hold the TCG
> code that is selected by CONFIG_TCG
>
> Signed-off-by: Song Gao <gaosong@loongson.cn>
> ---
> target/loongarch/{ => tcg}/constant_timer.c | 0
> target/loongarch/{ => tcg}/csr_helper.c | 0
> target/loongarch/{ => tcg}/fpu_helper.c | 0
> target/loongarch/{ => tcg}/iocsr_helper.c | 0
> target/loongarch/{ => tcg}/op_helper.c | 0
> target/loongarch/{ => tcg}/tlb_helper.c | 0
> target/loongarch/{ => tcg}/translate.c | 0
> target/loongarch/{ => tcg}/vec_helper.c | 0
> .../{ => tcg}/insn_trans/trans_arith.c.inc | 0
> .../{ => tcg}/insn_trans/trans_atomic.c.inc | 0
> .../{ => tcg}/insn_trans/trans_bit.c.inc | 0
> .../{ => tcg}/insn_trans/trans_branch.c.inc | 0
> .../{ => tcg}/insn_trans/trans_extra.c.inc | 0
> .../{ => tcg}/insn_trans/trans_farith.c.inc | 0
> .../{ => tcg}/insn_trans/trans_fcmp.c.inc | 0
> .../{ => tcg}/insn_trans/trans_fcnv.c.inc | 0
> .../{ => tcg}/insn_trans/trans_fmemory.c.inc | 0
> .../{ => tcg}/insn_trans/trans_fmov.c.inc | 0
> .../{ => tcg}/insn_trans/trans_memory.c.inc | 0
> .../{ => tcg}/insn_trans/trans_privileged.c.inc | 0
> .../{ => tcg}/insn_trans/trans_shift.c.inc | 0
> .../{ => tcg}/insn_trans/trans_vec.c.inc | 0
> target/loongarch/meson.build | 17 ++---------------
> target/loongarch/tcg/meson.build | 15 +++++++++++++++
> 24 files changed, 17 insertions(+), 15 deletions(-)
> rename target/loongarch/{ => tcg}/constant_timer.c (100%)
> rename target/loongarch/{ => tcg}/csr_helper.c (100%)
> rename target/loongarch/{ => tcg}/fpu_helper.c (100%)
> rename target/loongarch/{ => tcg}/iocsr_helper.c (100%)
> rename target/loongarch/{ => tcg}/op_helper.c (100%)
> rename target/loongarch/{ => tcg}/tlb_helper.c (100%)
> rename target/loongarch/{ => tcg}/translate.c (100%)
> rename target/loongarch/{ => tcg}/vec_helper.c (100%)
> rename target/loongarch/{ => tcg}/insn_trans/trans_arith.c.inc (100%)
> rename target/loongarch/{ => tcg}/insn_trans/trans_atomic.c.inc (100%)
> rename target/loongarch/{ => tcg}/insn_trans/trans_bit.c.inc (100%)
> rename target/loongarch/{ => tcg}/insn_trans/trans_branch.c.inc (100%)
> rename target/loongarch/{ => tcg}/insn_trans/trans_extra.c.inc (100%)
> rename target/loongarch/{ => tcg}/insn_trans/trans_farith.c.inc (100%)
> rename target/loongarch/{ => tcg}/insn_trans/trans_fcmp.c.inc (100%)
> rename target/loongarch/{ => tcg}/insn_trans/trans_fcnv.c.inc (100%)
> rename target/loongarch/{ => tcg}/insn_trans/trans_fmemory.c.inc (100%)
> rename target/loongarch/{ => tcg}/insn_trans/trans_fmov.c.inc (100%)
> rename target/loongarch/{ => tcg}/insn_trans/trans_memory.c.inc (100%)
> rename target/loongarch/{ => tcg}/insn_trans/trans_privileged.c.inc (100%)
> rename target/loongarch/{ => tcg}/insn_trans/trans_shift.c.inc (100%)
> rename target/loongarch/{ => tcg}/insn_trans/trans_vec.c.inc (100%)
> create mode 100644 target/loongarch/tcg/meson.build
> diff --git a/target/loongarch/meson.build b/target/loongarch/meson.build
> index 18e8191e2b..a004523439 100644
> --- a/target/loongarch/meson.build
> +++ b/target/loongarch/meson.build
> @@ -3,31 +3,18 @@ gen = decodetree.process('insns.decode')
> loongarch_ss = ss.source_set()
> loongarch_ss.add(files(
> 'cpu.c',
> + 'gdbstub.c'
Preferably a preliminary commit "gdbstub.c is not specific to TCG and
can be used by other accelerators ...". Otherwise just mention it in
this patch description.
> ))
> -loongarch_tcg_ss = ss.source_set()
> -loongarch_tcg_ss.add(gen)
> -loongarch_tcg_ss.add(files(
> - 'fpu_helper.c',
> - 'op_helper.c',
> - 'translate.c',
> - 'gdbstub.c',
> - 'vec_helper.c',
> -))
> -loongarch_tcg_ss.add(zlib)
>
> loongarch_system_ss = ss.source_set()
> loongarch_system_ss.add(files(
> 'loongarch-qmp-cmds.c',
> 'machine.c',
> - 'tlb_helper.c',
> - 'constant_timer.c',
> - 'csr_helper.c',
> - 'iocsr_helper.c',
> ))
>
> common_ss.add(when: 'CONFIG_LOONGARCH_DIS', if_true: [files('disas.c'), gen])
>
> -loongarch_ss.add_all(when: 'CONFIG_TCG', if_true: [loongarch_tcg_ss])
> +subdir('tcg')
>
> target_arch += {'loongarch': loongarch_ss}
> target_system_arch += {'loongarch': loongarch_system_ss}
> diff --git a/target/loongarch/tcg/meson.build b/target/loongarch/tcg/meson.build
> new file mode 100644
> index 0000000000..bb7411e5e5
> --- /dev/null
> +++ b/target/loongarch/tcg/meson.build
> @@ -0,0 +1,15 @@
You missed the CONFIG_TCG check, you can use either:
if 'CONFIG_TCG' in config_all
subdir('tcg')
endif
in target/loongarch/meson.build, but since your target seems well
designed and doesn't require TCG stub, you can do directly here:
if 'CONFIG_TCG' not in config_all
subdir_done()
endif
so the rest of this file isn't processed.
> +loongarch_ss.add([zlib, gen])
> +
> +loongarch_ss.add(files(
> + 'fpu_helper.c',
> + 'op_helper.c',
> + 'vec_helper.c',
> + 'translate.c',
(since moved/new, let's sort these files alphabetically).
> +))
> +
> +loongarch_system_ss.add(files(
> + 'constant_timer.c',
> + 'iocsr_helper.c',
> + 'tlb_helper.c',
> + 'csr_helper.c',
(Ditto)
> +))
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 1/1] target/loongarch: move translate modules to tcg/
2023-12-29 10:08 ` Philippe Mathieu-Daudé
@ 2024-01-02 1:25 ` gaosong
0 siblings, 0 replies; 3+ messages in thread
From: gaosong @ 2024-01-02 1:25 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, qemu-devel
Cc: richard.henderson, peter.maydell, alex.bennee, maobibo,
zhaotianrui
在 2023/12/29 下午6:08, Philippe Mathieu-Daudé 写道:
> Hi,
>
> On 29/12/23 10:24, Song Gao wrote:
>> Introduce the target/loongarch/tcg directory. Its purpose is to hold
>> the TCG
>> code that is selected by CONFIG_TCG
>>
>> Signed-off-by: Song Gao <gaosong@loongson.cn>
>> ---
>> target/loongarch/{ => tcg}/constant_timer.c | 0
>> target/loongarch/{ => tcg}/csr_helper.c | 0
>> target/loongarch/{ => tcg}/fpu_helper.c | 0
>> target/loongarch/{ => tcg}/iocsr_helper.c | 0
>> target/loongarch/{ => tcg}/op_helper.c | 0
>> target/loongarch/{ => tcg}/tlb_helper.c | 0
>> target/loongarch/{ => tcg}/translate.c | 0
>> target/loongarch/{ => tcg}/vec_helper.c | 0
>> .../{ => tcg}/insn_trans/trans_arith.c.inc | 0
>> .../{ => tcg}/insn_trans/trans_atomic.c.inc | 0
>> .../{ => tcg}/insn_trans/trans_bit.c.inc | 0
>> .../{ => tcg}/insn_trans/trans_branch.c.inc | 0
>> .../{ => tcg}/insn_trans/trans_extra.c.inc | 0
>> .../{ => tcg}/insn_trans/trans_farith.c.inc | 0
>> .../{ => tcg}/insn_trans/trans_fcmp.c.inc | 0
>> .../{ => tcg}/insn_trans/trans_fcnv.c.inc | 0
>> .../{ => tcg}/insn_trans/trans_fmemory.c.inc | 0
>> .../{ => tcg}/insn_trans/trans_fmov.c.inc | 0
>> .../{ => tcg}/insn_trans/trans_memory.c.inc | 0
>> .../{ => tcg}/insn_trans/trans_privileged.c.inc | 0
>> .../{ => tcg}/insn_trans/trans_shift.c.inc | 0
>> .../{ => tcg}/insn_trans/trans_vec.c.inc | 0
>> target/loongarch/meson.build | 17 ++---------------
>> target/loongarch/tcg/meson.build | 15 +++++++++++++++
>> 24 files changed, 17 insertions(+), 15 deletions(-)
>> rename target/loongarch/{ => tcg}/constant_timer.c (100%)
>> rename target/loongarch/{ => tcg}/csr_helper.c (100%)
>> rename target/loongarch/{ => tcg}/fpu_helper.c (100%)
>> rename target/loongarch/{ => tcg}/iocsr_helper.c (100%)
>> rename target/loongarch/{ => tcg}/op_helper.c (100%)
>> rename target/loongarch/{ => tcg}/tlb_helper.c (100%)
>> rename target/loongarch/{ => tcg}/translate.c (100%)
>> rename target/loongarch/{ => tcg}/vec_helper.c (100%)
>> rename target/loongarch/{ => tcg}/insn_trans/trans_arith.c.inc (100%)
>> rename target/loongarch/{ => tcg}/insn_trans/trans_atomic.c.inc (100%)
>> rename target/loongarch/{ => tcg}/insn_trans/trans_bit.c.inc (100%)
>> rename target/loongarch/{ => tcg}/insn_trans/trans_branch.c.inc (100%)
>> rename target/loongarch/{ => tcg}/insn_trans/trans_extra.c.inc (100%)
>> rename target/loongarch/{ => tcg}/insn_trans/trans_farith.c.inc (100%)
>> rename target/loongarch/{ => tcg}/insn_trans/trans_fcmp.c.inc (100%)
>> rename target/loongarch/{ => tcg}/insn_trans/trans_fcnv.c.inc (100%)
>> rename target/loongarch/{ => tcg}/insn_trans/trans_fmemory.c.inc
>> (100%)
>> rename target/loongarch/{ => tcg}/insn_trans/trans_fmov.c.inc (100%)
>> rename target/loongarch/{ => tcg}/insn_trans/trans_memory.c.inc (100%)
>> rename target/loongarch/{ => tcg}/insn_trans/trans_privileged.c.inc
>> (100%)
>> rename target/loongarch/{ => tcg}/insn_trans/trans_shift.c.inc (100%)
>> rename target/loongarch/{ => tcg}/insn_trans/trans_vec.c.inc (100%)
>> create mode 100644 target/loongarch/tcg/meson.build
>
>
>> diff --git a/target/loongarch/meson.build b/target/loongarch/meson.build
>> index 18e8191e2b..a004523439 100644
>> --- a/target/loongarch/meson.build
>> +++ b/target/loongarch/meson.build
>> @@ -3,31 +3,18 @@ gen = decodetree.process('insns.decode')
>> loongarch_ss = ss.source_set()
>> loongarch_ss.add(files(
>> 'cpu.c',
>> + 'gdbstub.c'
>
> Preferably a preliminary commit "gdbstub.c is not specific to TCG and
> can be used by other accelerators ...". Otherwise just mention it in
> this patch description.
>
I will split it to a new patch.
>> ))
>> -loongarch_tcg_ss = ss.source_set()
>> -loongarch_tcg_ss.add(gen)
>> -loongarch_tcg_ss.add(files(
>> - 'fpu_helper.c',
>> - 'op_helper.c',
>> - 'translate.c',
>> - 'gdbstub.c',
>> - 'vec_helper.c',
>> -))
>> -loongarch_tcg_ss.add(zlib)
>> loongarch_system_ss = ss.source_set()
>> loongarch_system_ss.add(files(
>> 'loongarch-qmp-cmds.c',
>> 'machine.c',
>> - 'tlb_helper.c',
>> - 'constant_timer.c',
>> - 'csr_helper.c',
>> - 'iocsr_helper.c',
>> ))
>> common_ss.add(when: 'CONFIG_LOONGARCH_DIS', if_true:
>> [files('disas.c'), gen])
>> -loongarch_ss.add_all(when: 'CONFIG_TCG', if_true: [loongarch_tcg_ss])
>> +subdir('tcg')
>> target_arch += {'loongarch': loongarch_ss}
>> target_system_arch += {'loongarch': loongarch_system_ss}
>> diff --git a/target/loongarch/tcg/meson.build
>> b/target/loongarch/tcg/meson.build
>> new file mode 100644
>> index 0000000000..bb7411e5e5
>> --- /dev/null
>> +++ b/target/loongarch/tcg/meson.build
>> @@ -0,0 +1,15 @@
>
> You missed the CONFIG_TCG check, you can use either:
>
> if 'CONFIG_TCG' in config_all
> subdir('tcg')
> endif
>
> in target/loongarch/meson.build, but since your target seems well
> designed and doesn't require TCG stub, you can do directly here:
>
> if 'CONFIG_TCG' not in config_all
> subdir_done()
> endif
>
> so the rest of this file isn't processed.
>
Got it
>> +loongarch_ss.add([zlib, gen])
>> +
>> +loongarch_ss.add(files(
>> + 'fpu_helper.c',
>> + 'op_helper.c',
>> + 'vec_helper.c',
>> + 'translate.c',
>
> (since moved/new, let's sort these files alphabetically).
>
I will corret it .
Thanks.
Song Gao
>> +))
>> +
>> +loongarch_system_ss.add(files(
>> + 'constant_timer.c',
>> + 'iocsr_helper.c',
>> + 'tlb_helper.c',
>> + 'csr_helper.c',
>
> (Ditto)
>
>> +))
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-01-02 1:26 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-12-29 9:24 [PATCH 1/1] target/loongarch: move translate modules to tcg/ Song Gao
2023-12-29 10:08 ` Philippe Mathieu-Daudé
2024-01-02 1:25 ` gaosong
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 NNTP newsgroup(s).