[PATCH] D24380: [migrate-tool] Framework for a codebase-dependent migration tool.

Eric Liu via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 9 11:20:37 PDT 2016


ioeric added a comment.

In https://reviews.llvm.org/D24380#538434, @Eugene.Zelenko wrote:

> I think will be good idea to await clang-refactor and merge code there.


This tool is not exactly a clang-tool; it is a framework that invokes multiple clang refactoring tool and also manipulates build rules. Also, I don't think two on-going projects should await on one another. If this tool fits into clang-refactor when it's actually there, I'd be happy to merge it into clang-refactor.

> Please run Include What You Use. Code use a lot of containers and will be good to include them explicitly.


Thanks for the tip!


https://reviews.llvm.org/D24380





More information about the cfe-commits mailing list