[PATCH] D141572: [C++] [Coroutines] Deprecates the '-fcoroutines-ts' flag

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added a reviewer: libc++.
aaron.ballman added a comment.

Thank you for working on this! I think the Debian CI failure is unrelated, but the libc++ one looks plausibly related:

  /home/libcxx-builder/.buildkite-agent/builds/e6f11fd202bc-1/llvm-project/libcxx-ci/build/generic-cxx03/include/c++/v1/module.modulemap:773:10: error: module 'std.coroutine' requires feature 'coroutines'
    module coroutine {
           ^
  /home/libcxx-builder/.buildkite-agent/builds/e6f11fd202bc-1/llvm-project/libcxx-ci/libcxx/test/libcxx/modules_include.sh.cpp:158:10: note: submodule of top-level module 'std' implicitly imported here
  #include <coroutine>
           ^
  1 error generated.

adding the libc++ reviewers for awareness.



================
Comment at: clang/docs/ReleaseNotes.rst:547
+- Clang now deprecates the ``-fcoroutines-ts`` flag and the flag will be
+  removed in LLVM17. Please use C++20 Coroutines instead.
 
----------------



================
Comment at: clang/include/clang/Basic/DiagnosticDriverKinds.td:630-633
+def warn_deperecated_fcoroutines_ts_flag : Warning<
+  "the \"-fcoroutines-ts\" flag is deprecated and it will be removed in LLVM17; "
+  "use C++20 Coroutines instead">,
+  InGroup<DeprecatedExperimentalCoroutine>;
----------------
Slight rewording for style.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141572