Re: [PATCH 0/3] configure: Do not build TCG or link with capstone if not necessary

2021-01-22 Thread Philippe Mathieu-Daudé
On 1/20/21 6:35 PM, Paolo Bonzini wrote: > On 20/01/21 18:02, Philippe Mathieu-Daudé wrote: >>> >>> For patch 1, which files are not compiled with the patch that were >>> compiled without? >> softfloat. > > Really?  I see this: > > specific_ss.add(when: 'CONFIG_TCG', if_true: files( >  

Re: [PATCH 0/3] configure: Do not build TCG or link with capstone if not necessary

2021-01-20 Thread Paolo Bonzini
On 20/01/21 18:02, Philippe Mathieu-Daudé wrote: For patch 1, which files are not compiled with the patch that were compiled without? softfloat. Really? I see this: specific_ss.add(when: 'CONFIG_TCG', if_true: files( 'fpu/softfloat.c', ...)) Maybe -subdir('fp') +if 'CONFIG_TCG' in

Re: [PATCH 0/3] configure: Do not build TCG or link with capstone if not necessary

2021-01-20 Thread Philippe Mathieu-Daudé
On 1/20/21 5:46 PM, Paolo Bonzini wrote: > On 20/01/21 16:19, Philippe Mathieu-Daudé wrote: >> We do not need TCG and capstone all the times. In some >> configuration we can leave them out. >> >> Last patch emit a warning when a user explicitly select an >> accelerator that the build with not use.

Re: [PATCH 0/3] configure: Do not build TCG or link with capstone if not necessary

2021-01-20 Thread Paolo Bonzini
On 20/01/21 16:19, Philippe Mathieu-Daudé wrote: We do not need TCG and capstone all the times. In some configuration we can leave them out. Last patch emit a warning when a user explicitly select an accelerator that the build with not use. Philippe Mathieu-Daudé (3): configure: Do not

[PATCH 0/3] configure: Do not build TCG or link with capstone if not necessary

2021-01-20 Thread Philippe Mathieu-Daudé
We do not need TCG and capstone all the times. In some configuration we can leave them out. Last patch emit a warning when a user explicitly select an accelerator that the build with not use. Philippe Mathieu-Daudé (3): configure: Do not build TCG if not necessary configure: Do not