[PATCH] D23058: [clang-rename] add support for template instantiations

Kirill Bobyrev via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 3 01:46:42 PDT 2016


omtcyfz added inline comments.

================
Comment at: test/clang-rename/TemplateClassInstantiationFindByDeclaration.cpp:2
@@ -1,3 +1,3 @@
 // RUN: cat %s > %t.cpp
-// RUN: clang-rename -offset=287 -new-name=Bar %t.cpp -i --
+// RUN: clang-rename -offset=159 -new-name=Bar %t.cpp -i --
 // RUN: sed 's,//.*,,' %t.cpp | FileCheck %s
----------------
alexfh wrote:
> It makes sense to move RUN lines to the end of the file (and add -fno-delayed-template-parsing, if windows buildbots start complaining).
> It makes sense to move RUN lines to the end of the file

You mean for all tests?


https://reviews.llvm.org/D23058





More information about the cfe-commits mailing list