[PATCH] D40381: Parse concept definition

Saar Raz via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 24 05:28:25 PST 2017


saar.raz added inline comments.


================
Comment at: test/CXX/concepts-ts/expr/expr.prim/expr.prim.id/p3.cpp:1
+// RUN:  %clang_cc1 -std=c++1z -fconcepts-ts -fcxx-exceptions -x c++ -verify %s
+// expected-no-diagnostics
----------------
Rakete1111 wrote:
> There is no `-fconcepts-ts` flag AFAIK. Also, why `-fcxx-exceptions`, it's not needed here. The `-x c++` part too. It should also be `-std=c++2a`.
There is a -fconcepts-ts flag (clang -Xclang -fconcepts-ts), I believe now that there is no more Concepts TS the flag should be removed as well and all concepts-related code be put under C++2a ifs.


https://reviews.llvm.org/D40381





More information about the cfe-commits mailing list