aaron.ballman added a comment. In D114639#3161440 <https://reviews.llvm.org/D114639#3161440>, @erichkeane wrote:
> In D114639#3161362 <https://reviews.llvm.org/D114639#3161362>, @aaron.ballman > wrote: > >> In D114639#3161303 <https://reviews.llvm.org/D114639#3161303>, @erichkeane >> wrote: >> >>> IMO, if we're updating the MSVC versions, we should do the same for the >>> GCC/Clang/AppleClang versions too. For example, GCC 5.1 is from 2015, and >>> Clang 3.5 is from 2014. >> >> I'd be in favor of that; we could even see if we're ready to allow C++17 in >> the code base (which would be a pretty nice win for if init statements >> alone). However, it's also a bit orthogonal to this particular patch. Do you >> (or someone else) want to start an RFC? > > I believe the C++17 RFC was already on LLVM-dev last I saw? Ah, I had missed that, thanks! > That said, I commented because we typically try to update the 'minimum > versions' in lockstep best we can. Bumping these versions is particularly > arduous, so I'd be against trying to do it piece-meal like this, particularly > as we're updating the NEWEST requirement (that is, VS2017 is newer than Clang > 3.5/GCC5.1!). Fair point, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114639/new/ https://reviews.llvm.org/D114639 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
