tree 8aeed14d9d6366a161b8a49484bb094fe4538683
parent d1c23f71840980fabc4ed38625989841afe0fd54
author Jakub Kuderski <kubakuderski@gmail.com> 1495418193 +0000
committer Jakub Kuderski <kubakuderski@gmail.com> 1495418193 +0000

[clang-tidy] Fix PR32896, PR33058: detect initializer lists in modernize-use-empalce

Summary:
The patch is backported from: r302281 to fix the 4.0.1 release blocker PR33058.
This patch fixes [[ https://bugs.llvm.org/show_bug.cgi?id=32896 | PR32896 ]].

The problem was that modernize-use-emplace incorrectly removed changed push_back into emplace_back, removing explicit constructor call with initializer list parameter, resulting in compiler error after applying fixits.
modernize-use-emplace used to check if matched constructor had InitListExpr, but didn't check against CXXStdInitializerListExpr.

Eg.

```
std::vector<std::vector<int>> v;
  v.push_back(std::vector<int>({1})); // --> v.emplace_back({1});
```

Reviewers: Prazek, alexfh, aaron.ballman

Reviewed By: Prazek, alexfh, aaron.ballman

Subscribers: xazax.hun, cfe-commits

Tags: #clang-tools-extra

Differential Revision: https://reviews.llvm.org/D32767


git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/branches/release_40@303524 91177308-0d34-0410-b5e6-96231b3b80d8