[PATCH] D28774: [clang-move] Ignore using decls which are defined in macros.

2017-01-17 Thread Haojian Wu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL292207: [clang-move] Ignore using decls which are defined in macros. (authored by hokein). Changed prior to commit: https://reviews.llvm.org/D28774?vs=84639&id=84643#toc Repository: rL LLVM https://

[PATCH] D28774: [clang-move] Ignore using decls which are defined in macros.

2017-01-17 Thread Haojian Wu via Phabricator via cfe-commits
hokein updated this revision to Diff 84639. hokein marked 2 inline comments as done. hokein added a comment. Add more tests. https://reviews.llvm.org/D28774 Files: clang-move/ClangMove.cpp test/clang-move/Inputs/macro_helper_test.cpp test/clang-move/Inputs/macro_helper_test.h test/clang

[PATCH] D28774: [clang-move] Ignore using decls which are defined in macros.

2017-01-16 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. Lg. Comment at: test/clang-move/no-move-macro-helpers.cpp:1 +// RUN: mkdir -p %T/no-move-macro-helper +// RUN: cp %S/Inputs/macro_helper_test.h %T/no-move-macro-helper/macr

[PATCH] D28774: [clang-move] Ignore using decls which are defined in macros.

2017-01-16 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: ioeric. hokein added a subscriber: cfe-commits. Also ignore helpers which are defined in macro. Currently clang-move doesn't handle macro well enough, especiall for complex macros. This patch will ignore declarations in macros to make the beha