<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="">Hi,<br class=""><br class="">Could I get a review of these patches?<br class=""><br class="">Thanks,<br class=""><br class="">Tyler<div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Jun 24, 2015, at 4:23 PM, Tyler Nowicki <<a href="mailto:tnowicki@apple.com" class="">tnowicki@apple.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><meta http-equiv="Content-Type" content="text/html charset=us-ascii" class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hi Hal,<div class=""><br class=""></div><div class="">Sorry about the 80 column limit. I put the main patch on phabricator, sorry if you received some spam related to this. I am not very familiar with it.</div><div class=""><br class=""></div><div class=""><a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D10714&d=AwMFAg&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=GRdW1eytZsy3Rbn1385c6J8p0965z9C8fGr133y6nyI&s=KWd2Xf15OK_S5MF0Uy-LS-0GBJBGOljhzIHOFNkFNYk&e=" class="">http://reviews.llvm.org/D10714</a></div><div class=""><br class=""></div><div class="">Here is the renaming patch again with the column limit fixed.</div><div class=""><br class=""></div><div class="">Thanks,</div><div class=""><br class=""></div><div class="">Tyler</div><div class=""><br class=""></div><div class=""></div></div><span id="cid:5374C427-AF07-4D83-8440-48375827468F@apple.com"><0002-Rename-some-uses-of-unroll-to-interleave.patch></span><meta http-equiv="Content-Type" content="text/html charset=us-ascii" class=""><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class=""></div><div class=""><br class=""><div class=""><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=""><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" class="">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><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=""><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" class="">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><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></div></div></blockquote></div><br class=""></div></body></html>