[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