Re: [PATCH 4/8] Enable SVE for GDB

Pedro Alves
2018-05-31T14:59:32Z

On 05/11/2018 11:52 AM, Alan Hayward wrote:
> --- a/gdb/aarch64-tdep.c
> +++ b/gdb/aarch64-tdep.c
> @@ -2873,6 +2873,26 @@ aarch64_read_description (long vq)
> return tdesc;
> }
>
> +/* Return the VQ used when creating the target description TDESC. */
> +
> +static long
> +aarch64_get_tdesc_vq (const struct target_desc *tdesc)

Is this use of "long" significant? I mean, is it assuming 64-bit?
I ask because longs are not 64-bit on x64 Windows, so it would
do the wrong thing when cross debugging.

> +{
> + const struct tdesc_feature *feature_sve;
> +
> + if (!tdesc_has_registers (tdesc))
> + return 0;
> +

Thanks,
Pedro Alves