ec117158a3
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. 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. So the existing codes wouldn't be break after update compiler. And in case the compiler found std::coroutine_traits and std::experimental::coroutine_traits at the same time, it would emit an error for it. The support for looking up std::experimental::coroutine_traits would be removed in Clang16. Reviewed By: lxfind, Quuxplusone Differential Revision: https://reviews.llvm.org/D108696
26 lines
615 B
C++
26 lines
615 B
C++
// RUN: %clang_cc1 -triple x86_64-pc-windows-msvc18.0.0 -emit-llvm %s -o - -std=c++20 -disable-llvm-passes | FileCheck %s
|
|
struct no_suspend {
|
|
bool await_ready() { return true; }
|
|
template <typename F> void await_suspend(F) {}
|
|
void await_resume() {}
|
|
};
|
|
|
|
struct A {
|
|
no_suspend operator co_await() { return {}; }
|
|
};
|
|
|
|
struct B {};
|
|
|
|
no_suspend operator co_await(B const&) { return {}; }
|
|
|
|
// CHECK-LABEL: f(
|
|
extern "C" void f() {
|
|
A a;
|
|
B b;
|
|
// CHECK: call void @"??__LA@@QEAA?AUno_suspend@@XZ"(
|
|
a.operator co_await();
|
|
// CHECK-NEXT: call i8 @"??__L@YA?AUno_suspend@@AEBUB@@@Z"(
|
|
operator co_await(b);
|
|
}
|
|
|