sbenza added inline comments.
================
Comment at: lib/AST/ASTContext.cpp:1279
@@ -1282,1 +1278,3 @@
+const ASTContext::CXXMethodVector *
+ASTContext::overridden_methods(const CXXMethodDecl *Method) const {
llvm::DenseMap<const CXXMethodDecl *, CXXMethodVector>::const_iterator Pos
----------------
It would be simpler to return `ArrayRef<const CXXMethodDecl*>`.
That way the caller doesn't need to deal with a potential null pointer or with
the custom vector type.
http://reviews.llvm.org/D19324
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
