[PATCH] D22465: [clang-rename] introduce better symbol finding and add few more tests

Kirill Bobyrev via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 22 06:33:02 PDT 2016


omtcyfz added inline comments.

================
Comment at: test/clang-rename/ComplicatedClassType.cpp:1-6
@@ -1,6 +1,7 @@
-// Unsupported test.
 // RUN: cat %s > %t.cpp
-// FIXME: This test contains very simple constructions likely to be seen in any
-// project and therefore passing this test is a slight sign of success.
-// Currently, the test fails badly.
+// RUN: clang-rename -offset=220 -new-name=Bar %t.cpp -i --
+// RUN: sed 's,//.*,,' %t.cpp | FileCheck %s
+
+// Forward declaration.
+class Foo;                            // CHECK: class Bar;
 
----------------
No problem :)


https://reviews.llvm.org/D22465





More information about the cfe-commits mailing list