[PATCH] D24022: Update the vectorizer cost model for getVectorInstrCost to reflect actual costs of the operations on Power8/Power9

[Nemanja Ivanovic via llvm-commits]

nemanjai updated this revision to Diff 69681.
nemanjai added a comment.

Updated the very confusing condition for whether the type is an integer of the right width at the right index.

Please note:
This is not necessarily meant to be in a state where it can be approved/committed. This patch implements the costs of these instructions based on latency of the instructions themselves. As such, this patch is meant to start a discussion on exactly how we want the cost model to work and what we want the returned costs to represent.

As things stand now, the relative cost of a direct move and an actual Load-hit-store hazard are not nearly representative of their actual relative costs. But I've tried to ensure that we fall through to the "LHS" code in cases where we will actually have an LHS hazard.


Repository:
  rL LLVM

https://reviews.llvm.org/D24022

Files:
  lib/Target/PowerPC/PPCISelLowering.cpp
  lib/Target/PowerPC/PPCInstrVSX.td
  lib/Target/PowerPC/PPCTargetTransformInfo.cpp
  test/Analysis/CostModel/PowerPC/insert_extract.ll
  test/CodeGen/PowerPC/swaps-le-5.ll
  test/CodeGen/PowerPC/swaps-le-6.ll
  test/CodeGen/PowerPC/vsx_insert_extract_le.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D24022.69681.patch
Type: text/x-patch
Size: 10543 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160830/a6b1c468/attachment.bin>