cor3ntin added inline comments.

================
Comment at: clang/test/SemaCXX/static-assert-cxx26.cpp:269
+                           //expected-error {{static assertion failed}} \
+                           // expected-note {{in call to 
'InvalidPtr{}.data()'}}
----------------
aaron.ballman wrote:
> One more test case to consider: do we properly handle dependent expressions 
> in the assert itself? e.g.,
> ```
> template <typename Ty>
> struct S {
>   static_assert(false, Ty{});
> };
> 
> struct Frobble {
>   constexpr int size() const { return 5; }
>   consterxpr const char *data() const { return "hello"; }
> };
> 
> S<Frobble> s; // Good, fails with "hello"
> S<int> t; // Incorrect
> ```
note that i did split the test in two to better show the appertainance of each 
diag.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154290

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to