[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