[PATCH] D92837: [X86] Support tilezero intrinsic and c interface for AMX.

[Pengfei Wang via Phabricator via llvm-commits]

pengfei added inline comments.


================
Comment at: llvm/lib/Target/X86/X86ExpandPseudo.cpp:499
+    MI.RemoveOperand(3); // Remove $tmmcfg
+    for (int i = 2; i > 0; --i)
+      MI.RemoveOperand(i);
----------------
Can change to i = 3?


================
Comment at: llvm/lib/Target/X86/X86ISelDAGToDAG.cpp:4624
+      SDValue Chain = Node->getOperand(0);
+      SDValue Ops[] = {Node->getOperand(2), Node->getOperand(3), CFG, Chain};
+      MachineSDNode *CNode =
----------------
Why tilezero needs chain?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92837