[PATCH] D31627: [coroutines] Skip over passthrough operator co_await
Gor Nishanov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon May 22 13:06:54 PDT 2017
GorNishanov added a comment.
In https://reviews.llvm.org/D31627#761320, @rsmith wrote:
> Does it still make sense for us to have a `UO_Coawait` at all? As I recall, the only purpose it ever had was to represent a dependent `co_await` expression that couldn't yet be resolved to a `CoawaitExpr`. But now we have (and need!) a separate `DependentCoawaitExpr` node to store unqualified lookup results, it seems that the `UnaryOperator` portion of the representation serves no purpose (and as seen in this patch, it's getting in the way). Can we remove it?
>
> Anyway, this change LGTM for now.
Thank you for the review!
I'll see if I can remove UO_Coawait in a separate patch
https://reviews.llvm.org/D31627
More information about the cfe-commits
mailing list