[PATCH] Extend LoopVectorizationLegality::isConsecutivePtr to handle multiple level GEPs

[Quentin Colombet]

> On Jun 17, 2015, at 11:29 AM, Wei Mi <wmi at google.com> wrote:
> 
>> I assume that since you found that limitation, you have a test case that exposes it. Could you add the test case to the patch please?
> 
> 
> I add the testcase which is the same one as test/Transforms/InstCombine/gep-merge1.ll in http://reviews.llvm.org/D9865.

I do not understand. I thought that test case was to make sure that the geps inside the loop are well formed for the current loop vectorizer (i.e., without this patch).

Q.

> 
> Thanks,
> Wei.
> 
> 
> REPOSITORY
>  rL LLVM
> 
> http://reviews.llvm.org/D10281
> 
> Files:
>  lib/Transforms/Vectorize/LoopVectorize.cpp
>  test/Transforms/LoopVectorize/pr23580.ll
> 
> EMAIL PREFERENCES
>  http://reviews.llvm.org/settings/panel/emailpreferences/
> <D10281.27851.patch>