[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