[PATCH] D95508: [X86] Fix tile config register spill issue for AMX

Xiang Zhang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 30 01:10:49 PST 2021


xiangzhangllvm added inline comments.


================
Comment at: llvm/lib/Target/X86/X86PreTileConfig.cpp:300
+    if (Collect2)
+      collectCfgCalls(MBB, MI);
+
----------------
pengfei wrote:
> Doesn't it always just return `BB2AMX[TileCfgBB]`?
If call collectCfgCalls 2 times, the first time call will make sure at least 1 loop dependent BB status (can directly reach amx or not), So the 2nd can make sure all  BBs' status.


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

https://reviews.llvm.org/D95508



More information about the llvm-commits mailing list