erichkeane added a subscriber: rsmith. erichkeane added a comment. In https://reviews.llvm.org/D38596#891603, @aaron.ballman wrote:
> The attribute and sema bits look good to me, but I agree that you might want > Richard's opinions before committing. Oh, definitely! I actually put this up for @echristo to take a look at as a sanity check. I realized it was a big patch so didn't want to spam a bunch of people. @rsmith was definitely on the list! I'll get this last const-auto fix (FWIW, I'm seeing the benefits now of for (X: Xs) being the same as for (const auto &&X : Xs)...) and add him and @rnk to start doing this as a 'final' review. Thank you very much @echristo, @hfinkel and @aaron.ballman ! https://reviews.llvm.org/D38596 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits