EricWF added a comment.

In https://reviews.llvm.org/D30776#697258, @GorNishanov wrote:

> In https://reviews.llvm.org/D30776#697233, @EricWF wrote:
>
> > Good to know. I'll update this tomorrow.
>
>
> Well, that is just the thought. Possibly we can check for the types of 
> await_ready and await_suspend in BuildResolvedCoawaitExpr. Then, we don't 
> have to keep the vector of await/yield-expressions.


I guess it depends on when we want the diagnostics to be issued. And I think 
emitting them as they are built makes the most sense. If we attempt to issue 
the diagnostics at the end of the body it may be too late.

Do you still want me to put the vector back in now or when the need actually 
arises?


https://reviews.llvm.org/D30776



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to