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