On 3/24/21 11:03 PM, Richard Henderson wrote:
> On 3/23/21 9:46 AM, Claudio Fontana wrote:
>> it is required by arch-dump.c and cpu.c, so apparently
>> we need this for KVM too
>>
>> Signed-off-by: Claudio Fontana <cfont...@suse.de>
>
>
> Reviewed-by: Richard Henderson <richard.hender...@linaro.org>
>
>
>> +/*
>> + * these are AARCH64-only, but due to the chain of dependencies,
>> + * between HELPER prototypes, hflags, cpreg definitions and functions in
>> + * tcg/ etc, it becomes incredibly messy to add what should be here:
>> + *
>> + * #ifdef TARGET_AARCH64
>> + */
>> +
>> +static uint32_t sve_zcr_get_valid_len(ARMCPU *cpu, uint32_t start_len)
>> +{
>> + uint32_t end_len;
>> +
>> + end_len = start_len &= 0xf;
>> + if (!test_bit(start_len, cpu->sve_vq_map)) {
>> + end_len = find_last_bit(cpu->sve_vq_map, start_len);
>> + assert(end_len < start_len);
>> + }
>> + return end_len;
>> +}
>
> I guess you could
>
> #ifdef TARGET_AARCH64
> ...
> #else
> g_assert_not_reached();
> #endif
>
> Dunno if it's worth it or not, since they're small.
>
>
> r~
>
I'll check this again. IIRC I _did_ handle the HELPER() / hflags / cpreg
problem in the end,
so it should be possible to make this actually right (ie #ifdef TARGET_AARCH64)
