diff mbox series

[v3,6/6] tests/meson: Only build softfloat objects if TCG is selected

Message ID 20210415122633.4054644-7-philmd@redhat.com (mailing list archive)
State New, archived
Headers show
Series qtests: Check accelerator available at runtime via QMP 'query-accels' | expand

Commit Message

Philippe Mathieu-Daudé April 15, 2021, 12:26 p.m. UTC
From: Philippe Mathieu-Daudé <f4bug@amsat.org>

The previous attempt (commit f77147cd4de) doesn't work as
expected, as we still have CONFIG_TCG=1 when using:

  configure --disable-system --disable-user

Now than we have removed the use of CONFIG_TCG from target-dependent
files in tests/qtest/, we can remove the unconditional definition of
CONFIG_TCG in config_host.

This avoid to build a bunch of unrequired objects when building with
--disable-tcg (in particular the softfloat tests):


  $ make
  [1/812] Generating trace-qom.h with a custom command


  $ make
  [1/349] Generating trace-qom.h with a custom command

A difference of 463 objects...

Reported-by: Claudio Fontana <cfontana@suse.de>
Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
v3: Include Paolo's feedback:
therefore o not include Alex's R-b tag.

Cc: Richard Henderson <richard.henderson@linaro.org>
Cc: Alex Bennée <alex.bennee@linaro.org>
Cc: Emilio G. Cota <cota@braap.org>
 meson.build | 1 -
 1 file changed, 1 deletion(-)
diff mbox series


diff --git a/meson.build b/meson.build
index c6f4b0cf5e8..623cbe50685 100644
--- a/meson.build
+++ b/meson.build
@@ -262,7 +262,6 @@ 
                         language: ['c', 'cpp', 'objc'])
   accelerators += 'CONFIG_TCG'
-  config_host += { 'CONFIG_TCG': 'y' }
 if 'CONFIG_KVM' not in accelerators and get_option('kvm').enabled()