[PATCH] D26224: NewGVN

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 8 10:41:58 PST 2016


So, i kinda don't feel comfortable reviewing this, but if nobody else steps
up i'll take a stab at some point.
My reasoning is:

1. I wrote it, which means, outside of style issues, i already thought it
was not horrible.
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 :)
3. I'm on paternity leave, having had a newborn on saturday :)


On Mon, Dec 5, 2016 at 6:57 AM, Davide Italiano via Phabricator <
reviews at reviews.llvm.org> wrote:

> davide added a comment.
>
> Monday morning ping.
> 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.
>
>
> https://reviews.llvm.org/D26224
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161208/f70b137a/attachment.html>


More information about the llvm-commits mailing list