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

Alex Lorenz via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 29 02:40:57 PDT 2017


arphaman added a comment.

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?


Repository:
  rL LLVM

https://reviews.llvm.org/D34696





More information about the cfe-commits mailing list