[PATCH] D23058: [clang-rename] add support for template instantiations
Kirill Bobyrev via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 3 01:38:56 PDT 2016
omtcyfz added inline comments.
Comment at: test/clang-rename/TemplateClassInstantiationFindByUninstantiatedType.cpp:2
@@ +1,3 @@
+// RUN: cat %s > %t.cpp
+// RUN: clang-rename -offset=440 -new-name=Bar %t.cpp -i --
+// RUN: sed 's,//.*,,' %t.cpp | FileCheck %s
> This test looks exactly as the one above except for the offset. Instead of duplicating tests, we can run clang-rename multiple times and use different -check-prefix with FileCheck. In this case moving the RUN lines to the bottom makes even more sense.
> Also, instructions for updating -offset are incorrect, if you have multiple different offsets.
Will it work if I add multiple invocations while not adding different `-check-prefix`? The replacements are the same across the runs.
More information about the cfe-commits