[PATCH 5/7] target/xtensa: Build 'xtensa-isa.c' in common source set

Philippe Mathieu-Daudé posted 7 patches 1 month ago
Maintainers: Max Filippov <jcmvbkbc@gmail.com>, Laurent Vivier <laurent@vivier.eu>
[PATCH 5/7] target/xtensa: Build 'xtensa-isa.c' in common source set
Posted by Philippe Mathieu-Daudé 1 month ago
'xtensa-isa.c' is now free of target-specific symbols,
build it as common files (thus forbidding further
uses of such target-specific symbols).

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 target/xtensa/meson.build | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/target/xtensa/meson.build b/target/xtensa/meson.build
index 69c6946420a..3fdb20db655 100644
--- a/target/xtensa/meson.build
+++ b/target/xtensa/meson.build
@@ -12,6 +12,10 @@ xtensa_ss.add(files(
   'op_helper.c',
   'translate.c',
   'win_helper.c',
+))
+
+xtensa_common_ss = ss.source_set()
+xtensa_common_ss.add(files(
   'xtensa-isa.c',
 ))
 
@@ -24,4 +28,5 @@ xtensa_system_ss.add(files(
 ))
 
 target_arch += {'xtensa': xtensa_ss}
+target_common_arch += {'xtensa': xtensa_common_ss}
 target_common_system_arch += {'xtensa': xtensa_system_ss}
-- 
2.52.0


Re: [PATCH 5/7] target/xtensa: Build 'xtensa-isa.c' in common source set
Posted by Anton Johansson via qemu development 3 weeks, 6 days ago
On 07/01/26, Philippe Mathieu-Daudé wrote:
> 'xtensa-isa.c' is now free of target-specific symbols,
> build it as common files (thus forbidding further
> uses of such target-specific symbols).
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>  target/xtensa/meson.build | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/target/xtensa/meson.build b/target/xtensa/meson.build
> index 69c6946420a..3fdb20db655 100644
> --- a/target/xtensa/meson.build
> +++ b/target/xtensa/meson.build
> @@ -12,6 +12,10 @@ xtensa_ss.add(files(
>    'op_helper.c',
>    'translate.c',
>    'win_helper.c',
> +))
> +
> +xtensa_common_ss = ss.source_set()
> +xtensa_common_ss.add(files(
>    'xtensa-isa.c',
>  ))
>  
> @@ -24,4 +28,5 @@ xtensa_system_ss.add(files(
>  ))
>  
>  target_arch += {'xtensa': xtensa_ss}
> +target_common_arch += {'xtensa': xtensa_common_ss}
>  target_common_system_arch += {'xtensa': xtensa_system_ss}
> -- 
> 2.52.0
> 

Mentioned this in the 7th patch, but xtensa-isa.h is strange to me,
why not move hw/xtensa/xtensa-isa.h to target/ and drop the current
one-line target/xtensa/xtensa-isa.h file? Also include with "target/".

Otherwise,

Reviewed-by: Anton Johansson <anjo@rev.ng>