On 6/11/20 7:45 AM, Peter Maydell wrote:
> +static bool trans_VTBL(DisasContext *s, arg_VTBL *a)
> +{
> + int n;
> + TCGv_i32 tmp, tmp2, tmp3, tmp4;
> + TCGv_ptr ptr1;
> +
> + if (!arm_dc_feature(s, ARM_FEATURE_NEON)) {
> + return false;
> + }
> +
> + /* UNDEF accesses to D16-D31 if they don't exist. */
> + if (!dc_isar_feature(aa32_simd_r32, s) &&
> + ((a->vd | a->vn | a->vm) & 0x10)) {
> + return false;
> + }
> +
> + if (!vfp_access_check(s)) {
> + return true;
> + }
> +
> + n = a->len + 1;
> + if ((a->vn + n) > 32) {
> + /*
> + * This is UNPREDICTABLE; we choose to UNDEF to avoid the
> + * helper function running off the end of the register file.
> + */
> + return 1;
> + }
Oops, meant to point out: return false.
r~
