[PATCH] D45380: [X86] Add SchedWrites for CMOV and SETCC. Use them to remove InstRWs.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 7 06:26:49 PDT 2018


RKSimon added inline comments.


================
Comment at: lib/Target/X86/X86ScheduleBtVer2.td:145
+defm : JWriteResIntPair<WriteCMOV,  [JALU01], 1>; // Conditional move.
+def  : WriteRes<WriteSETCC, [JALU01]>; // Setcc.
+
----------------
craig.topper wrote:
> RKSimon wrote:
> > Move the WriteSETCCStore def into the models so we don't get the regression on btver2 tests?
> You mean make it a regular SchedWrite and not a WriteSequence?
If you can, otherwise please can you add a InstRW like znver1 has to btver2?


Repository:
  rL LLVM

https://reviews.llvm.org/D45380





More information about the llvm-commits mailing list