On 1/29/21 1:16 PM, Peter Maydell wrote:
> On Fri, 29 Jan 2021 at 20:13, Richard Henderson
> <richard.hender...@linaro.org> wrote:
>>
>> The opcodes always exist, regardless of whether or not they
>> are enabled. Remove the unnecessary ifdefs.
>>
>> Signed-off-by: Richard Henderson <richard.hender...@linaro.org>
>> ---
>> tcg/tci/tcg-target.c.inc | 82 ----------------------------------------
>> 1 file changed, 82 deletions(-)
>>
>> diff --git a/tcg/tci/tcg-target.c.inc b/tcg/tci/tcg-target.c.inc
>> index 9c45f5f88f..b62e14d5ce 100644
>> --- a/tcg/tci/tcg-target.c.inc
>> +++ b/tcg/tci/tcg-target.c.inc
>> @@ -71,70 +71,42 @@ static const TCGTargetOpDef tcg_target_op_defs[] = {
>> { INDEX_op_add_i32, { R, RI, RI } },
>> { INDEX_op_sub_i32, { R, RI, RI } },
>> { INDEX_op_mul_i32, { R, RI, RI } },
>> -#if TCG_TARGET_HAS_div_i32
>> { INDEX_op_div_i32, { R, R, R } },
>> { INDEX_op_divu_i32, { R, R, R } },
>> { INDEX_op_rem_i32, { R, R, R } },
>> { INDEX_op_remu_i32, { R, R, R } },
>> -#elif TCG_TARGET_HAS_div2_i32
>> - { INDEX_op_div2_i32, { R, R, "0", "1", R } },
>> - { INDEX_op_divu2_i32, { R, R, "0", "1", R } },
>> -#endif
>
>> -#if TCG_TARGET_HAS_div_i64
>> { INDEX_op_div_i64, { R, R, R } },
>> { INDEX_op_divu_i64, { R, R, R } },
>> { INDEX_op_rem_i64, { R, R, R } },
>> { INDEX_op_remu_i64, { R, R, R } },
>> -#elif TCG_TARGET_HAS_div2_i64
>> - { INDEX_op_div2_i64, { R, R, "0", "1", R } },
>> - { INDEX_op_divu2_i64, { R, R, "0", "1", R } },
>> -#endif
>
> Why are div2/divu2 special cases such that their entries
> get deleted rather than unconditionally included ?
Because div/div2 are mutually exclusive.
r~
