[PATCH] D108697: [Coroutines] [libcxx] Move coroutine components out from std::experimental namespace
Louis Dionne via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 20 14:13:19 PDT 2021
ldionne requested changes to this revision.
ldionne added a comment.
As I explained in D109433 <https://reviews.llvm.org/D109433>, we don't need to "move" the header. What we normally do is introduce the `<coroutine>` header that contains the C++20 conforming implementation without touching `<experimental/coroutine>`. Then, after 2 releases (LLVM 16), we remove `<experimental/coroutine>` altogether. I suggest that
1. D109433 <https://reviews.llvm.org/D109433> is used to check-in a conforming `<coroutine>` implementation, and
2. This patch is repurposed to remove `<experimental/coroutine>` altogether. Once that's the case, this patch can sleep for two releases and I'll rebase + merge it as soon as LLVM 15 is shipped (to make sure it makes it into LLVM 16).
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D108697/new/
https://reviews.llvm.org/D108697
More information about the llvm-commits
mailing list