[PATCH] D24022: Update the vectorizer cost model for getVectorInstrCost to reflect actual costs of the operations on Power8/Power9
Nemanja Ivanovic via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 30 07:53:20 PDT 2016
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.
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.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 10543 bytes
Desc: not available
More information about the llvm-commits