[PATCH] D36574: [refactor] add clang-refactor tool with initial testing support and local-rename action

[Alex Lorenz via Phabricator via cfe-commits]

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.


Repository:
  rL LLVM

https://reviews.llvm.org/D36574