[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