[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