ilya-biryukov added inline comments.
================ Comment at: test/clang-tidy/Inputs/modernize-smart-ptr/initializer_list.h:30 vector(initializer_list<_E> init); + ~vector(); }; ---------------- hokein wrote: > ilya-biryukov wrote: > > Why do we need to add this destructor in this patch? > Yeah, we do need it to reproduce the issue in real world. The AST is > different with/without the destructor. Because without destructor `CXXBindTemporaryExpr` does not show up? Repository: rL LLVM https://reviews.llvm.org/D41852 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
