dblaikie added a comment. In D5980#3253378 <https://reviews.llvm.org/D5980#3253378>, @dblaikie wrote:
> Looks like @dexonsmith got this (putting the ownership in `Chain` & using > some iterator adapters to narrow the cleanup scope/fallout) in > rGa897f7cd40b12f693acbb04b81ef928244637aae > <https://reviews.llvm.org/rGa897f7cd40b12f693acbb04b81ef928244637aae>. Thanks > @dexonsmith! And just because I was looking at the code anyway, I migrated the `FirstVisitState` linked list to `unique_ptr` (thought about `forward_list` but given the handling of raw nodes, it seemed simpler not to do that) in rGbaa9b7c3c83ab6e4dfb15b8d7815a9958d5b5810 <https://reviews.llvm.org/rGbaa9b7c3c83ab6e4dfb15b8d7815a9958d5b5810> CHANGES SINCE LAST ACTION https://reviews.llvm.org/D5980/new/ https://reviews.llvm.org/D5980 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
