[PATCH] D44924: [X86] Add WriteFMOVMSK/WriteVecMOVMSK scheduler classes

[Clement Courbet via Phabricator via llvm-commits]

courbet added inline comments.


================
Comment at: lib/Target/X86/X86SchedBroadwell.td:300
                                            "MMX_MOVD64grr",
-                                           "MMX_PMOVMSKBrr",
                                            "MMX_PSLLDri",
----------------
courbet wrote:
> RKSimon wrote:
> > courbet wrote:
> > > I'm measuring a latency of 1 for MMX_PMOVMSKBrr, removing the override would break this.
> > Should I put the override back for Haswell/SandyBridge?
> Yes, please put it back for HW/BD. As for SNB, it does not have an override for MMX_PMOVMSKBrr right now, I'm going to add one in a separate commit.
> 
> To summarize my measurements:
> (I don't have an SKX at hand, but I expect it to be similar to SKL)
> 
> |                                       | SNB | BDW/HSW | SKL |
> | MMX_PMOVMSKBrr |   1    |            1        |    3   |
> | (V)PMOVMSKBrr        |   2    |            3        |    3   |
> 
> I'm going to add one in a separate commit.

D44933


Repository:
  rL LLVM

https://reviews.llvm.org/D44924