[PATCH] D146187: [docs] Update the status for coroutines

Corentin Jabot via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 17 05:35:17 PDT 2023


cor3ntin added a comment.

In D146187#4201905 <https://reviews.llvm.org/D146187#4201905>, @aaron.ballman wrote:

> In D146187#4201223 <https://reviews.llvm.org/D146187#4201223>, @ChuanqiXu wrote:
>
>>> but we don't define __cpp_­impl_­coroutine: http://eel.is/c++draft/tab:cpp.predefined.ft
>>
>> We defined `__cpp_­impl_­coroutine`.  And `__cpp_coroutines` is for Coroutines TS. I forgot to remove this too. I'll handle it in a separate patch.
>
> Thanks for handling the TS bits, but what am I misunderstanding here: https://godbolt.org/z/KdrM713r5 ?

Hahaha.
For some reason - i should ask why to Tim Song one day - the html version of the standard has weird injected non breaking spaces after underscores, you got bit by that
https://godbolt.org/z/1ceKfbTPc


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D146187/new/

https://reviews.llvm.org/D146187



More information about the cfe-commits mailing list