[PATCH] D125075: [X86][AMX] Multiple configure for AMX register.

Xiang Zhang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 9 01:22:28 PDT 2022


xiangzhangllvm added inline comments.


================
Comment at: llvm/lib/Target/X86/X86FastPreTileConfig.cpp:439
+}
+
+bool X86FastPreTileConfig::runOnMachineFunction(MachineFunction &MFunc) {
----------------
Maybe better to add the comments with change steps of a simple case ahead the key functions.
(Some like the old AMX fast allocation do) That will make it easy for readers who not very familiar here code.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125075



More information about the llvm-commits mailing list