On Tue, Aug 10, 2021 at 4:11 PM Hongyu Wang via Gcc-patches
<gcc-patches@gcc.gnu.org> wrote:
>
> Hi,
>
> This patch fixes some typo in amxbf16-dpbf16ps-2 test.
>
> Tested under sde/spr machine and passed.
>
> OK for master and backport to GCC 11?
Ok for master, and i don't think the backport is necessary.
>
> gcc/testsuite/ChangeLog:
>
>         * gcc.target/i386/amxbf16-dpbf16ps-2.c: Fix typos.
> ---
>  gcc/testsuite/gcc.target/i386/amxbf16-dpbf16ps-2.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/gcc/testsuite/gcc.target/i386/amxbf16-dpbf16ps-2.c 
> b/gcc/testsuite/gcc.target/i386/amxbf16-dpbf16ps-2.c
> index 349ec58ada2..f7002ca5ea5 100644
> --- a/gcc/testsuite/gcc.target/i386/amxbf16-dpbf16ps-2.c
> +++ b/gcc/testsuite/gcc.target/i386/amxbf16-dpbf16ps-2.c
> @@ -57,7 +57,7 @@ void calc_matrix_dpbf16ps (__tile *dst, __tile *src1, 
> __tile *src2)
>               (make_f32(src1_buf[i * 4 * N + 4 * j + t]) *
>               make_f32(src2_buf[j * 4 * K + 4 * k + t])) +
>               (make_f32(src1_buf[i * 4 * N + 4 * j + t + 1]) *
> -             make_f32(src1_buf[i * 4 * N + 4 * j + t + 1]));
> +             make_f32(src2_buf[j * 4 * K + 4 * k + t + 1]));
>           }
>
>  }
> @@ -72,8 +72,8 @@ void test_amx_bf16_dpbf16ps ()
>
>    init_tile_config (&cfg);
>    init_tile_reg_and_src_with_buffer (1, dst, tmp_dst_buf);
> -  init_tile_reg_and_src_with_buffer (2, dst, tmp_dst_buf);
> -  init_tile_reg_and_src_with_buffer (3, dst, tmp_dst_buf);
> +  init_tile_reg_and_src_with_buffer (2, src1, tmp_dst_buf);
> +  init_tile_reg_and_src_with_buffer (3, src2, tmp_dst_buf);
>
>    calc_matrix_dpbf16ps (&dst, &src1, &src2);
>
> --
> 2.18.1
>


-- 
BR,
Hongtao

Reply via email to