[PATCH] D31492: Add `replace` interface with range in AtomicChange.

Eric Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 30 03:15:37 PDT 2017


ioeric added a comment.

In https://reviews.llvm.org/D31492#713908, @alexshap wrote:

> to avoid misunderstanding - are the tools like clang-rename, change-namespace and clang-reorder-fields supposed to use AtomicChange (via the methods insert, replace etc) ?
>  (i am asking just to make sure i understand correctly the intentions behind AtomicChange interface)


In the future, yes. We have a plan to redesign `ClangTool` interface and the refactoring infrastructure (e.g. to support multi-TU/codebase-wide refactoring) which would use `AtomicChange` interface in the user level.


https://reviews.llvm.org/D31492





More information about the cfe-commits mailing list