[PATCH] D63236: [mips] Add empty scheduler info and `NoItinerary` to the `MipsAsmPseudoInst`

Simon Atanasyan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 12 17:03:35 PDT 2019


atanasyan created this revision.
atanasyan added reviewers: dsanders, sdardis, Petar.Avramovic.
Herald added subscribers: jrtc27, hiraditya, arichardson.
Herald added a project: LLVM.

Add empty scheduler info and `NoItinerary` to the`MipsAsmPseudoInst`. These pseudo-instructions are never used by codegen. `NoItinerary` and empty scheduler info allow to reduce number of "No schedule information for" and "lacks information for" errors in case of marking a scheduler model as complete.

This patch is one of a series of patches. The goal is to make P5600 <https://reviews.llvm.org/P5600> scheduler model complete and turn on the `CompleteModel` flag.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D63236

Files:
  llvm/lib/Target/Mips/MipsInstrFormats.td


Index: llvm/lib/Target/Mips/MipsInstrFormats.td
===================================================================
--- llvm/lib/Target/Mips/MipsInstrFormats.td
+++ llvm/lib/Target/Mips/MipsInstrFormats.td
@@ -143,7 +143,7 @@
 // These are aliases that require C++ handling to convert to the target
 // instruction, while InstAliases can be handled directly by tblgen.
 class MipsAsmPseudoInst<dag outs, dag ins, string asmstr>:
-  MipsInst<outs, ins, asmstr, [], IIPseudo, Pseudo> {
+  MipsInst<outs, ins, asmstr, [], NoItinerary, Pseudo>, Sched<[]> {
   let isPseudo = 1;
   let Pattern = [];
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63236.204390.patch
Type: text/x-patch
Size: 598 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190613/c6b8ee3d/attachment.bin>


More information about the llvm-commits mailing list