================
@@ -1198,6 +1198,11 @@ void Sema::CheckCompletedCoroutineBody(FunctionDecl *FD, 
Stmt *&Body) {
   if (FD->hasAttr<AlwaysInlineAttr>())
     Diag(FD->getLocation(), diag::warn_always_inline_coroutine);
 
+  // We don't allow use of VLAs within a coroutine, so diagnose if we've seen
----------------
ChuanqiXu9 wrote:

```suggestion
  // VLAs are not allowed within a coroutine, so diagnose if we've seen
```

nit: it is not us (clang/LLVM) who decide not to support VLA in coroutines. But 
C++20 coroutines itself doesn't support VLA by its design naturally.

https://github.com/llvm/llvm-project/pull/70341
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to