hokein marked an inline comment as done.
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:
> 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.


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

Reply via email to