[PATCH] D25879: [coroutines] Add allocation and deallocation substatements.

[Gor Nishanov via llvm-commits]

GorNishanov marked 2 inline comments as done.
GorNishanov added inline comments.


================
Comment at: lib/CodeGen/CGCoroutine.cpp:71-72
+  if (!createCoroData(*this, CurCoro, CoroId, nullptr)) {
+    // User inserted __builtin_coro_id by hand. Should not try to emit anything.
+    return;
+  }
----------------
rsmith wrote:
> Can this really happen? The `CoroutineBodyStmt` should be the first thing we emitted into a new function, so it seems like we shouldn't have pre-existing coro data yet.
Yep. You are right! I took it out and made createCoroData return void, since nobody cares about its result anymore.


https://reviews.llvm.org/D25879