[PATCH] D93173: [X86] Add test case for commit e52bc1d2bba794b.

LuoYuanke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 14 04:12:22 PST 2020


LuoYuanke added inline comments.


================
Comment at: llvm/test/CodeGen/X86/AMX/amx-intrinsic-chain.ll:12
+; CHECK-NEXT:    movb $1, -{{[0-9]+}}(%rsp)
+; CHECK-NEXT:    movb $16, -{{[0-9]+}}(%rsp)
+; CHECK-NEXT:    movw $64, -{{[0-9]+}}(%rsp)
----------------
LuoYuanke wrote:
> pengfei wrote:
> > Does the adding chain that generates more `__tile1024i` initialization here?
> > But why do we need to generate more initialization for the same row and col?
> Adding chain doesn't generate more `__tile1024i` initialization. There 5 movb for row, 5 movw for column and `movb $1, ...` for palette. There are 5 tmm register allocated, so the initialization is correct.
One of the tilestore is deleted during instruction selection if we don't add chain.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93173



More information about the llvm-commits mailing list