hokein added a comment. In https://reviews.llvm.org/D49658#1171410, @sammccall wrote:
> Mostly LG. > > I think we can simplify in a couple of places, and you should decide if this > patch is *implementing* the new index operation or not. (Currently it > implements it for 1/3 implementations, which doesn't seem that useful). Implementing all index operations in the patch would be end up with a large patch, I intend to split the patch as small as possible. Let's scope this patch to interface only. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D49658 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
