[PATCH] D38882: [clang-rename] Add function unit tests.

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 13 06:57:12 PDT 2017


hokein created this revision.
Herald added subscribers: mgorny, klimek.

Also contain some fixes:

- Get rid of the "TranslationUnitDecl" special case in RenameClass, as

we switch to use USR instead of AST matcher to find symbols. A bonus
point is that now the test cases make more sense.

- Fix a false positive of renaming a using shadow function declaration.


https://reviews.llvm.org/D38882

Files:
  lib/Tooling/Refactoring/Rename/USRLocFinder.cpp
  unittests/Rename/CMakeLists.txt
  unittests/Rename/RenameClassTest.cpp
  unittests/Rename/RenameFunctionTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D38882.118903.patch
Type: text/x-patch
Size: 17059 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20171013/a63ba733/attachment-0001.bin>


More information about the cfe-commits mailing list