[PATCH] D18000: [x86] fix cost model inaccuracy for vector memory ops

[Sanjay Patel via llvm-commits]

spatel added a comment.

In http://reviews.llvm.org/D18000#371254, @DavidKreitzer wrote:

> This seems reasonable to me also. It might be nice to have a separate X86Subtarget property for double pumped 32-byte load/stores, but that may be overkill for this one use.


Thanks, Zia and Dave. I agree a separate property would be good if we really want to model this better. For reference, I noticed this bug as part of the TTI cost model discussion in PR26837:
https://llvm.org/bugs/show_bug.cgi?id=26837

There's a lot of bigger stuff that could be modeled better. :)
Ideally, I think we would lift latency/throughput data from the SchedMachineModel, but I'm not sure how to do that yet.


Repository:
  rL LLVM

http://reviews.llvm.org/D18000