[PATCH] D63892: [LibTooling] Extend `RewriteRule` with support for adding includes.
Dmitri Gribenko via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jun 28 06:48:06 PDT 2019
gribozavr accepted this revision.
gribozavr added inline comments.
This revision is now accepted and ready to land.
================
Comment at: clang/include/clang/Tooling/Refactoring/Transformer.h:123
TextGenerator Explanation;
+ // Include paths that to add to the file affected by this case. These are
+ // bundled with the `Case`, rather than the `RewriteRule`, because each case
----------------
s/that//
================
Comment at: clang/include/clang/Tooling/Refactoring/Transformer.h:152
+/// common use is assumed to be a rule with only one case.
+/// For example, to replace a function call, one could write:
+/// \code
----------------
The example looks incomplete to me because it does not call `addInclude`...
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63892/new/
https://reviews.llvm.org/D63892
More information about the cfe-commits
mailing list