[PATCH] D35012: [refactor] Add the AST source selection component

Alex Lorenz via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 18 05:44:57 PDT 2017


arphaman updated this revision to Diff 107067.
arphaman marked 7 inline comments as done.
arphaman added a comment.

- Address review comments.
- Remove the `Location` parameter and `ContainsSelectionPoint` enum value.
- Stop traversing early when a declaration that ends after the selection range was reached.


Repository:
  rL LLVM

https://reviews.llvm.org/D35012

Files:
  include/clang/Basic/SourceLocation.h
  include/clang/Basic/SourceManager.h
  include/clang/Tooling/Refactoring/ASTSelection.h
  lib/Tooling/Refactoring/ASTSelection.cpp
  lib/Tooling/Refactoring/CMakeLists.txt
  unittests/Tooling/ASTSelectionTest.cpp
  unittests/Tooling/CMakeLists.txt

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35012.107067.patch
Type: text/x-patch
Size: 32199 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170718/20dadf2a/attachment-0001.bin>


More information about the cfe-commits mailing list