[PATCH] D34329: [clang-diff] Initial implementation.

Johannes Altmanninger via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 27 13:26:01 PDT 2017


johannes updated this revision to Diff 104249.
johannes added a comment.

- pass Options as a const reference instead of a pointer
- rename TreeComparator -> ASTDiff::Impl, rename Comparator -> DiffImpl
- move declaration of ASTDiff::Impl from the header to the source file so that Options does not need to be an opaque pointer
- rename SyntaxTreeI -> SyntaxTreeImpl


https://reviews.llvm.org/D34329

Files:
  include/clang/Tooling/ASTDiff/ASTDiff.h
  include/clang/Tooling/ASTDiff/ASTDiffInternal.h
  lib/Tooling/ASTDiff/ASTDiff.cpp
  lib/Tooling/ASTDiff/CMakeLists.txt
  lib/Tooling/CMakeLists.txt
  test/Tooling/clang-diff-basic.cpp
  tools/CMakeLists.txt
  tools/clang-diff/CMakeLists.txt
  tools/clang-diff/ClangDiff.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D34329.104249.patch
Type: text/x-patch
Size: 47092 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170627/5dd9a682/attachment-0001.bin>


More information about the cfe-commits mailing list