[PATCH] D59376: [LibTooling] Add Transformer, a library for source-to-source transformations.

Yitzhak Mandelbaum via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 4 07:02:51 PDT 2019


ymandel added a comment.

In D59376#1454768 <https://reviews.llvm.org/D59376#1454768>, @ilya-biryukov wrote:

> Per @ioeric's suggestion: why not move the helper into `Tooling/Refactoring/ExtendedRange.h`?
>  If it's in `ToolingRefactoring`, both stencil and transformer can access it.
>
> For external users, a dependency on either `ToolingCore` or `ToolingRefactoring` should be fine, since they're fine with a dependency on `Tooling` already.


This sounds perfect. Can I do the same for the future additions -- small, focused libraries for each group of functions? I just want to be sure that we don't regret the name "ExtendedRange" when I need to add the next batch.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59376/new/

https://reviews.llvm.org/D59376





More information about the cfe-commits mailing list