arphaman added a comment. If nothing uses `getCXXScopeSpecifier` right now we can't really test it with a clang or c-index-test regression test. A completion unit test could work here. I don't think we actually have existing completion unit tests though, so you would have to create one from scratch. But if `getCXXScopeSpecifier` will be used in a follow up patch maybe it will be easier to commit this without a test together with the followup patch?
https://reviews.llvm.org/D40563 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits