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/ _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
