[PATCH] D34696: [refactor] Move the core of clang-rename to lib/Tooling/Refactoring

Manuel Klimek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 29 03:25:36 PDT 2017

klimek added a comment.

In https://reviews.llvm.org/D34696#795020, @arphaman wrote:

> In https://reviews.llvm.org/D34696#793613, @klimek wrote:
> > The main thing I'm concerned about is having the main code in core, but having all tests in tools-extra. I think if we go that route we should also move clang-rename and its tests to core. Thoughts?
> Would it be better if I make a patch for the `clang-refactor` tool first and then update this patch to support some of clang-rename's functionality in clang-refactor and move the tests over? Or should I just move all of clang-rename and its tests to clang's repository first and then focus on clang-refactor?

Given the end goal is clear, I'm fine with either :)



More information about the cfe-commits mailing list