[PATCH] D23649: [ADT] Actually mutate the iterator VisitStack.back().second, not its copy.

[Daniel Berlin via llvm-commits]

At a glance , looks great to me.  Thanks so much for taking the time to do
this right

On Fri, Aug 19, 2016, 6:30 PM Tim Shen via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> timshen updated this revision to Diff 68756.
> timshen marked an inline comment as done.
> timshen added a comment.
>
> Added a testcase.
>
>
> https://reviews.llvm.org/D23649
>
> Files:
>   include/llvm/ADT/DepthFirstIterator.h
>   unittests/ADT/CMakeLists.txt
>   unittests/ADT/DepthFirstIteratorTest.cpp
>   unittests/ADT/SCCIteratorTest.cpp
>   unittests/ADT/TestGraph.h
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160821/d0abc456/attachment.html>