This is an archive of the discontinued LLVM Phabricator instance.

Add -std=c++17
ClosedPublic

Authored by hans on Aug 9 2017, 10:20 AM.

Details

Summary

As suggested on PR33912.

Trying to keep this small to make it easy to merge to the 5.0 branch. We can do a follow-up with more thorough renaming (diagnostic text, options, ids, etc.) later.

(For c++14 this was done in r215982, and I think a smaller patch for the 3.5 branch: http://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-20140818/113013.html)

Diff Detail

Event Timeline

hans created this revision.Aug 9 2017, 10:20 AM
rsmith accepted this revision.Aug 9 2017, 12:48 PM
rsmith added inline comments.
include/clang/Frontend/LangStandards.def
113

Missing "with amendments" compared to the other languages. We already implement some DRs against c++17.

This revision is now accepted and ready to land.Aug 9 2017, 12:48 PM
hans added inline comments.Aug 9 2017, 1:11 PM
include/clang/Frontend/LangStandards.def
113

Thanks! I wasn't sure about that part.

This revision was automatically updated to reflect the committed changes.