[PATCH] D25349: [coroutines] Build fallthrough and set_exception statements.

Richard Smith via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 27 00:33:14 PDT 2016


rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D25349#571778, @GorNishanov wrote:

> @rsmith, I am wondering what were your thoughts on where to generate try { body } catch (...) { p.set_exception(std::exception()); }
>
> Would it be in SemaCoroutine.cpp?


Ideally, we'd avoid doing this desugaring at all, and just generate the correct code from CGCoroutine.cpp. Generally, we try to avoid desugaring in the AST whenever we can.


https://reviews.llvm.org/D25349





More information about the cfe-commits mailing list