[PATCH] D114066: [X86][AMX] Don't emit tilerelease for old AMX instrisic.

[Xiang Zhang via Phabricator via llvm-commits]

xiangzhangllvm added inline comments.


================
Comment at: llvm/lib/Target/X86/X86FastTileConfig.cpp:293-294
   }
+  if (Changed)
+    X86FI->setHasVirtualTileReg(true);
   return Changed;
----------------
LuoYuanke wrote:
> xiangzhangllvm wrote:
> > Another way to escape add a variable in X86MachineFunctionInfo is
> >   if (Changed)
> >     add tilerelease at MF.end()
> There may be multi exit, so insert tilerelease to end block is not enough.
Make sense, here not merge the multi exits to one exit yet.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114066