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>


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>
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to