[Patch][LoopVectorize] Renaming and Diagnostics for Loop Interleaving

Hal Finkel hfinkel at anl.gov
Mon Jun 22 17:18:53 PDT 2015


Hi Tyler,

Please upload these patches to phabricator for review. I'd find that much easier. Also, you have many lines in these patches that violate the 80 column limit.

Thanks again,
Hal

----- Original Message -----
> From: "Tyler Nowicki" <tnowicki at apple.com>
> To: llvm-commits at cs.uiuc.edu
> Sent: Monday, June 22, 2015 12:52:34 PM
> Subject: Re: [Patch][LoopVectorize] Renaming and Diagnostics for Loop	Interleaving
> 
> Hi,
> 
> I would appreciate a review of these patches.
> 
> Thanks,
> 
> Tyler
> 
> > On Jun 15, 2015, at 6:30 PM, Tyler Nowicki <tnowicki at apple.com>
> > wrote:
> > 
> > Hi,
> > 
> > Here is a renaming patch for names like unroll factor to names like
> > interleave count and a patch that improves the diagnostics for
> > interleaving so the user knows when they have explicitly disabled
> > interleaving and when it is not profitable. Includes test cases.
> > 
> > Any comments are much appreciated!
> > 
> > Tyler
> > 
> > <0002-Rename-some-uses-of-unroll-to-interleave.patch><0003-Modify-diagnostic-messages-to-clearly-indicate-the-w.patch>_______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> 

-- 
Hal Finkel
Assistant Computational Scientist
Leadership Computing Facility
Argonne National Laboratory



More information about the llvm-commits mailing list