[PATCH] D36574: [refactor] add clang-refactor tool with initial testing support and local-rename action
Alex Lorenz via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Sep 1 07:43:12 PDT 2017
arphaman added a comment.
In https://reviews.llvm.org/D36574#858763, @klimek wrote:
> One of my main concerns is still that I don't see the need for all the template magic yet :) Why doesn't everybody use the RefactoringResult we define here?
This refactoring result is only really useful for the test part of `clang-refactor` so you can compare two results.
The non-test part of `clang-refactor` and other clients don't really need an abstract interface for results as they will have different code for different results anyway.
More information about the cfe-commits