[PATCH] D146187: [docs] Update the status for coroutines
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Mar 16 05:01:19 PDT 2023
aaron.ballman added a comment.
Should we also be updating InitPreprocessor.cpp at the same time, for non-Windows targets? I see we define:
// TS features.
if (LangOpts.Coroutines)
Builder.defineMacro("__cpp_coroutines", "201703L");
but we don't define `__cpp_impl_coroutine`: http://eel.is/c++draft/tab:cpp.predefined.ft
And this should definitely come with a release note so users know about the change in status.
================
Comment at: clang/www/cxx_status.html:1225
<details><summary>Partial</summary>
- The optimizer does not yet handle TLS with
- <tt>__attribute__((const))</tt> 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.
+ It's supported fully everywhere but on Windows targets.
</details></td>
----------------
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146187/new/
https://reviews.llvm.org/D146187
More information about the cfe-commits
mailing list