[PATCH] D126519: [X86][AMX] Replace LDTILECFG with PLDTILECFGV on auto-config.

[LuoYuanke via Phabricator via llvm-commits]

LuoYuanke added inline comments.


================
Comment at: llvm/lib/Target/X86/X86InstrInfo.cpp:7363
   if (Opcode == X86::ENDBR64 || Opcode == X86::ENDBR32 ||
-      Opcode == X86::LDTILECFG)
+      Opcode == X86::PLDTILECFGV)
     return true;
----------------
xiangzhangllvm wrote:
> I think the old one should also keep in isSchedulingBoundary
This is to prevent shape def instruction be scheduled after PLDTILECFGV. The old one (LDTILECFG) is only generated from user's intrinsic, there is no shape dependency issue. I prefer to adding LDTILECFG as schedule boudary when it is really necessary.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126519