[PATCH] D137923: [X86][AMX] Fix the shape dependency issue.

LuoYuanke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 13 22:02:43 PST 2022


LuoYuanke added a comment.

In D137923#3924006 <https://reviews.llvm.org/D137923#3924006>, @xiangzhangllvm wrote:

> Maybe it is time to re-review the previous way for implement "let share define before use". 
> e.g if let frond end do same "make sure" is more beautiful.

Yes, if it can be prevented in FE, that's great. I try to use memory barriar as below code, but it doesn't work. Welcome for ideas.

  static __inline__ void __tile_dpbsud(__tile1024i *dst, __tile1024i src0,
                                       __tile1024i src1) {
    short m = src0.row;
    short n = src1.col;
    short k = src0.col;
    asm volatile ("" : : : "memory");
    dst->tile = _tile_dpbsud_internal(m, n, k, dst->tile,
                                      src0.tile, src1.tile);
  }


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137923/new/

https://reviews.llvm.org/D137923



More information about the llvm-commits mailing list