[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