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

[Silviu Baranga via llvm-commits]

Thanks! I've committed all changes in  r246540, r246541, r246542.

-Silviu

>> -----Original Message-----
>> From: Gerolf Hoflehner [mailto:ghoflehner at apple.com]
>> Sent: 31 August 2015 07:32
>> To: Silviu Baranga
>> Cc: Amara Emerson; llvm-commits at lists.llvm.org; Michael Zolotukhin;
>> Arnold; adam nemet
>> Subject: Re: [PATCH] D12149: [AArch64] Turn on by default interleaved
>> access vectorization
>>
>> 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_W6SiLK23
>> 42zz39teo6h5Mk5YZtJKpREVTCZg&s=zVLWYsnZl8mO3kPBb5a0GuXiGeDoj9f
>> c9bHOipzFTfA&e= ).
>> >
>> > Thanks,
>> > Silviu
>> >
>> >


-- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium.  Thank you.

ARM Limited, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No:  2557590
ARM Holdings plc, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No:  2548782