[PATCH] D51950: [clangtidy] Remove old copy of ExprMutationAnalyzer

2018-09-11 Thread Shuai Wang via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rCTE342006: [clangtidy] Remove old copy of ExprMutationAnalyzer (authored by shuaiwang, committed by ). Changed prior to commit: https://reviews.llvm.org/D51950?vs=164977&id=164991#toc Repository: rCTE

[PATCH] D51950: [clangtidy] Remove old copy of ExprMutationAnalyzer

2018-09-11 Thread Jonas Toth via Phabricator via cfe-commits
JonasToth added a comment. LGTM (not owner, but alexfh was not involved in the ExprMutAnalyzer so far, i think he has no concerns with moving that around as it is fairly new as well) Am 11.09.2018 um 23:44 schrieb George Karpenkov via Phabricator: > george.karpenkov added a comment. > > Looks g

[PATCH] D51950: [clangtidy] Remove old copy of ExprMutationAnalyzer

2018-09-11 Thread George Karpenkov via Phabricator via cfe-commits
george.karpenkov added a comment. Looks good to me, but I'm not the code owner. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D51950 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listi

[PATCH] D51950: [clangtidy] Remove old copy of ExprMutationAnalyzer

2018-09-11 Thread Shuai Wang via Phabricator via cfe-commits
shuaiwang created this revision. shuaiwang added reviewers: george.karpenkov, JonasToth. Herald added subscribers: cfe-commits, Szelethus, a.sidorin, mgorny. This is 2/2 of moving ExprMutationAnalyzer from clangtidy to clang/Analysis. ExprMutationAnalyzer is moved to clang/Analysis in https://rev