[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