Re: [PATCH 2/3] Hexagon (tests/tcg/hexagon) Fix alignment in load_unpack.c

[Richard Henderson]

On 7/18/22 16:03, Taylor Simpson wrote:
> The increment used in :brev tests was causing unaligned addresses
> Change the increment and the relevant expected values
>
> Signed-off-by: Taylor Simpson <tsimpson@quicinc.com>
> ---
>   tests/tcg/hexagon/load_unpack.c | 14 +++++++-------
>   1 file changed, 7 insertions(+), 7 deletions(-)

Acked-by: Richard Henderson <richard.henderson@linaro.org>

r~

> diff --git a/tests/tcg/hexagon/load_unpack.c b/tests/tcg/hexagon/load_unpack.c
> index 3575a37a28..4aa26fc388 100644
> --- a/tests/tcg/hexagon/load_unpack.c
> +++ b/tests/tcg/hexagon/load_unpack.c
> @@ -245,7 +245,7 @@ TEST_pr(loadbsw4_pr, long long, S, 4, 0x0000ff000000ff00LL,
>    */
>   #define BxW_LOAD_pbr(SZ, RES, PTR) \
>       __asm__( \
> -        "r4 = #(1 << (16 - 3))\n\t" \
> +        "r4 = #(1 << (16 - 4))\n\t" \
>           "m0 = r4\n\t" \
>           "%0 = mem" #SZ "(%1++m0:brev)\n\t" \
>           : "=r"(RES), "+r"(PTR) \
> @@ -273,15 +273,15 @@ void test_##NAME(void) \
>   }
>   
>   TEST_pbr(loadbzw2_pbr, int, Z, 0x00000000,
> -    0x00020081, 0x00060085, 0x00040083, 0x00080087)
> +    0x00020081, 0x000a0089, 0x00060085, 0x000e008d)
>   TEST_pbr(loadbsw2_pbr, int, S, 0x0000ff00,
> -    0x00020081, 0x00060085, 0x00040083, 0x00080087)
> +    0x00020081, 0x000aff89, 0x0006ff85, 0x000eff8d)
>   TEST_pbr(loadbzw4_pbr, long long, Z, 0x0000000000000000LL,
> -    0x0004008300020081LL, 0x0008008700060085LL,
> -    0x0006008500040083LL, 0x000a008900080087LL)
> +    0x0004008300020081LL, 0x000c008b000a0089LL,
> +    0x0008008700060085LL, 0x0010008f000e008dLL)
>   TEST_pbr(loadbsw4_pbr, long long, S, 0x0000ff000000ff00LL,
> -    0x0004008300020081LL, 0x0008008700060085LL,
> -    0x0006008500040083LL, 0x000a008900080087LL)
> +    0x0004008300020081LL, 0x000cff8b000aff89LL,
> +    0x0008ff870006ff85LL, 0x0010ff8f000eff8dLL)
>   
>   /*
>    ****************************************************************************