[PATCH] [Concepts] lex keywords: concept and requires

Hubert Tong hubert.reinterpretcast at gmail.com
Thu Jun 4 08:49:15 PDT 2015


In http://reviews.llvm.org/D10233#183453, @fraggamuffin wrote:

> 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?


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.


http://reviews.llvm.org/D10233

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list