[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 mailing list