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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
