[PATCH 3/3] ARM cost model: Cost for scalar integer casts and floating point conversions

Renato Golin renato.golin at linaro.org
Fri Feb 1 08:03:35 PST 2013


Thanks for adding these!

However, the table is called NEONConversionTbl and there's a test below (if
ST->hasNEON), so it'd be better if the non-vector instructions could be
added to a separate table.

Possibly adding an Ty->isVector() && ST->hasNEON() or some combination that
allows for both vectorized and scalar codes to execute correctly on all
possible combinations, but doesn't iterate through the wrong table when the
type is obviously not there.

--renato


On 1 February 2013 15:26, Arnold Schwaighofer <aschwaighofer at apple.com>wrote:

> Add costs for scalar integer casts and  floating point conversions, also
> adds some costs for vector integer float conversions.
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130201/18a5f484/attachment.html>


More information about the llvm-commits mailing list