[all-commits] [llvm/llvm-project] 2fbd25: [Coroutines] [Clang] Look up coroutine component i...

[Chuanqi Xu via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 2fbd254aa46b4934c88cf4bac5ad756471463862
      https://github.com/llvm/llvm-project/commit/2fbd254aa46b4934c88cf4bac5ad756471463862
  Author: Chuanqi Xu <yedeng.yd at linux.alibaba.com>
  Date:   2021-09-03 (Fri, 03 Sep 2021)

  Changed paths:
    M clang-tools-extra/test/clang-tidy/checkers/Inputs/readability-identifier-naming/system/coroutines.h
    M clang/docs/LanguageExtensions.rst
    M clang/include/clang/Basic/DiagnosticSemaKinds.td
    M clang/include/clang/Sema/Sema.h
    M clang/lib/Sema/SemaCoroutine.cpp
    M clang/test/AST/Inputs/std-coroutine.h
    M clang/test/AST/coroutine-locals-cleanup.cpp
    M clang/test/AST/coroutine-source-location-crash.cpp
    M clang/test/Analysis/more-dtors-cfg-output.cpp
    M clang/test/CodeGenCXX/ubsan-coroutines.cpp
    M clang/test/CodeGenCoroutines/Inputs/coroutine.h
    M clang/test/CodeGenCoroutines/coro-alloc.cpp
    M clang/test/CodeGenCoroutines/coro-always-inline.cpp
    M clang/test/CodeGenCoroutines/coro-await-domination.cpp
    M clang/test/CodeGenCoroutines/coro-await-resume-eh.cpp
    M clang/test/CodeGenCoroutines/coro-await.cpp
    M clang/test/CodeGenCoroutines/coro-cleanup.cpp
    M clang/test/CodeGenCoroutines/coro-dest-slot.cpp
    M clang/test/CodeGenCoroutines/coro-dwarf.cpp
    M clang/test/CodeGenCoroutines/coro-eh-cleanup.cpp
    M clang/test/CodeGenCoroutines/coro-gro-nrvo.cpp
    M clang/test/CodeGenCoroutines/coro-gro.cpp
    M clang/test/CodeGenCoroutines/coro-lambda.cpp
    M clang/test/CodeGenCoroutines/coro-newpm-pipeline.cpp
    M clang/test/CodeGenCoroutines/coro-params.cpp
    M clang/test/CodeGenCoroutines/coro-promise-dtor.cpp
    M clang/test/CodeGenCoroutines/coro-ret-void.cpp
    M clang/test/CodeGenCoroutines/coro-return-voidtype-initlist.cpp
    M clang/test/CodeGenCoroutines/coro-return.cpp
    M clang/test/CodeGenCoroutines/coro-symmetric-transfer-01.cpp
    M clang/test/CodeGenCoroutines/coro-symmetric-transfer-02.cpp
    M clang/test/CodeGenCoroutines/coro-unhandled-exception.cpp
    M clang/test/CoverageMapping/coroutine.cpp
    M clang/test/Index/coroutines.cpp
    M clang/test/PCH/coroutines.cpp
    M clang/test/SemaCXX/Inputs/std-coroutine.h
    M clang/test/SemaCXX/co_await-range-for.cpp
    M clang/test/SemaCXX/coreturn-eh.cpp
    M clang/test/SemaCXX/coreturn.cpp
    M clang/test/SemaCXX/coroutine-final-suspend-noexcept.cpp
    M clang/test/SemaCXX/coroutine-rvo.cpp
    M clang/test/SemaCXX/coroutine-seh.cpp
    M clang/test/SemaCXX/coroutine-traits-undefined-template.cpp
    M clang/test/SemaCXX/coroutine-unhandled_exception-warning.cpp
    M clang/test/SemaCXX/coroutine-uninitialized-warning-crash.cpp
    M clang/test/SemaCXX/coroutine_handle-addres-return-type.cpp
    A clang/test/SemaCXX/coroutines-exp-namespace.cpp
    M clang/test/SemaCXX/coroutines.cpp
    M clang/unittests/ASTMatchers/ASTMatchersTraversalTest.cpp

  Log Message:
  -----------
  [Coroutines] [Clang] Look up coroutine component in std namespace first

Summary: Now in libcxx and clang, all the coroutine components are
defined in std::experimental namespace.
And now the coroutine TS is merged into C++20. So in the working draft
like N4892, we could find the coroutine components is defined in std
namespace instead of std::experimental namespace.
And the coroutine support in clang seems to be relatively stable. So I
think it may be suitable to move the coroutine component into the
experiment namespace now.

But move the coroutine component into the std namespace may be an break
change. So I planned to split this change into two patch. One in clang
and other in libcxx.

This patch would make clang lookup coroutine_traits in std namespace
first. For the compatibility consideration, clang would lookup in
std::experimental namespace if it can't find definitions in std
namespace and emit a warning in this case. So the existing codes
wouldn't be break after update compiler.

Test Plan: check-clang, check-libcxx

Reviewed By: lxfind

Differential Revision: https://reviews.llvm.org/D108696