Tyker updated this revision to Diff 219205.
Tyker retitled this revision from "[C++20] Add consteval-specifique semantic"
to "[C++20] Add consteval-specifique semantic for functions".
Tyker added a comment.
Herald added a subscriber: mgrang.
I narrowed the patch because it was getting quite big.
Tyker marked 2 inline comments as done.
Tyker added a comment.
sorry i didn't realize the full complexity of immediate invocations.
i am working on a patch fixing issues.
Comment at: clang/lib/Sema/SemaExpr.cpp:5761-5762
// in ArgExprs.
- if ((FDecl =
-
rsmith added inline comments.
Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:2317
+def err_invalid_consteval_take_address : Error<
+ "cannot take address of consteval function %0 in non-constexpr context">;
+def err_consteval_address_accessible : Error<
Tyker updated this revision to Diff 217615.
Tyker added a comment.
Rebased
@rsmith Ping
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63960/new/
https://reviews.llvm.org/D63960
Files:
clang/include/clang/Basic/DiagnosticSemaKinds.td
clang/include/clang/Sema/Sema.h
Tyker added a comment.
ping @rsmith
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63960/new/
https://reviews.llvm.org/D63960
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
Tyker added a comment.
@rsmith ping
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63960/new/
https://reviews.llvm.org/D63960
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
Tyker added a comment.
ping @rsmith
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63960/new/
https://reviews.llvm.org/D63960
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
Tyker created this revision.
Tyker added a reviewer: rsmith.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Changes:
- Calls to consteval function and constructors are not evaluated as soon as
they are reached.
- Add diagnostic for taking address of a consteval function
