[PATCH] D22998: [coroutines] Part 4a: Coroutine Devirtualization: Lower coro.resume and coro.destroy.

David Majnemer via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 31 12:37:00 PDT 2016


majnemer added inline comments.

================
Comment at: lib/Transforms/Coroutines/CoroEarly.cpp:71
@@ +70,3 @@
+std::unique_ptr<Lowerer> Lowerer::createIfNeeded(Module& M) {
+  if (M.getNamedValue(CORO_RESUME_STR) || M.getNamedValue(CORO_DESTROY_STR))
+    return std::unique_ptr<Lowerer>(new Lowerer(M));
----------------
mehdi_amini wrote:
> majnemer wrote:
> > Could you use the llvm_coro_{destroy,resume} intrinsic IDs instead?
> Can we find if an intrinsic is declared/used in a Module using the ID?
Ah, maybe not...


Repository:
  rL LLVM

https://reviews.llvm.org/D22998





More information about the llvm-commits mailing list