[PATCH] D28111: Misc cleanups and simplifications for NewGVN.Mostly use a bit more idiomatic C++ where we can,so we can combine some things later.

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 25 14:47:27 PST 2016


Most of these changes are performance neutral.
The initial value initialization change avoids a walk of all values.

We could actually avoid that walk entirely through various machinations,
but it doesn't seem worth it yet.



On Sun, Dec 25, 2016 at 2:43 PM, Daniel Berlin via Phabricator <
reviews at reviews.llvm.org> wrote:

> dberlin created this revision.
> dberlin added a reviewer: davide.
> dberlin added a subscriber: llvm-commits.
>
> https://reviews.llvm.org/D28111
>
> Files:
>   include/llvm/Transforms/Scalar/GVNExpression.h
>   lib/Transforms/Scalar/NewGVN.cpp
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161225/773f0da2/attachment.html>


More information about the llvm-commits mailing list