[PATCH] Fix ARM LNT test-suite bot

[Renato Golin]

We spoke on IRC, the attached patch is a much better fix for now.

cheers,
--renato


On 2 August 2013 17:14, Nadav Rotem <nrotem at apple.com> wrote:

> Renato,
>
> Excellent. Thanks for the quick analysis and fix. If Arnold gives a thumb
> up then we can commit this.
>
> Thanks,
> Nadav
>
> On Aug 2, 2013, at 5:22 AM, Renato Golin <renato.golin at linaro.org> wrote:
>
> Nadav,
>
> This patch fixes *ALL* test-suite problems! :D
>
> Description:
>
> This patch fixes the multiple breakages on ARM test-suite after the SLP
> vectorizer was introduced by default on O3. The problem was a missing
> legalization of the vector types on ARMTTI::getCmpSelInstrCost(). We
> might have to review all cost functions for the proper legalization
> of types to avoid problems like these in the future.
>
> The cost of boolean vector comparison changed radically due to this patch,
> which may be better (or not), depending on how the back-end will lower
> these selects. Needs further investigation as well.
>
> Arnold,
>
> If the new costs are totally wrong, we might need further impediments and
> hard-code high costs on getCmpSelInstrCost(). Let me know if it rings a
> bell.
>
> cheers,
> --renato
> <armtt.patch>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130802/448a34e5/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: armtt-2.patch
Type: application/octet-stream
Size: 1899 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130802/448a34e5/attachment.obj>