Re: [PATCH] tcg: Unify TCG_TARGET_HAS_extr[lh]_i64_i32

[Richard Henderson <richard.henderson@linaro.org>]

On 8/22/23 15:05, Philippe Mathieu-Daudé wrote:
> On 22/8/23 19:51, Richard Henderson wrote:
>> Replace the separate defines with TCG_TARGET_HAS_extr_i64_i32,
>> so that the two parts of backend-specific type changing cannot
>> be out of sync.
>>
>> Reported-by: Philippe Mathieu-Daudé <philmd@linaro.org>
>> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
>> ---
>>   include/tcg/tcg-opc.h        | 4 ++--
>>   include/tcg/tcg.h            | 3 +--
>>   tcg/aarch64/tcg-target.h     | 3 +--
>>   tcg/i386/tcg-target.h        | 3 +--
>>   tcg/loongarch64/tcg-target.h | 3 +--
>>   tcg/mips/tcg-target.h        | 3 +--
>>   tcg/ppc/tcg-target.h         | 3 +--
>>   tcg/riscv/tcg-target.h       | 3 +--
>>   tcg/s390x/tcg-target.h       | 3 +--
>>   tcg/sparc64/tcg-target.h     | 3 +--
>>   tcg/tci/tcg-target.h         | 3 +--
>>   tcg/tcg-op.c                 | 4 ++--
>>   tcg/tcg.c                    | 3 +--
>>   13 files changed, 15 insertions(+), 26 deletions(-)
> 
> Clever than what I was thinking of, thanks!
> 
> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> 

Queued to tcg-next.


r~