avikivity wrote:

> > > @avikivity I'd also consider adding this to the test
> > > ```c++
> > > call([&](auto &t) -> coro_void {
> > >     co_await suspend_never{};
> > >     co_return use_void(p, t);
> > > });
> > > ```
> > 
> > 
> > Can you explain why? Does it exercise another path?
> 
> Just to keep it consistent with the issue description so that the test cover 
> both mentioned case, eventually we do reach the same path with `co_return` 
> and `co_await` and the current test is already good to prove the fix works.

Added.

https://github.com/llvm/llvm-project/pull/220234
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to