[PATCH] D38882: [clang-rename] Add function unit tests.
Haojian Wu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 16 02:39:42 PDT 2017
hokein added a comment.
Sorry, I thought I have run all the tests. The previous version broke one test in RenameClass. We still need the workaround for the `std::function<void(T)>` special case. I added it back, and also added a corresponding test case for renaming function, but it is currently not supported and disabled, I think it is fine now, as it is a rare case IMO.
Please take a look again.
https://reviews.llvm.org/D38882
More information about the cfe-commits
mailing list