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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits