[PATCH 2/3] ARM cost model: Address computation in vector mem ops not free

[Arnold Schwaighofer]

Updated patch. We now add the cost of address computation as part of the memory instruction cost.


Thanks


On Feb 1, 2013, at 1:39 PM, Renato Golin <renato.golin at linaro.org> wrote:

> On 1 February 2013 18:07, Nadav Rotem <nrotem at apple.com> wrote:
> The problem is that we decide on the kind of GEP to use only when we vectorize the load/stores.  This happens in vectorizeMemoryInstruction. I think that we need to fix this in LoopVectorizationCostModel::getInstructionCost in the load/store switch cases. We have code for checking if the load/store is wide or if it is scalarized.
> 
> Good point! Shouldn't be too hard, though.
> 
> cheers,
> --renato

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130205/9e942a63/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-ARM-cost-model-Address-computation-in-vector-mem-ops.patch
Type: application/octet-stream
Size: 15454 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130205/9e942a63/attachment.obj>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130205/9e942a63/attachment-0001.html>