[PATCH] D43174: [clang-move] Fix the incorrect expansion end location.

2018-02-12 Thread Haojian Wu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL324886: [clang-move] Fix the incorrect expansion end location. (authored by hokein, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/D43174 Files:

[PATCH] D43174: [clang-move] Fix the incorrect expansion end location.

2018-02-12 Thread Eric Liu via Phabricator via cfe-commits
ioeric accepted this revision. ioeric added a comment. This revision is now accepted and ready to land. lgtm Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D43174 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.l

[PATCH] D43174: [clang-move] Fix the incorrect expansion end location.

2018-02-12 Thread Eric Liu via Phabricator via cfe-commits
ioeric added a comment. The fix is quite subtle and not obvious to tell from the test. Could you elaborate on the issue this fixes in the patch description? Thanks! Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D43174 ___ cfe-commi

[PATCH] D43174: [clang-move] Fix the incorrect expansion end location.

2018-02-12 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: ioeric. Herald added a subscriber: klimek. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D43174 Files: clang-move/ClangMove.cpp unittests/clang-move/ClangMoveTests.cpp Index: unittests/clang-move/ClangMoveTests.cpp