[PATCH] D145362: [clang] Update test according to P1937

Mariya Podchishchaeva via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 6 11:33:21 PST 2023


Fznamznon added inline comments.


================
Comment at: clang/test/CXX/expr/expr.const/p8-2a.cpp:14
 
-// bad_assert_copyable<N> is not needed for constant evaluation
-// (and thus not instantiated)
+// consteval assert is not instantiated as well.
 template<typename T> consteval void assert_copyable() { T t; T t2 = t; }
----------------
shafik wrote:
> How about "Prior to P1937R2 consteval functions were evaluated even in otherwise unevaluated context, now this is well-formed"
> 
> and move it right above the "using check"
Ok, sure.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D145362/new/

https://reviews.llvm.org/D145362



More information about the cfe-commits mailing list