[PATCH] D41852: [clang-tidy] Don't generate fix for argument constructed from std::initializer_list.
Haojian Wu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 19 01:38:58 PST 2018
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
More information about the cfe-commits
mailing list