[PATCH] D54399: Move ExprMutationAnalyzer to Tooling/Analysis (1/3)

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Feb 9 23:19:37 PST 2019


lebedev.ri added a comment.

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..)


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