[PATCH] D108697: [libc++] Remove <experimental/coroutine>

Wed Jan 11 04:36:44 PST 2023

aaron.ballman added a comment.

In D108697#4042119 <https://reviews.llvm.org/D108697#4042119>, @ChuanqiXu wrote:

>> To me, that should apply to command line flags as well as header files because removing either one causes a stop-the-world problem.
>
> It looks not bad to me to remove the header files first and  keep `-fcoroutines-ts` flag a little bit longer. Do I misunderstand anything?

If the user's source code was looking for that header file, they now get a *fatal* error instead of a regular error (so all further compilation stops for that TU): https://godbolt.org/z/zzYEc46fW -- I don't see what value the command line flag adds without the header file, so maybe I'm missing something?

>> (Speaking of the command line flag deprecation warning, is that something you're working on @ChuanqiXu? If not, I can put something together pretty quickly before the Clang 16 branch.)
>
> I haven't started to work on it. I just saw no one else is assigned and it is going to be branched. So that I assign myself to offer some labor help. Feel free to take it you would love to : )

I'd appreciate it if you can tackle it if you have the chance in the next week or so. I'm still trying to get caught up on reviews (just in time for the C standards meetings in a a few weeks, lol).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108697