ilya-biryukov accepted this revision. ilya-biryukov added a comment. This revision is now accepted and ready to land.
LGTM modulo a change to the 'initializer_list.h'. Do we really need it for this patch? ================ Comment at: test/clang-tidy/Inputs/modernize-smart-ptr/initializer_list.h:30 vector(initializer_list<_E> init); + ~vector(); }; ---------------- Why do we need to add this destructor in this patch? Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D41852 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
