[all-commits] [llvm/llvm-project] ff2743: [libTooling] In Transformer, allow atomic changes ...

Yitzhak Mandelbaum via All-commits all-commits at lists.llvm.org
Tue May 26 06:23:21 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: ff2743bf047deac7ef6cc6c3efd30ff05e55b2ad
      https://github.com/llvm/llvm-project/commit/ff2743bf047deac7ef6cc6c3efd30ff05e55b2ad
  Author: Yitzhak Mandelbaum <yitzhakm at google.com>
  Date:   2020-05-26 (Tue, 26 May 2020)

  Changed paths:
    M clang/lib/Tooling/Transformer/Transformer.cpp
    M clang/unittests/Tooling/TransformerTest.cpp

  Log Message:
  -----------
  [libTooling] In Transformer, allow atomic changes to span multiple files.

Summary:
Currently, all changes returned by a single application of a rule must fit in
one atomic change and therefore must apply to one file. However, there are
patterns in which a single rule will want to modify multiple files; for example,
a header and implementation to change a declaration and its definition. This
patch relaxes Transformer, libTooling's interpreter of RewriteRules, to support
multiple changes.

Reviewers: gribozavr

Subscribers: mgrang, jfb, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D80239




More information about the All-commits mailing list