hokein added inline comments.
================ Comment at: test/clang-tidy/Inputs/modernize-smart-ptr/initializer_list.h:30 vector(initializer_list<_E> init); + ~vector(); }; ---------------- ilya-biryukov wrote: > 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? Yes. 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
