[PATCH] D115778: [docs] Give the reason why the support for coroutine is partial

Chuanqi Xu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 15 04:56:07 PST 2021


ChuanqiXu updated this revision to Diff 394529.
ChuanqiXu added a comment.

Address comments. Thanks for reviewing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115778

Files:
  clang/www/cxx_status.html


Index: clang/www/cxx_status.html
===================================================================
--- clang/www/cxx_status.html
+++ clang/www/cxx_status.html
@@ -1199,7 +1199,13 @@
     <tr>
       <td>Coroutines</td>
       <td><a href="https://wg21.link/p0912r5">P0912R5</a></td>
-      <td class="partial" align="center">Partial</td>
+      <td class="partial" align="center">
+        <details><summary>Partial</summary>
+          The optimizer does not yet handle TLS with
+  `__attribute__((const))` attribute correctly. There can be issues where the
+  coroutine may resume on a different thread. <br \>This feature requires further
+  analysis of the C++ Standard to determine what work is necessary for conformance.
+        </details></td>
     </tr>
     <!-- Cologne 2019 papers -->
     <tr>


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D115778.394529.patch
Type: text/x-patch
Size: 809 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20211215/449cb6ac/attachment.bin>


More information about the cfe-commits mailing list