<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class=""></div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Jun 22, 2015, at 5:18 PM, Hal Finkel <<a href="mailto:hfinkel@anl.gov" class="">hfinkel@anl.gov</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">Hi Tyler,<br class=""><br class="">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.<br class=""><br class="">Thanks again,<br class="">Hal<br class=""><br class="">----- Original Message -----<br class=""><blockquote type="cite" class="">From: "Tyler Nowicki" <<a href="mailto:tnowicki@apple.com" class="">tnowicki@apple.com</a>><br class="">To: <a href="mailto:llvm-commits@cs.uiuc.edu" class="">llvm-commits@cs.uiuc.edu</a><br class="">Sent: Monday, June 22, 2015 12:52:34 PM<br class="">Subject: Re: [Patch][LoopVectorize] Renaming and Diagnostics for Loop<span class="Apple-tab-span" style="white-space:pre"> </span>Interleaving<br class=""><br class="">Hi,<br class=""><br class="">I would appreciate a review of these patches.<br class=""><br class="">Thanks,<br class=""><br class="">Tyler<br class=""><br class=""><blockquote type="cite" class="">On Jun 15, 2015, at 6:30 PM, Tyler Nowicki <<a href="mailto:tnowicki@apple.com" class="">tnowicki@apple.com</a>><br class="">wrote:<br class=""><br class="">Hi,<br class=""><br class="">Here is a renaming patch for names like unroll factor to names like<br class="">interleave count and a patch that improves the diagnostics for<br class="">interleaving so the user knows when they have explicitly disabled<br class="">interleaving and when it is not profitable. Includes test cases.<br class=""><br class="">Any comments are much appreciated!<br class=""><br class="">Tyler<br class=""><br class=""><0002-Rename-some-uses-of-unroll-to-interleave.patch><0003-Modify-diagnostic-messages-to-clearly-indicate-the-w.patch>_______________________________________________<br class="">llvm-commits mailing list<br class=""><a href="mailto:llvm-commits@cs.uiuc.edu" class="">llvm-commits@cs.uiuc.edu</a><br class="">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits<br class=""></blockquote><br class=""><br class="">_______________________________________________<br class="">llvm-commits mailing list<br class=""><a href="mailto:llvm-commits@cs.uiuc.edu" class="">llvm-commits@cs.uiuc.edu</a><br class="">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits<br class=""><br class=""></blockquote><br class="">-- <br class="">Hal Finkel<br class="">Assistant Computational Scientist<br class="">Leadership Computing Facility<br class="">Argonne National Laboratory<br class=""></div></blockquote></div><br class=""></div></body></html>