[PATCH] D54399: Move ExprMutationAnalyzer to Tooling/Analysis (1/3)
Shuai Wang via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Feb 10 12:03:57 PST 2019
shuaiwang added a comment.
In D54399#1391975 <https://reviews.llvm.org/D54399#1391975>, @lebedev.ri wrote:
> In D54399#1391935 <https://reviews.llvm.org/D54399#1391935>, @shuaiwang wrote:
> > Hi @rsmith what do you think of just get this in since Eric is not responding? Or do you think I should run certain test to verify?
> I'm having a déjà vu.
> Wasn't there already an attempt to have EMA in clang proper,
> and it was scrapped with reasoning along the lines of "it is slow by design"?
> If yes, what changed? (one might expect that to be in the description..)
This is moving to (a new dir) Tooling/Analysis with Tooling/ being a place not part of clang proper but can be depended by static analyzer.
Previously EMA was moved from clang-tidy to it's current location so that it could be used by static analyzer, but caused http://llvm.org/viewvc/llvm-project?view=revision&revision=342827
CHANGES SINCE LAST ACTION
More information about the cfe-commits