skalinichev added a comment. Generally looks pretty good to me. Maybe it's worth to factor out duplicating code, but I guess it's not so important.
Also the index-file.cpp test failing for me: test/Index/index-file.cpp:57:11: error: expected string not found in input // CHECK: [indexDeclaration]: kind: c++-instance-method | name: foo | {{.*}} (deleted) | loc: 36:8 @jbcoe, can you take a look? http://reviews.llvm.org/D15469 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits