This patch enables lexing of concept and requires as keywords.
Further changes which add messages for future keyword compat are to
follow.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
This looks fine to me. Just one question: since we are setting -std=c++14,
is there no define for -DCXX14 instead of having to set -DCXX11? If this is
clang policy, then LGTM, otherwise, why are we not adding a define for
C++14?
Comment Actions
In the context of this test file, CXX14 would be redundant because there are no new keywords introduced through C++14. The corresponding implementation of keywords in the compiler does not call out C++14 apart from C++11. This change is consistent with that.