[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


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54399/new/

https://reviews.llvm.org/D54399





More information about the cfe-commits mailing list