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

[Chuanqi Xu via Phabricator via cfe-commits]

ChuanqiXu created this revision.
ChuanqiXu added reviewers: aaron.ballman, jyknight.
ChuanqiXu added a project: clang.
ChuanqiXu requested review of this revision.
Herald added a subscriber: cfe-commits.

See https://lists.llvm.org/pipermail/cfe-dev/2021-December/069608.html. We couldn't mark coroutine as done now due to there to critical bugs. And as Aaron suggests, it would be better to give the reason in the docs.


Repository:
  rG LLVM Github Monorepo

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,7 @@
     <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">Partial <a href="#CoroPartialReason">(12)</a></td>
     </tr>
     <!-- Cologne 2019 papers -->
     <tr>
@@ -1263,6 +1263,12 @@
 <tt>-std=c++20</tt>, but can be enabled with <tt>-fchar8_t</tt>.
 </span>
 </p>
+<p>
+<span id="CoroPartialReason">(12): The optimizer couldn't handle TLS with
+  `__attribute__((const))` attribute correctly. It implies that there would
+  be problems if the coroutine is possible to resume on a different thread.
+</span>
+</p>
 </details>
 
 <h2 id="cxx23">C++2b implementation status</h2>


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