[PATCH] D12149: [AArch64] Turn on by default interleaved access vectorization

[Gerolf Hoflehner via llvm-commits]

Thanks, Silviu. It doesn’t look like there is reason for concern. LGTM.

Cheers
Gerolf

> On Aug 28, 2015, at 10:27 AM, Silviu Baranga <silviu.baranga at arm.com> wrote:
> 
> Hi Gerolf,
> 
>>> Thanks for following up and collecting the data. I haven’t seen any objection
>>> form the vectorizer gurus, so I assume any issue with the interleave code
>>> has been addressed already. I’m wondering about your compile-time data:
>>> j1 data per benchmark would provide a higher confidence in the data. I think
>>> that would provide a clearer picture whether there is an outlier to be aware
>>> of or not.
>>> 
> 
> I've attached the analysis with the build-times per spec benchmark to the review (https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D12149&d=BQIFaQ&c=eEvniauFctOgLOKGJOplqw&r=PlQHl7sshPU7FSzb4jGZyKbtJGJEL8ML0yYUKuWLs60&m=ybZIga_W6SiLK2342zz39teo6h5Mk5YZtJKpREVTCZg&s=zVLWYsnZl8mO3kPBb5a0GuXiGeDoj9fc9bHOipzFTfA&e= ).
> 
> Thanks,
> Silviu
> 
>