omtcyf0 added a subscriber: omtcyf0. omtcyf0 added a comment. This one doesn't fix it though.
It only deals with declarations of overridden functions. This is the test that still fails: / RUN: cat %s > %t.cpp // RUN: clang-rename -offset=161 -new-name=boo %t.cpp -i -- // RUN: sed 's,//.*,,' %t.cpp | FileCheck %s // XFAIL: * class A { public: virtual void foo() {} }; class B : public A { public: void foo() override {} // OK: B::foo overrides A::foo }; int main() { A a; a.foo(); B b; b.foo(); return 0; } // Use grep -FUbo 'Cla' <file> to get the correct offset of foo when changing // this file. http://reviews.llvm.org/D22237 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits