[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",
----------------
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   |



Repository:
  rL LLVM

https://reviews.llvm.org/D44924