[PATCH] Loop Vectorizer: Handle pointer stores/loads in getWidestType()

Renato Golin renato.golin at linaro.org
Mon Feb 4 06:36:21 PST 2013


LGTM!

--renato


On 4 February 2013 14:01, Arnold Schwaighofer <aschwaighofer at apple.com>wrote:

> Hi Nadav,
>
> thanks for reviewing.
>
> Updated patch with fixed test:
>   fixed the -mcpu flag to corei7-avx
>   moved test to LoopVectorize/X86 sub directory
>
> - Arnold
>
>
> On Feb 3, 2013, at 10:56 PM, Nadav Rotem <nrotem at apple.com> wrote:
> > The code LGTM, I have a comment about the test case:
> >
> >>> +;RUN: opt -loop-vectorize -mcpu=core-i7-avx -debug -S < %s 2>&1 |
> FileCheck %s
> >
> > The triple is incorrect. It should be "corei7-avx". Second, this test is
> target dependent and should be inside the X86 directory.
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130204/dc8d8757/attachment.html>


More information about the llvm-commits mailing list