aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
In D149276#4341552 <https://reviews.llvm.org/D149276#4341552>, @cor3ntin wrote: > @aaron.ballman this won;t get approve by core un > > In D149276#4302700 <https://reviews.llvm.org/D149276#4302700>, @aaron.ballman > wrote: > >> The changes generally LGTM, but: >> >>> in a way consistent with the proposed resolution to CWG1223. >> >> What are the chances that CWG changes their mind and picks a different >> direction? > > They were happy with the direction friday when we looked at it ( not the > wording though). I think we should go ahead, waiting until after issaquah > might be a bit too late (and if we are claiming auto(x) is implemented we > should merge that in 17) We don't have to wait for the wording to be final so long as the direction is in line with what you've done here, so LGTM! Thank you! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149276/new/ https://reviews.llvm.org/D149276 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
