[PATCH] D28934: Write a new SSAUpdater

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 24 16:34:38 PST 2017


On Tue, Jan 24, 2017 at 4:13 PM, Davide Italiano via Phabricator <
reviews at reviews.llvm.org> wrote:

> davide added inline comments.
>
>
> ================
> Comment at: lib/Transforms/Utils/SSAUpdaterNew.cpp:197
> +  SmallPtrSet<Value *, 8> InComponent;
> +  DenseMap<Value *, unsigned int> Root;
> +  SmallPtrSet<Value *, 8> PHISet;
> ----------------
> This seems to be more `PhiToDFSNum` than `Root`, unless I'm missing
> something.
>
> Short note:
I copied the paper's name for it, so that the algorithm matches the paper,
happy to change it.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170124/606459ef/attachment.html>


More information about the llvm-commits mailing list