
<html><head><style type='text/css'>p { margin: 0; }</style></head><body><div style='font-family: arial,helvetica,sans-serif; font-size: 10pt; color: #000000'><br><br><hr id="zwchr"><blockquote style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; padding-left: 5px; color: rgb(0, 0, 0); font-weight: normal; font-style: normal; text-decoration: none; font-family: Helvetica,Arial,sans-serif; font-size: 12pt;"><b>From: </b>"Daniel Berlin via llvm-commits" <llvm-commits@lists.llvm.org><br><b>To: </b>reviews+D26224+public+3b1fe8d104b1c17a@reviews.llvm.org<br><b>Cc: </b>"llvm-commits" <llvm-commits@lists.llvm.org>, "Jack Howarth" <howarth.mailing.lists@gmail.com>, "filcab+llvm phabricator" <filcab+llvm.phabricator@gmail.com>, "Davide Italiano" <dccitaliano@gmail.com>, deadalnix+llvmreview@gmail.com<br><b>Sent: </b>Thursday, December 8, 2016 12:41:58 PM<br><b>Subject: </b>Re: [PATCH] D26224: NewGVN<br><br><div id="DWT38204" dir="ltr">So, i kinda don't feel comfortable reviewing this, but if nobody else steps up i'll take a stab at some point.</div></blockquote>I'll also look at it.<br><blockquote style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; padding-left: 5px; color: rgb(0, 0, 0); font-weight: normal; font-style: normal; text-decoration: none; font-family: Helvetica,Arial,sans-serif; font-size: 12pt;"><div dir="ltr"><div>My reasoning is:<br><br><div>1. I wrote it, which means, outside of style issues, i already thought it was not horrible.</div><div>2. Having worked on it for years at this point, i understand it very very well, which means that the best hope of making it understandable to others is to have others read it and point out things they don't get that i would gloss over :)</div></div><div id="DWT38205">3. I'm on paternity leave, having had a newborn on saturday :)</div></div></blockquote>Congratulations :)<br><br> -Hal<br><blockquote style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; padding-left: 5px; color: rgb(0, 0, 0); font-weight: normal; font-style: normal; text-decoration: none; font-family: Helvetica,Arial,sans-serif; font-size: 12pt;"><div dir="ltr"><div></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 5, 2016 at 6:57 AM, Davide Italiano via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">davide added a comment.<br>

<br>

Monday morning ping.<br>

I would like to get the first cut in-tree soon (let's say, this week or the next) so we can iterate in-tree.<br>

<br>

<br>

<a href="https://reviews.llvm.org/D26224" rel="noreferrer" target="_blank">https://reviews.llvm.org/D26224</a><br>

<br>

<br>

<br>

</blockquote></div><br></div>

<br>_______________________________________________<br>llvm-commits mailing list<br>llvm-commits@lists.llvm.org<br>http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits<br></blockquote><br><br><br>-- <br><div><span name="x"></span>Hal Finkel<br>Lead, Compiler Technology and Programming Languages<br>Leadership Computing Facility<br>Argonne National Laboratory<span name="x"></span><br></div></div></body></html>